]> Creatis software - gdcm.git/blobdiff - src/gdcmElValSet.cxx
UpdateGroupLength re-written using H-Table
[gdcm.git] / src / gdcmElValSet.cxx
index b125cdaa623858e2f29eee72816bba95d7b50a0e..beb99fd364d5dc826c64b7b4c0ccfe39d9a88276 100644 (file)
@@ -118,90 +118,79 @@ int gdcmElValSet::SetElValueLengthByName(guint32 length, string TagName) {
        return 1 ;              
 }
 
-void gdcmElValSet::UpdateGroupLength(bool SkipSequence) {
-       // On parcourt la table pour recalculer la longueur des 'elements 0x0000'
-       // au cas ou un tag ai été ajouté par rapport à ce qui a été lu
-       // dans l'image native
-       //
-       // cf : code IdDcmWriteFile dans libido/src/dcmwrite.c
-                               
-       // On fait de l'implicit VR little Endian 
-       // (pour moins se fairche sur processeur INTEL)
-       // On force le TRANSFERT SYNTAX UID
+
+
+void gdcmElValSet::UpdateGroupLength(bool SkipSequence, FileType type) {
    guint16 gr, el;
    string vr;
-   guint32 lgrCalcGroupe=0;
-   gdcmElValue *elem, *elemZ, *elemZPrec;
-   guint16 grCourant = 0;
-                               
-       TagElValueHT::iterator tag = tagHt.begin();
-       
-       elem = tag->second;
-       gr   = elem->GetGroup();
-       el   = elem->GetElement();
-                       
-       if (el != 0x0000) {
-               if(DEBUG)printf("ajout elem OOOO premiere fois\n");
-               gdcmDictEntry * tagZ = new gdcmDictEntry(gr, 0x0000, "UL");
-               elemZPrec = new gdcmElValue(tagZ);      // on le cree
-               elemZPrec->SetLength(4);
-               Add(elemZPrec);                         // On l'accroche à sa place
-       } else {
-               elemZPrec = elem;
-               if(DEBUG)printf("Pas d'ajout elem OOOO premiere fois\n");
-       }
-       lgrCalcGroupe = 0;
-       if(DEBUG)printf("init-1 lgr (%d) pour gr %04x\n",lgrCalcGroupe, gr);
-       grCourant = gr;
-       
-       for (tag = ++tagHt.begin();
-                 tag != tagHt.end();
-                 ++tag){
-                 
-               elem = tag->second;
-               gr = elem->GetGroup();
-               el = elem->GetElement();
-      vr = elem->GetVR();
+   
+   gdcmElValue *elem;
+   char trash[10];
+   string str_trash;
+   GroupKey key;
+   GroupHT groupHt;
+   TagKey tk;
+   gdcmElValue *elemZ;
+   
+   for (TagElValueHT::iterator tag2 = tagHt.begin();
+        tag2 != tagHt.end();
+        ++tag2){
 
-      if (SkipSequence && vr == "SQ")
+      elem  = tag2->second;
+      gr = elem->GetGroup();
+      el = elem->GetElement();
+      vr = elem->GetVR(); 
+                 
+      sprintf(trash, "%04x", gr);
+      key = trash;
+            
+      if (SkipSequence && vr == "SQ") continue;
          // pas SEQUENCE en ACR-NEMA
-         // WARNING : risque de pb 
+         // WARNING : pb CERTAIN
          //           si on est descendu 'a l'interieur' des SQ 
-         continue;
-            //
-            //
-               if ( (gr != grCourant) /*&&     // On arrive sur un nv Groupe     
-                    (el != 0xfffe) */  ) {
-                           
-                       if (el != 0x0000) {
-                               gdcmDictEntry * tagZ = new gdcmDictEntry(gr, 0x0000, "UL");
-                               elemZ = new gdcmElValue(tagZ); // on le cree
-                               elemZ->SetLength(4);
-                               Add(elemZ);                     // On l'accroche à sa place 
-                               if(DEBUG)printf("ajout elem OOOO pour gr %04x\n",gr);
-                       } else { 
-                               elemZ=elem;
-                               if(DEBUG)printf("maj elemZ\n");
-                       }
-                       
-                       ostringstream f;
-                       f << lgrCalcGroupe; 
-                       //sprintf(str_lgrCalcGroupe,"%d",lgrCalcGroupe);
-                       elemZPrec->SetValue(f.str());
-                       if(DEBUG)printf("ecriture lgr (%d, %s) pour gr %04x\n",lgrCalcGroupe, f.str().c_str(), grCourant);
-                       if(DEBUG)printf ("%04x %04x [%s]\n",elemZPrec->GetGroup(), elemZPrec->GetElement(),
-                                                       elemZPrec->GetValue().c_str());
-                       if(DEBUG)cout << "Addresse elemZPrec " << elemZPrec<< endl;
-                       elemZPrec=elemZ;
-                       lgrCalcGroupe = 0;
-                       grCourant     = gr;     
-                       if(DEBUG)printf("init-2 lgr (%d) pour gr %04x\n",lgrCalcGroupe, gr);                    
-               } else {                        // On n'EST PAS sur un nv Groupe
-                       lgrCalcGroupe += 2 + 2 + 4 + elem->GetLength();  // Gr + Num + Lgr + LgrElem
-                       if(DEBUG)printf("increment (%d) el %04x-->lgr (%d) pour gr %04x\n",
-                                                               elem->GetLength(), el, lgrCalcGroupe, gr);
-               }               
-       }
+         //
+         // --> la descente a l'interieur' des SQ 
+         // devra etre faite avec une liste chainee, pas avec une HTable...
+             
+      if ( groupHt.count(key) == 0) { 
+         if (el ==0x0000) {
+            groupHt[key] = 0;
+         } else {
+            groupHt[key] =2 + 2 + 4 + elem->GetLength();
+         } 
+      } else {       
+         if (type = ExplicitVR) {
+            if ( (vr == "OB") || (vr == "OW") || (vr == "SQ") ) {
+               groupHt[key] +=  4;
+            }
+         }
+         groupHt[key] += 2 + 2 + 4 + elem->GetLength(); 
+      } 
+   }
+  
+   unsigned short int gr_bid;
+  
+   for (GroupHT::iterator g = groupHt.begin();
+        g != groupHt.end();
+        ++g){ 
+  
+      tk = g->first + "|0000";
+      sscanf(g->first.c_str(),"%x",&gr_bid);
+               
+      if ( tagHt.count(tk) == 0) { 
+         gdcmDictEntry * tagZ = new gdcmDictEntry(gr_bid, 0x0000, "UL");       
+         elemZ = new gdcmElValue(tagZ);
+         elemZ->SetLength(4);
+         Add(elemZ);
+      } else {
+         elemZ=GetElementByNumber(gr_bid, 0x0000);
+      }
+      str_trash=trash;
+      elemZ->SetValue(str_trash);
+   }   
+   
+   // Liberer groupHt !
+   
 }
 
 void gdcmElValSet::WriteElements(FileType type, FILE * _fp) {
@@ -211,13 +200,15 @@ void gdcmElValSet::WriteElements(FileType type, FILE * _fp) {
    string vr;
    guint32 val_uint32;
    guint16 val_uint16;
-
+   
    vector<string> tokens;
 
    void *ptr;
 
-       // restent à tester les echecs en écriture (apres chaque fwrite)
-   for (TagElValueHT::iterator tag2 = tagHt.begin();
+   // Tout ceci ne marche QUE parce qu'on est sur un proc Little Endian 
+   // restent à tester les echecs en écriture (apres chaque fwrite)
+
+   for (TagElValueHT::iterator tag2=tagHt.begin();
         tag2 != tagHt.end();
         ++tag2){
 
@@ -234,7 +225,6 @@ void gdcmElValSet::WriteElements(FileType type, FILE * _fp) {
          if (vr == "SQ" ) continue;
       } 
 
-
       fwrite ( &gr,(size_t)2 ,(size_t)1 ,_fp);  //group
       fwrite ( &el,(size_t)2 ,(size_t)1 ,_fp);  //element
 
@@ -282,44 +272,38 @@ void gdcmElValSet::WriteElements(FileType type, FILE * _fp) {
    }
 }
 
-int gdcmElValSet::Write(FILE * _fp) {
+int gdcmElValSet::Write(FILE * _fp, FileType type) {
 
-       string implicitVRTransfertSyntax = "1.2.840.10008.1.2";
-       SetElValueByNumber(implicitVRTransfertSyntax, 0x0002, 0x0010);  
-   //FIXME Refer to standards on page 21, chapter 6.2 "Value representation":
-   //      values with a VR of UI shall be padded with a single trailing null
-   //      Dans le cas suivant on doit pader manuellement avec un 0.
-       SetElValueLengthByNumber(18, 0x0002, 0x0010); 
-       
-   UpdateGroupLength();
-   WriteElements(TrueDicom, _fp);
+   if (type == ImplicitVR) {
+      string implicitVRTransfertSyntax = "1.2.840.10008.1.2";
+      SetElValueByNumber(implicitVRTransfertSyntax, 0x0002, 0x0010);
+      
+      //FIXME Refer to standards on page 21, chapter 6.2 "Value representation":
+      //      values with a VR of UI shall be padded with a single trailing null
+      //      Dans le cas suivant on doit pader manuellement avec un 0.
+      SetElValueLengthByNumber(18, 0x0002, 0x0010);
+   }
+   
+       // Question :
+       // Comment pourrait-on si le DcmHeader vient d'un fichoer DicomV3 ou non ,
+       // (FileType est un champ de gdcmHeader ...)
+       // WARNING : Si on veut ecrire du DICOM V3 a partir d'un DcmHeader ACR-NEMA
+       // no way
        
-       return(1);
-}
-
-
-int gdcmElValSet::WriteAcr(FILE * _fp) {
-   UpdateGroupLength(true);
-       // Si on fait de l'implicit VR little Endian 
-       // (pour moins se fairche sur processeur INTEL)
-       // penser a forcer le TRANSFERT SYNTAX UID
-   WriteElements(ACR, _fp);
        
-       return(1);
-}
+   if (type == ExplicitVR) {
+      string explicitVRTransfertSyntax = "1.2.840.10008.1.2.1";
+      SetElValueByNumber(explicitVRTransfertSyntax, 0x0002, 0x0010);
+      // See above comment 
+      SetElValueLengthByNumber(20, 0x0002, 0x0010);
+   }
 
-int gdcmElValSet::WriteExplVR(FILE * _fp) {
+   if ( (type == ImplicitVR) || (type == ExplicitVR) )
+      UpdateGroupLength(false,type);
+   if ( type == ACR)
+      UpdateGroupLength(true,ACR);
 
-       string explicitVRTransfertSyntax = "1.2.840.10008.1.2.1";
-       SetElValueByNumber(explicitVRTransfertSyntax, 0x0002, 0x0010);  
-   // See comment in gdcmElValSet::Write
-       SetElValueLengthByNumber(20, 0x0002, 0x0010);
-                       
-   UpdateGroupLength();
-   WriteElements(ExplicitVR, _fp);
+   WriteElements(type, _fp);
 
-       return(1);
+   return(1);
 }
-
-
-