X-Git-Url: https://git.creatis.insa-lyon.fr/pubgit/?a=blobdiff_plain;f=Testing%2FTestChangeHeader.cxx;h=72d55563bcd7f5bb680aafed39371adfaf17bde7;hb=3306c0c4d4513e73e1ca1ab60922ca02dc86a870;hp=7ffbd7e1fd2b7661e0b738725a69829a4d21b54f;hpb=8f68ff1b50850c4b4890d15af2218e3ef12da4e2;p=gdcm.git

diff --git a/Testing/TestChangeHeader.cxx b/Testing/TestChangeHeader.cxx
index 7ffbd7e1..72d55563 100644
--- a/Testing/TestChangeHeader.cxx
+++ b/Testing/TestChangeHeader.cxx
@@ -1,80 +1,102 @@
-#include <iostream>
-//#include "gdcm.h"
+/*=========================================================================
+                                                                                
+  Program:   gdcm
+  Module:    $RCSfile: TestChangeHeader.cxx,v $
+  Language:  C++
+  Date:      $Date: 2004/12/03 20:16:56 $
+  Version:   $Revision: 1.28 $
+                                                                                
+  Copyright (c) CREATIS (Centre de Recherche et d'Applications en Traitement de
+  l'Image). All rights reserved. See Doc/License.txt or
+  http://www.creatis.insa-lyon.fr/Public/Gdcm/License.html for details.
+                                                                                
+     This software is distributed WITHOUT ANY WARRANTY; without even
+     the implied warranty of MERCHANTABILITY or FITNESS FOR A PARTICULAR
+     PURPOSE.  See the above copyright notices for more information.
+                                                                                
+=========================================================================*/
 #include "gdcmHeader.h"
 #include "gdcmFile.h"
