]> Creatis software - gdcm.git/blobdiff - Testing/TestWriteSimple.cxx
Fix mistypings
[gdcm.git] / Testing / TestWriteSimple.cxx
index 45a1769bba9a2e6a6fbdd619d5778dfc8481ac33..5e981eee707f219dd9ec66348f38f53f765bcfd4 100644 (file)
@@ -3,8 +3,8 @@
   Program:   gdcm
   Module:    $RCSfile: TestWriteSimple.cxx,v $
   Language:  C++
-  Date:      $Date: 2005/10/20 08:58:15 $
-  Version:   $Revision: 1.40 $
+  Date:      $Date: 2007/09/28 14:20:22 $
+  Version:   $Revision: 1.53 $
                                                                                 
   Copyright (c) CREATIS (Centre de Recherche et d'Applications en Traitement de
   l'Image). All rights reserved. See Doc/License.txt or
@@ -25,6 +25,8 @@
 #include "gdcmFile.h"
 #include "gdcmFileHelper.h"
 #include "gdcmDebug.h"
+#include "gdcmGlobal.h"
+#include "gdcmDictSet.h"
 
 #include <iostream>
 #include <sstream>
@@ -35,8 +37,8 @@ typedef struct
    int sizeY;         // Size Y of the image
    int sizeZ;         // Size Z of the image
    int components;    // Number of components for a pixel
-   int componentSize; // Component size (in bits : 8, 16)
-   int componentUse ; // Component size (in bits)
+   int componentSize; // Component size (in bits : 8, 16) // Bits Allocated
+   int componentUse ; // Component size (in bits)         // Bits Stored
    int sign;          // Sign of components
    char writeMode;    // Write mode
                       //  - 'a' : ACR
@@ -45,87 +47,110 @@ typedef struct
 } Image;
 
 Image Images [] = {
-   {256, 256, 1, 1, 8,  8,  0, 'a'},
-   {256, 256, 1, 1, 8,  8,  0, 'e'},
-   {256, 256, 1, 1, 8,  8,  0, 'i'},
-
-   {512, 256, 1, 1, 8,  8,  0, 'a'},
-   {512, 256, 1, 1, 8,  8,  0, 'e'},
-   {512, 256, 1, 1, 8,  8,  0, 'i'},
-
-   {256, 512, 1, 1, 8,  8,  0, 'a'},
-   {256, 512, 1, 1, 8,  8,  0, 'e'},
-   {256, 512, 1, 1, 8,  8,  0, 'i'},
-
-   {256, 512, 1, 1, 16, 16, 0, 'a'},
-   {256, 512, 1, 1, 16, 16, 0, 'e'},
-   {256, 512, 1, 1, 16, 16, 0, 'i'},
-   {256, 512, 1, 1, 16, 16, 0, 'a'},
-   {256, 512, 1, 1, 16, 16, 0, 'e'},
-   {256, 512, 1, 1, 16, 16, 0, 'i'},
-
-   {512, 256, 10, 1, 8, 8,  0, 'a'},
-   {512, 256, 10, 1, 8, 8,  0, 'e'},
-   {512, 256, 10, 1, 8, 8,  0, 'i'},
-   {512, 256, 10, 3, 8, 8,  0, 'a'},
-   {512, 256, 10, 3, 8, 8,  0, 'e'},
-   {512, 256, 10, 3, 8, 8,  0, 'i'},
-
-   {256, 256, 1, 1, 8,  8,  1, 'a'},
-   {256, 256, 1, 1, 8,  8,  1, 'e'},
-   {256, 256, 1, 1, 8,  8,  1, 'i'},
-
-   {512, 256, 1, 1, 8,  8,  1, 'a'},
-   {512, 256, 1, 1, 8,  8,  1, 'e'},
-   {512, 256, 1, 1, 8,  8,  1, 'i'},
-
-   {256, 512, 1, 1, 8,  8,  1, 'a'},
-   {256, 512, 1, 1, 8,  8,  1, 'e'},
-   {256, 512, 1, 1, 8,  8,  1, 'i'},
-
-   {256, 512, 1, 1, 16, 16, 1, 'a'},
-   {256, 512, 1, 1, 16, 16, 1, 'e'},
-   {256, 512, 1, 1, 16, 16, 1, 'i'},
-   {256, 512, 1, 1, 16, 16, 1, 'a'},
-   {256, 512, 1, 1, 16, 16, 1, 'e'},
-   {256, 512, 1, 1, 16, 16, 1, 'i'},
-
-   {512, 256, 10, 1, 8, 8,  1, 'a'},
-   {512, 256, 10, 1, 8, 8,  1, 'e'},
-   {512, 256, 10, 1, 8, 8,  1, 'i'},
-   {512, 256, 10, 3, 8, 8,  1, 'a'},
-   {512, 256, 10, 3, 8, 8,  1, 'e'},
-   {512, 256, 10, 3, 8, 8,  1, 'i'},
+// these ones are use to check further oddities.
+
+   {63, 127, 1, 1, 16,  8,  0 ,'e'},
+   {63, 127, 1, 1, 16,  9,  0 ,'e'},
+   {63, 127, 1, 1, 16, 10,  0 ,'e'},
+   {63, 127, 1, 1, 16, 11,  0 ,'e'},
+   {63, 127, 1, 1, 16, 12,  0 ,'e'},
+   {63, 127, 1, 1, 16, 13,  0 ,'e'},
+   {63, 127, 1, 1, 16, 14,  0 ,'e'},
+   {63, 127, 1, 1, 16, 15,  0 ,'e'}, 
+   {63, 127, 1, 1, 16, 16,  0 ,'e'},
+   
+   {63, 127, 1, 1, 32, 32,  0 ,'e'},   // Pixel VR should be OL?
+      
+   {128, 128, 1, 1, 8,  8,  0, 'e'},
+   {256, 128, 1, 1, 8,  8,  0, 'a'},
+   {128, 128, 1, 1, 8,  8,  0, 'i'},
+
+   {128, 128, 1, 1, 8,  8,  0, 'a'},    
+   {256, 128, 1, 1, 8,  8,  0, 'i'},
+     
+
+   {256, 128, 1, 1, 8,  8,  0, 'e'},
+   {128, 128, 1, 1, 16, 16, 0, 'e'},      
+   {128, 256, 1, 1, 16, 16, 0, 'e'},   
+   
+   {128, 256, 1, 1, 8,  8,  0, 'e'},
+   {128, 256, 1, 1, 8,  8,  0, 'i'},
+   {128, 256, 1, 1, 8,  8,  0, 'a'},
+   
+   {128, 256, 1, 1, 16, 16, 0, 'i'},
+   {128, 256, 1, 1, 16, 16, 0, 'i'},
+   {128, 256, 1, 1, 16, 16, 0, 'a'},
+   {128, 256, 1, 1, 16, 16, 0, 'a'},
+
+   {256, 128, 10, 1, 8, 8,  0, 'e'},
+   {256, 128, 10, 3, 8, 8,  0, 'e'},
+   {256, 128, 10, 3, 8, 8,  0, 'i'},
+   {256, 128, 10, 1, 8, 8,  0, 'i'},
+   {256, 128, 10, 1, 8, 8,  0, 'a'},
+   {256, 128, 10, 3, 8, 8,  0, 'a'},
+      
+   {128, 128, 1, 1, 8,  8,  1, 'e'},
+   {128, 128, 1, 1, 8,  8,  1, 'i'},
+   {128, 128, 1, 1, 8,  8,  1, 'a'},
+   
+   {256, 128, 1, 1, 8,  8,  1, 'e'},
+   {256, 128, 1, 1, 8,  8,  1, 'i'},
+   {256, 128, 1, 1, 8,  8,  1, 'a'},
+   
+   {128, 256, 1, 1, 8,  8,  1, 'a'},
+   {128, 256, 1, 1, 8,  8,  1, 'e'},
+   {128, 256, 1, 1, 8,  8,  1, 'i'},
+
+   {128, 256, 1, 1, 16, 16, 1, 'e'},
+   {128, 256, 1, 1, 16, 16, 1, 'e'},
+   {128, 256, 1, 1, 16, 16, 1, 'i'},
+   {128, 256, 1, 1, 16, 16, 1, 'i'},
+   {128, 256, 1, 1, 16, 16, 1, 'a'},
+   {128, 256, 1, 1, 16, 16, 1, 'a'},     
+
+   {256, 128, 10, 1, 8, 8,  1, 'e'},
+   {256, 128, 10, 1, 8, 8,  1, 'i'},
+   {256, 128, 10, 1, 8, 8,  1, 'a'},   
+   
+   {256, 128, 10, 3, 8, 8,  1, 'e'},
+   {256, 128, 10, 3, 8, 8,  1, 'i'},
+   {256, 128, 10, 3, 8, 8,  1, 'a'},
    {0,   0,   1,  1, 8, 8,  0, 'i'} // to find the end
 };
 
