]> Creatis software - gdcm.git/blobdiff - Testing/TestWriteSimple.cxx
make output more readable
[gdcm.git] / Testing / TestWriteSimple.cxx
index 19b216bc1e901c5a116bcfcd7584bb821cda2da5..3c0dbafd6a00afa8046f38a916f90745a72ac07e 100644 (file)
@@ -3,8 +3,8 @@
   Program:   gdcm
   Module:    $RCSfile: TestWriteSimple.cxx,v $
   Language:  C++
-  Date:      $Date: 2005/02/10 14:23:18 $
-  Version:   $Revision: 1.25 $
+  Date:      $Date: 2005/10/28 13:16:48 $
+  Version:   $Revision: 1.46 $
                                                                                 
   Copyright (c) CREATIS (Centre de Recherche et d'Applications en Traitement de
   l'Image). All rights reserved. See Doc/License.txt or
@@ -45,116 +45,136 @@ 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'},
+
+   {128, 128, 1, 1, 8,  8,  0, 'e'},
+   {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, 128, 1, 1, 8,  8,  0, 'i'},
+   {256, 128, 1, 1, 8,  8,  0, 'i'},
+   {256, 128, 1, 1, 8,  8,  0, 'a'},
+   {128, 128, 1, 1, 8,  8,  0, 'a'},
+   
+   {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::string fileName = "TestWriteSimple.dcm";
+   std::ostringstream fileName;
+   fileName.str("");
+   fileName << "TestWriteSimple";
+
+// Step 1 : Create an empty FileHelper
 
-// Step 1 : Create the header of the image
    std::cout << "        1...";
-   gdcm::File *fileToBuild = new gdcm::File();
+   gdcm::FileHelper *fileH = gdcm::FileHelper::New();
+ //  Get the (empty) image header.  
+   gdcm::File *fileToBuild = fileH->GetFile();
    std::ostringstream str;
 
    // Set the image size
    str.str("");
    str << img.sizeX;
-   fileToBuild->InsertValEntry(str.str(),0x0028,0x0011); // Columns
-
+   fileToBuild->InsertEntryString(str.str(),0x0028,0x0011); // Columns
    str.str("");
    str << img.sizeY;
-   fileToBuild->InsertValEntry(str.str(),0x0028,0x0010); // Rows
+   fileToBuild->InsertEntryString(str.str(),0x0028,0x0010); // Rows
 
    if(img.sizeZ>1)
    {
       str.str("");
       str << img.sizeZ;
-      fileToBuild->InsertValEntry(str.str(),0x0028,0x0008); // Number of Frames
+      fileToBuild->InsertEntryString(str.str(),0x0028,0x0008); // Number of Frames
    }
 
+   fileName << "-" << img.sizeX << "-" << img.sizeY << "-" << img.sizeZ;
+
    // Set the pixel type
    str.str("");
    str << img.componentSize;
-   fileToBuild->InsertValEntry(str.str(),0x0028,0x0100); // Bits Allocated
+   fileToBuild->InsertEntryString(str.str(),0x0028,0x0100); // Bits Allocated
 
    str.str("");
    str << img.componentUse;
-   fileToBuild->InsertValEntry(str.str(),0x0028,0x0101); // Bits Stored
+   fileToBuild->InsertEntryString(str.str(),0x0028,0x0101); // Bits Stored
 
    str.str("");
-   str << img.componentSize - 1;
-   fileToBuild->InsertValEntry(str.str(),0x0028,0x0102); // High Bit
+   str << ( img.componentSize - 1 );
+   fileToBuild->InsertEntryString(str.str(),0x0028,0x0102); // High Bit
 
    // Set the pixel representation
    str.str("");
    str << img.sign;
-   fileToBuild->InsertValEntry(str.str(),0x0028,0x0103); // Pixel Representation
-
+   fileToBuild->InsertEntryString(str.str(),0x0028,0x0103); // Pixel Representation
+
+   fileName << "-" << img.componentSize;
+   if(img.sign == 0)
+      fileName << "U";
+   else
+      fileName << "S";
+   
+   switch (img.writeMode)
+   {
+      case 'a' :
+         fileName << ".ACR";  break; 
+      case 'e' :
+         fileName << ".EXPL"; break; 
+      case 'i' :
+         fileName << ".IMPL"; break;
+   } 
+
+   std::cout << "[" << fileName.str() << "]...";
    // Set the samples per pixel
    str.str("");
    str << img.components;
-   fileToBuild->InsertValEntry(str.str(),0x0028,0x0002); // Samples per Pixel
-
-   if( !fileToBuild->IsReadable() )
-   {
-      std::cout << "Failed\n"
-                << "        Prepared image isn't readable\n";
-
-      delete fileToBuild;
-      return 1;
-   }
+   fileToBuild->InsertEntryString(str.str(),0x0028,0x0002); // Samples per Pixel
 
 // Step 2 : Create the output image
    std::cout << "2...";
@@ -176,10 +196,10 @@ int WriteSimple(Image &img)
          {
             for(int c=0;c<img.components;c++)
             {
-               *tmp = j%256;
+               *tmp = (unsigned char)(j%256);
                if( img.componentSize>8 )
                {
-                  *(tmp+1) = j/256;
+                  *(tmp+1) = (unsigned char)(j/256);
                }
                tmp += img.componentSize/8;
             }
@@ -187,60 +207,71 @@ int WriteSimple(Image &img)
       }
    }
 
-// Step 3 : Create the file of the image
+// Step 3 : Set the image Pixel Data
    std::cout << "3...";
-   gdcm::FileHelper *file = new gdcm::FileHelper(fileToBuild);
-   file->SetImageData(imageData,size);
+   fileH->SetImageData(imageData,size);
 
 // Step 4 : Set the writting mode and write the image
    std::cout << "4...";
 
-   file->SetWriteModeToRaw();
+   fileH->SetWriteModeToRaw();
    switch (img.writeMode)
    {
       case 'a' : // Write an ACR file
-         file->SetWriteTypeToAcr();
+         fileH->SetWriteTypeToAcr();
          break;
 
       case 'e' : // Write a DICOM Explicit VR file
-         file->SetWriteTypeToDcmExplVR();
+         fileH->SetWriteTypeToDcmExplVR();
          break;
 
       case 'i' : // Write a DICOM Implicit VR file
-         file->SetWriteTypeToDcmImplVR();
+         fileH->SetWriteTypeToDcmImplVR();
          break;
 
       default :
-         std::cout << "Failed\n"
+         std::cout << "Failed for [" << fileName.str() << "]\n"
                    << "        Write mode '"<<img.writeMode<<"' is undefined\n";
 
-         delete file;
-         delete fileToBuild;
+         fileH->Delete();
          delete[] imageData;
          return 1;
    }
 
-   if( !file->Write(fileName) )
+   if( !fileH->Write(fileName.str()) )
    {
-      std::cout << "Failed\n"
-                << "File in unwrittable\n";
+      std::cout << "Failed for [" << fileName.str() << "]\n"
+                << "           File is unwrittable\n";
+
+      fileH->Delete();
 
-      delete file;
-      delete fileToBuild;
       delete[] imageData;
       return 1;
    }
 
 // Step 5 : Read the written image
    std::cout << "5...";
-   gdcm::FileHelper *reread = new gdcm::FileHelper( fileName );
+   // old form.
+   //gdcm::FileHelper *reread = new gdcm::FileHelper( fileName.str() );
+   // Better use :
+   gdcm::FileHelper *reread = gdcm::FileHelper::New( );
+   reread->SetFileName( fileName.str() );
+   reread->SetLoadMode(gdcm::LD_ALL); // Load everything
+                           // Possible values are 
+                           //              gdcm::LD_ALL, 
+                           //              gdcm::LD_NOSEQ, 
+                           //              gdcm::LD_NOSHADOW,
+                           //              gdcm::LD_NOSEQ|gdcm::LD_NOSHADOW, 
+                           //              gdcm::LD_NOSHADOWSEQ
+   reread->Load();
+
    if( !reread->GetFile()->IsReadable() )
    {
       std::cerr << "Failed" << std::endl
-                << "Could not reread image written: " << fileName << std::endl;
-      delete fileToBuild;
-      delete file;
-      delete reread;
+                << "Could not read written image : " << fileName.str() << std::endl;
+      fileToBuild->Delete();
+      fileH->Delete();
+      reread->Delete();
       delete[] imageData;
       return 1;
    }
@@ -250,12 +281,27 @@ int WriteSimple(Image &img)
    size_t dataSizeWritten = reread->GetImageDataSize();
    uint8_t *imageDataWritten = reread->GetImageData();
 
+   // Test the image write mode
+   if (reread->GetFile()->GetFileType() != fileH->GetWriteType())
+   {
+      std::cout << "Failed" << std::endl
+         << "        File type differ: "
+         << fileH->GetWriteType() << " # " 
+         << reread->GetFile()->GetFileType() << std::endl;
+      fileToBuild->Delete();
+      fileH->Delete();
+      reread->Delete();
+      delete[] imageData;
+
+      return 1;
+   }
+
    // Test the image size
    if (fileToBuild->GetXSize() != reread->GetFile()->GetXSize() ||
        fileToBuild->GetYSize() != reread->GetFile()->GetYSize() ||
        fileToBuild->GetZSize() != reread->GetFile()->GetZSize())
    {
-      std::cout << "Failed" << std::endl
+      std::cout << "Failed for [" << fileName.str() << "]" << std::endl
          << "        X Size differs: "
          << "X: " << fileToBuild->GetXSize() << " # " 
                   << reread->GetFile()->GetXSize() << " | "
@@ -263,9 +309,9 @@ int WriteSimple(Image &img)
                   << reread->GetFile()->GetYSize() << " | "
          << "Z: " << fileToBuild->GetZSize() << " # " 
                   << reread->GetFile()->GetZSize() << std::endl;
-      delete fileToBuild;
-      delete file;
-      delete reread;
+      fileToBuild->Delete();
+      fileH->Delete();
+      reread->Delete();
       delete[] imageData;
 
       return 1;
@@ -277,9 +323,9 @@ int WriteSimple(Image &img)
       std::cout << "Failed" << std::endl
          << "        Pixel areas lengths differ: "
          << size << " # " << dataSizeWritten << std::endl;
-      delete fileToBuild;
-      delete file;
-      delete reread;
+      fileToBuild->Delete();
+      fileH->Delete();
+      reread->Delete();
       delete[] imageData;
 
       return 1;
@@ -290,9 +336,26 @@ int WriteSimple(Image &img)
    {
       std::cout << "Failed" << std::endl
                 << "        Pixel differ (as expanded in memory)." << std::endl;
-      delete fileToBuild;
-      delete file;
-      delete reread;
+      std::cout << "        list of the first " << MAX_NUMBER_OF_DIFFERENCE
+                  << " pixels differing (pos : test - ref) :" 
+                  << std::endl;
+      unsigned int i;
+      unsigned int j;
+      for(i=0, j=0;i<dataSizeWritten && j<MAX_NUMBER_OF_DIFFERENCE;i++)
+      {
+         if(imageDataWritten[i]!=imageData[i])
+            {
+            std::cout << std::hex << "(" << i << " : " 
+                        << std::hex << (int)(imageDataWritten[i]) << " - "
+                        << std::hex << (int)(imageData[i]) << ") "
+                        << std::dec;
+            ++j;
+            }
+      }
+      std::cout << std::endl;
+      fileToBuild->Delete();
+      fileH->Delete();
+      reread->Delete();
       delete[] imageData;
 
       return 1;
@@ -300,9 +363,8 @@ int WriteSimple(Image &img)
 
    std::cout << "OK" << std::endl;
 
-   delete fileToBuild;
-   delete file;
-   delete reread;
+   fileH->Delete();
+   reread->Delete();
    delete[] imageData;
 
    return 0;
@@ -318,11 +380,14 @@ int TestWriteSimple(int argc, char *argv[])
       return 1;
    }
 
+   gdcm::Debug::DebugOn();
+       
    int ret=0;
    int i=0;
    while( Images[i].sizeX>0 && Images[i].sizeY>0 )
    {
-      ret += WriteSimple(Images[i]);
+      std::cout << "Test n :" << i; 
+      ret += WriteSimple(Images[i] );
       i++;
    }