-#include <string>
-
-
-	// ecriture d'un fichier DICOM à partir d'un dcmHeader correct.
-	// et des pixels d'une autre image
-	
-	
-int testChangeEntete(int argc, char* argv[]) {  
-   std::string premier, deuxieme;
-   char resultat[200];
-	
-   gdcmFile  *f1, *f2;
-	//gdcmHeader *e1, *e2;
-   void* imageData;
-   int dataSize;
-		
-   if (argc < 3) {
+
+// Writting of a DICOM file, using a correct gdcmHeader.
+// and pixels of an other image
+
+
+int TestChangeHeader(int argc, char* argv[])
+{
+   if (argc < 3)
+   {
       std::cerr << "usage :" << std::endl << 
-      argv[0] << " nomFichierPourEntete nomFichierPourDonnées" << std::endl;
-      return 0;	
+      argv[0] << " FileForHeader FileForPixels" << std::endl;
+      return 1;
    }
 
-   premier = argv[1];
-   f1 = new gdcmFile(premier);
-	
-   deuxieme = argv[2];
-   f2 = new gdcmFile(deuxieme);
-	
-	//f1->PrintPubElVal();
-	
-	// On suppose que les champs DICOM du 2ieme fichier existent *effectivement*
-	
+   std::string premier = argv[1];
+   gdcmFile  *f1 = new gdcmFile(premier);
+
+   std::string deuxieme = argv[2];
+   gdcmFile  *f2 = new gdcmFile(deuxieme);
+
+   //f1->PrintPubElVal();
+
+   // We suppose the DICOM Entries of the second file *do* exist !
+
    std::string nbFrames = f2->GetHeader()->GetEntryByNumber(0x0028, 0x0008);
-   if(nbFrames != "gdcm::Unfound") {
+   if(nbFrames != "gdcm::Unfound")
+   {
       f1->GetHeader()->ReplaceOrCreateByNumber( nbFrames, 0x0028, 0x0008);
    }
+
+
+// WARNING : If user tries to 'merge' two mismatching images
+// e.g. a LUT image and a RGB image
+// or a compressed and an uncompressed one
+// or a single frame and a multiframe,
+// or a '12 Bits Allocated' image and anything else, etc
+// Probabely TestChangeHeader will fail !
+// It was not designed as a 'Test' program, but as a utility
+// provided to 'transform' an image 'Siemens MRI New version' into an image 'Siemens MRI old version'
          
-   f1->GetHeader()->ReplaceOrCreateByNumber(f2->GetHeader()->GetEntryByNumber(0x0028, 0x0010),
-	  0x0028, 0x0010);// nbLig
-   f1->GetHeader()->ReplaceOrCreateByNumber( f2->GetHeader()->GetEntryByNumber(0x0028, 0x0011),
-          0x0028, 0x0011);// nbCol
-	
-	
-	// sans doute d'autres à mettre à jour...
-	
-	// TODO : rajouter une valeur par defaut.	
-	// TODO : une routine qui recoit une liste de couples (gr,el), 
-	//                        et qui fasse le boulot.
-
-				
-   dataSize = f2->GetImageDataSize();
+   f1->GetHeader()->ReplaceOrCreateByNumber(
+      f2->GetHeader()->GetEntryByNumber(0x0028, 0x0010), 0x0028, 0x0010);// nbLig
+   f1->GetHeader()->ReplaceOrCreateByNumber( 
+      f2->GetHeader()->GetEntryByNumber(0x0028, 0x0011), 0x0028, 0x0011);// nbCol
+   f1->GetHeader()->ReplaceOrCreateByNumber( 
+      f2->GetHeader()->GetEntryByNumber(0x0028, 0x0100), 0x0028, 0x0100);// BitsAllocated
+   f1->GetHeader()->ReplaceOrCreateByNumber( 
+      f2->GetHeader()->GetEntryByNumber(0x0028, 0x0101), 0x0028, 0x0101);// BitsStored
+   f1->GetHeader()->ReplaceOrCreateByNumber( 
+      f2->GetHeader()->GetEntryByNumber(0x0028, 0x0102), 0x0028, 0x0102);// HighBit
+   f1->GetHeader()->ReplaceOrCreateByNumber( 
+      f2->GetHeader()->GetEntryByNumber(0x0028, 0x0103), 0x0028, 0x0103);// Pixel Representation
+// Probabely some more to update (?)
+
+// TODO : add a default value
+// TODO : add a method that receives a list of pairs  (gr,el), 
+//                        and that does the work.
+
+   int dataSize = f2->GetImageDataSize();
    printf ("dataSize %d\n",dataSize);
-   imageData= f2->GetImageData();
-			
-	// TODO : ne devrait-on pas fusionner ces 2 fonctions ?
+   uint8_t* imageData= f2->GetImageData();
+
+// TODO : Why don't we merge theese 2 functions ?
+
    f1->SetImageData(imageData,dataSize);
-   f1->GetHeader()->SetImageDataSize(dataSize);
-	
-   f1->GetHeader()->PrintEntry();
-	
+
+   f1->GetHeader()->Print();
+
    std::string s0 =f2->GetHeader()->GetEntryByNumber(0x7fe0, 0x0000);
    std::string s10=f2->GetHeader()->GetEntryByNumber(0x7fe0, 0x0010);
    printf("lgr 7fe0, 0000 %s\n",s0.c_str());
-   printf("lgr 7fe0, 0010 %s\n",s10.c_str());	
+   printf("lgr 7fe0, 0010 %s\n",s10.c_str());
+
+   char resultat[512];
 
    sprintf(resultat, "%s.vol", deuxieme.c_str());
    printf ("WriteDCM\n");
   //f1->WriteDcmExplVR("itk-gdcm-ex.dcm");
-	//f1->WriteDcmImplVR(resultat);	
-   f1->WriteAcr(resultat);	
+  //f1->WriteDcmImplVR(resultat);
+   f1->WriteAcr(resultat);
 
    return 0;
 }
-
-
-