+
 const unsigned int MAX_NUMBER_OF_DIFFERENCE = 10;
 
 int WriteSimple(Image &img)
 {
+   
+   std::cout << "======================= WriteSimple =========(begin of processing current image)" << std::endl;
    std::ostringstream fileName;
    fileName.str("");
    fileName << "TestWriteSimple";
 
-// Step 1 : Create the header of the image
+// Step 1 : Create an empty FileHelper
 
    std::cout << "        1...";
-   gdcm::File *fileToBuild = new gdcm::File();
+   GDCM_NAME_SPACE::FileHelper *fileH = GDCM_NAME_SPACE::FileHelper::New();
+ //  Get the (empty) image header.  
+   GDCM_NAME_SPACE::File *fileToBuild = fileH->GetFile();
    std::ostringstream str;
 
    // Set the image size
    str.str("");
    str << img.sizeX;
-   fileToBuild->InsertEntryString(str.str(),0x0028,0x0011); // Columns
+   fileToBuild->InsertEntryString(str.str(),0x0028,0x0011,"US"); // Columns
    str.str("");
    str << img.sizeY;
-   fileToBuild->InsertEntryString(str.str(),0x0028,0x0010); // Rows
+   fileToBuild->InsertEntryString(str.str(),0x0028,0x0010,"US"); // Rows
 
    if(img.sizeZ>1)
    {
       str.str("");
       str << img.sizeZ;
-      fileToBuild->InsertEntryString(str.str(),0x0028,0x0008); // Number of Frames
+      fileToBuild->InsertEntryString(str.str(),0x0028,0x0008, "IS"); // Number of Frames
    }
 
    fileName << "-" << img.sizeX << "-" << img.sizeY << "-" << img.sizeZ;
@@ -133,43 +158,62 @@ int WriteSimple(Image &img)
    // Set the pixel type
    str.str("");
    str << img.componentSize;
-   fileToBuild->InsertEntryString(str.str(),0x0028,0x0100); // Bits Allocated
+   fileToBuild->InsertEntryString(str.str(),0x0028,0x0100,"US"); // Bits Allocated
 
    str.str("");
    str << img.componentUse;
-   fileToBuild->InsertEntryString(str.str(),0x0028,0x0101); // Bits Stored
+   fileToBuild->InsertEntryString(str.str(),0x0028,0x0101,"US"); // Bits Stored
 
    str.str("");
-   str << ( img.componentSize - 1 );
-   fileToBuild->InsertEntryString(str.str(),0x0028,0x0102); // High Bit
+   str << ( img.componentUse - 1 );
+   fileToBuild->InsertEntryString(str.str(),0x0028,0x0102,"US"); // High Bit
 
    // Set the pixel representation
    str.str("");
    str << img.sign;
-   fileToBuild->InsertEntryString(str.str(),0x0028,0x0103); // Pixel Representation
+   fileToBuild->InsertEntryString(str.str(),0x0028,0x0103,"US"); // Pixel Representation
 
    fileName << "-" << img.componentSize;
    if(img.sign == 0)
       fileName << "U";
    else
       fileName << "S";
-   
+      
+   fileName << "-" << img.componentSize << "-" << img.componentUse;
+   fileToBuild->InsertEntryString("0",0x0008,0x0000,"UL"); // Should be removed
+                                                            // except for ACR 
    switch (img.writeMode)
    {
       case 'a' :
-         fileName << ".ACR";  break; 
+         fileName << ".ACR";
+         break; 
       case 'e' :
-         fileName << ".EXPL"; break; 
+         fileName << ".EXPL"; 
+         break; 
       case 'i' :
-         fileName << ".IMPL"; break;
+         fileName << ".IMPL"; 
+         break;
    } 
 
+   if(img.componentSize == 32)
+   {
+      // Create a Private DataElement; VR =OL
+      fileToBuild->InsertEntryString("gdcm test OL",0x0009,0x0010,"LO");
+      uint32_t binArea[4];
+      binArea[0] = 0x01234567;
+      binArea[1] = 0x89abcdef;      
+      binArea[2] = 0x2468ace0;
+      binArea[2] = 0xfdb97531;
+      fileToBuild->InsertEntryBinArea((uint8_t *)binArea,16,0x0009,0x0100,"OL");              
+   }
+   
+   
    std::cout << "[" << fileName.str() << "]...";
-
    // Set the samples per pixel
    str.str("");
    str << img.components;
-   fileToBuild->InsertEntryString(str.str(),0x0028,0x0002); // Samples per Pixel
+   fileToBuild->InsertEntryString(str.str(),0x0028,0x0002,"US"); // Samples per Pixel
 
 // Step 2 : Create the output image
    std::cout << "2...";
@@ -202,15 +246,14 @@ int WriteSimple(Image &img)
       }
    }
 
-// Step 3 : Create the file of the image
+// Step 3 : Set the image Pixel Data
    std::cout << "3...";
-   gdcm::FileHelper *fileH = new gdcm::FileHelper(fileToBuild);
    fileH->SetImageData(imageData,size);
 
 // Step 4 : Set the writting mode and write the image
-   std::cout << "4...";
 
    fileH->SetWriteModeToRaw();
+   std::cout << "4'...";   
    switch (img.writeMode)
    {
       case 'a' : // Write an ACR file
@@ -229,52 +272,62 @@ int WriteSimple(Image &img)
          std::cout << "Failed for [" << fileName.str() << "]\n"
                    << "        Write mode '"<<img.writeMode<<"' is undefined\n";
 
-         delete fileH;
-         delete fileToBuild;
+         fileH->Delete();
          delete[] imageData;
          return 1;
    }
 
+   std::cout << std::endl;
+   //fileToBuild->Print();
+
    if( !fileH->Write(fileName.str()) )
    {
       std::cout << "Failed for [" << fileName.str() << "]\n"
-                << "           File is unwrittable\n";
+                << "           File is unwrittable" << std::endl;
+
+      fileH->Delete();
 
-      delete fileH;
-      delete fileToBuild;
       delete[] imageData;
       return 1;
    }
 
 // Step 5 : Read the written image
-   std::cout << "5...";
+   std::cout << "5..." << std::endl;
    // old form.
-   //gdcm::FileHelper *reread = new gdcm::FileHelper( fileName.str() );
+   //GDCM_NAME_SPACE::FileHelper *reread = new GDCM_NAME_SPACE::FileHelper( fileName.str() );
    // Better use :
-   gdcm::FileHelper *reread = new gdcm::FileHelper( );
-   reread->SetFileName( fileName.str() );
-   reread->SetLoadMode(0); // Load everything
-                           // Other possible values are 
-                           //              gdcm::LD_ALL, 
-                           //              gdcm::LD_NOSEQ, 
-                           //              gdcm::LD_NOSHADOW,
-                           //              gdcm::LD_NOSEQ|gdcm::LD_NOSHADOW, 
-                           //              gdcm::LD_NOSHADOWSEQ
-   reread->Load();
+   GDCM_NAME_SPACE::File *f = GDCM_NAME_SPACE::File::New( );
+   f->SetLoadMode(GDCM_NAME_SPACE::LD_ALL);
+   f->SetFileName( fileName.str() );
+   
+   //reread->SetFileName( fileName.str() );
+   //reread->SetLoadMode(GDCM_NAME_SPACE::LD_ALL); // Load everything
+                           // Possible values are 
+                           //              GDCM_NAME_SPACE::LD_ALL, 
+                           //              GDCM_NAME_SPACE::LD_NOSEQ, 
+                           //              GDCM_NAME_SPACE::LD_NOSHADOW,
+                           //              GDCM_NAME_SPACE::LD_NOSEQ|GDCM_NAME_SPACE::LD_NOSHADOW, 
+                           //              GDCM_NAME_SPACE::LD_NOSHADOWSEQ
+   
+   f->Load();
+  // reread->Load();
+   GDCM_NAME_SPACE::FileHelper *reread = GDCM_NAME_SPACE::FileHelper::New( f );  
+
+   //reread->Print();
 
    if( !reread->GetFile()->IsReadable() )
    {
       std::cerr << "Failed" << std::endl
                 << "Could not read written image : " << fileName.str() << std::endl;
-      delete fileToBuild;
-      delete fileH;
-      delete reread;
+      fileToBuild->Delete();
+      fileH->Delete();
+      reread->Delete();
       delete[] imageData;
       return 1;
    }
 
 // Step 6 : Compare to the written image
-   std::cout << "6...";
+   std::cout << "6..." << std::endl;
    size_t dataSizeWritten = reread->GetImageDataSize();
    uint8_t *imageDataWritten = reread->GetImageData();
 
@@ -285,14 +338,14 @@ int WriteSimple(Image &img)
          << "        File type differ: "
          << fileH->GetWriteType() << " # " 
          << reread->GetFile()->GetFileType() << std::endl;
-      delete fileToBuild;
-      delete fileH;
-      delete reread;
+      fileToBuild->Delete();
+      fileH->Delete();
+      reread->Delete();
       delete[] imageData;
 
       return 1;
    }
-
+   std::cout << "6.1..." << std::endl;
    // Test the image size
    if (fileToBuild->GetXSize() != reread->GetFile()->GetXSize() ||
        fileToBuild->GetYSize() != reread->GetFile()->GetYSize() ||
@@ -306,32 +359,32 @@ int WriteSimple(Image &img)
                   << reread->GetFile()->GetYSize() << " | "
          << "Z: " << fileToBuild->GetZSize() << " # " 
                   << reread->GetFile()->GetZSize() << std::endl;
-      delete fileToBuild;
-      delete fileH;
-      delete reread;
+      fileToBuild->Delete();
+      fileH->Delete();
+      reread->Delete();
       delete[] imageData;
 
       return 1;
    }
-
+ std::cout << "6.2..." << std::endl;
    // Test the data size
    if (size != dataSizeWritten)
    {
       std::cout << "Failed" << std::endl
          << "        Pixel areas lengths differ: "
          << size << " # " << dataSizeWritten << std::endl;
-      delete fileToBuild;
-      delete fileH;
-      delete reread;
+      fileToBuild->Delete();
+      fileH->Delete();
+      reread->Delete();
       delete[] imageData;
 
       return 1;
    }
-
-   // Test the data's content
+std::cout << "6.3..." << std::endl;
+   // Test the data content
    if ( memcmp(imageData, imageDataWritten, size) !=0 )
    {
-      std::cout << "Failed" << std::endl
+      std::cout << fileName.str() << " Failed " << std::endl
                 << "        Pixel differ (as expanded in memory)." << std::endl;
       std::cout << "        list of the first " << MAX_NUMBER_OF_DIFFERENCE
                   << " pixels differing (pos : test - ref) :" 
@@ -350,21 +403,17 @@ int WriteSimple(Image &img)
             }
       }
       std::cout << std::endl;
-      delete fileToBuild;
-      delete fileH;
-      delete reread;
+      fileToBuild->Delete();
+      fileH->Delete();
+      reread->Delete();
       delete[] imageData;
 
       return 1;
    }
-
-   std::cout << "OK" << std::endl;
-
-   delete fileToBuild;
-   delete fileH;
-   delete reread;
+std::cout << "======================= OK" << std::endl;
+   fileH->Delete();
+   reread->Delete();
    delete[] imageData;
-
    return 0;
 }
 
@@ -378,15 +427,16 @@ int TestWriteSimple(int argc, char *argv[])
       return 1;
    }
 
-   //gdcm::Debug::DebugOn();
-       
+  // GDCM_NAME_SPACE::Debug::DebugOn();
+
    int ret=0;
    int i=0;
    while( Images[i].sizeX>0 && Images[i].sizeY>0 )
    {
-      std::cout << "Test n :" << i; 
+      std::cout << "Test n :" << i <<std::endl;
       ret += WriteSimple(Images[i] );
       i++;
+
    }
 
    return ret;