]> Creatis software - gdcm.git/blobdiff - src/gdcmElValSet.cxx
* distutilsSwigCPlusPlus.py work around some swig portability
[gdcm.git] / src / gdcmElValSet.cxx
index 39ba3c1a02c989f99a9b82cac71ad1d43d5a9fad..d716e6af4afa7782926199f484256a50146216dd 100644 (file)
@@ -82,19 +82,21 @@ int ElValSet::SetElValueByNumber(string content, guint32 group, guint32 element)
                return (0); 
        }                                      
        tagHt[key]->SetValue(content);
+
        // Question : m à j LgrElem ?
        tagHt[key]->SetLength(strlen(content.c_str()));  
-                               // Ou trouver les fonctions d'une classe donnée?
-                               // lgr d'une string, p.ex 
+
+       // FIXME should we really update the element length ?
+       tagHt[key]->SetLength(content.length());         
+
        return(1);              
 }
 
-
 int ElValSet::SetElValueByName(string content, string TagName) {
        if ( ! NameHt.count(TagName))
                return 0;
        if (NameHt.count(TagName) > 1) {
-               dbg.Verbose(0, "ElValSet::SetElValue",
+               dbg.Verbose(0, "ElValSet::SetElValueByName",
                            "multipe entries for this key (FIXME) !");
                return 0;
        }
@@ -104,6 +106,33 @@ int ElValSet::SetElValueByName(string content, string TagName) {
 }
 
 
+int ElValSet::SetElValueLengthByNumber(guint32 l, guint32 group, guint32 element) {
+       TagKey key = gdcmDictEntry::TranslateToKey(group, element);
+       if ( ! tagHt.count(key))
+               return 0;
+       if (tagHt.count(key) > 1) {
+               dbg.Verbose(0, "ElValSet::SetElValueLengthByNumber",
+                           "multiple entries for this key (FIXME) !");
+               return (0); 
+       }                                      
+       // m à j LgrElem 
+       tagHt[key]->SetLength(l);        
+       return(1);              
+}
+
+
+int ElValSet::SetElValueLengthByName(guint32 l, string TagName) {
+       if ( ! NameHt.count(TagName))
+               return 0;
+       if (NameHt.count(TagName) > 1) {
+               dbg.Verbose(0, "ElValSet::SetElValueByName",
+                           "multipe entries for this key (FIXME) !");
+               return 0;
+       }
+       NameHt.find(TagName)->second->SetLength(l);      
+       return(1);              
+}
+
 
 int ElValSet::Write(FILE * _fp) {
 
@@ -116,19 +145,18 @@ int ElValSet::Write(FILE * _fp) {
        guint32 val_int32;
        guint16 val_int16;
        void *ptr;
+       char str_lgrCalcGroupe[10];
        
        string implicitVRTransfertSyntax = "1.2.840.10008.1.2";
        
-/*     // Utilisées pour le calcul Group Length
+       // Utilisées pour le calcul Group Length
        int deja = 0;
-       guint32 lgrCalcGroupe;
-       ElValue * elemZ, *elemZPrec;
+       guint32 lgrCalcGroupe=0;
+       ElValue *elem, *elemZ, *elemZPrec;
        guint16 grCourant = 0;
        
-*/
-
        // Question :
-       // Comment pourrait-on tester si on est TRueDicom ou non ,
+       // Comment pourrait-on tester si on est TrueDicom ou non ,
        // (FileType est un champ de gdcmHeader ...)
        //
 
@@ -136,81 +164,77 @@ int ElValSet::Write(FILE * _fp) {
        // au cas ou un tag ai été ajouté par rapport à ce qui a été lu
        // dans l'image native
        //
-       // cf : code IdDcmWriteFile
-
-/*
-               // Pas le temps de finir 
-               // voir libido/src/dcmwrite.c
-               //
-               // mais avant ... voir si le 'group length', lorsqu'il est present
-               // sert encore a qq chose
-               // patcher une image DICOM, mettre une lgr erronnée
-               // et voir si e-film la reconnait ...
-               
+       // cf : code IdDcmWriteFile dans libido/src/dcmwrite.c
                
        for (TagElValueHT::iterator tag = tagHt.begin();
                  tag != tagHt.end();
                  ++tag){
-                 
-               elemZ = tag->second;
+
+               elem = tag->second;
+               printf("gr %04x el %04x lgr %d\n",elem->GetGroup(), elem->GetElement(), elem->GetLength());
        
-               if ( (elemZ->GetGroup() != grCourant) &&          
-                        (elemZ->GetGroup() != 0xfffe)  ) {     // On arrive sur un nv Groupe
+               if ( (elem->GetGroup() != grCourant) &&   
+                        (elem->GetGroup() != 0xfffe)   ) {     // On arrive sur un nv Groupe
                         
-                       if(elemZ->GetNum != 0x0000) {   // pas d'element 'Lgr groupe'
-                               // On le crée
-                               gdcmDictEntry * tagZ = IsInDicts(tag->second->GetGroup(), 0);
-                               elemZ = new (ElValue(tagZ)); // on le cree
-                               elemZ.SetLength(4);
+               printf("Nouv Groupegr %04x el %04x \n",elem->GetGroup(), elem->GetElement());
+
+                       elemZ = elem; 
+                        
+                       if(elemZ->GetElement() != 0x0000) {     // pas d'element 'Lgr groupe'
+                               // On crée
+                               gdcmDictEntry * tagZ = new gdcmDictEntry(grCourant, 0x0000, "UL");
+                               elemZ = new ElValue(tagZ); // on le cree
+                               elemZ->SetLength(4);
                                Add(elemZ);                                      // On l'accroche à sa place    
                        }       
                        
                        if (deja) {
-                       
-                       // A FINIR
+                               sprintf(str_lgrCalcGroupe,"%d",lgrCalcGroupe);
+                               elemZPrec->SetValue(str_lgrCalcGroupe);
+                               lgrCalcGroupe = 0;
                        }
                        deja = 1;
+                       
+                       lgrCalcGroupe =  12; //2 + 2 + 4 + 4; // Gr + Num + Lgr + LgrGroupe
+                       printf ("lgrCalcGroupe %d\n",lgrCalcGroupe);
+                       
                        elemZPrec = elemZ;
-                       grCourant = elemZ->GetGroup();
-                       lgrCalcGroupe =  12; //2 + 2 + 4 + 4; // lgr (Gr + Num + LgrElem + LgrGroupe)
+                       grCourant = elem->GetGroup();
                                                        
                } else {                // On n'EST PAS sur un nv Groupe
                
-                       lgrCalcGroupe += 2 + 2;  // lgr (Gr + Num )
-               
-                       if (IsVrUnknowkn()) {
-                       
-                       // A FINIR
-                       
-                       }
+                       printf ("lgrCalcGroupe avant : %d LgrElem %d\n",lgrCalcGroupe,elem->GetLength());
+
+                       lgrCalcGroupe += 2 + 2 + 4 + elem->GetLength();  // Gr + Num + Lgr + LgrElem 
                        
-                       // A FINIR        
-       }
+                       printf ("lgrCalcGroupe apres %d\n",lgrCalcGroupe);
 
-*/
+               }               
+       }
        
-       // Si on fait de l'implicit VR littele Endian 
+       // Si on fait de l'implicit VR little Endian 
        // (pour moins se fairche sur processeur INTEL)
-       // penser a forcer le SYNTAX TRANSFERT UID
+       // penser a forcer le TRANSFERT SYNTAX UID
                                
        SetElValueByNumber(implicitVRTransfertSyntax, 0x0002, 0x0010);  
+       SetElValueLengthByNumber(18, 0x0002, 0x0010);  // Le 0 de fin de chaine doit etre stocké, dans ce cas   
                
        // restent à tester les echecs en écriture (apres chaque fwrite)
        
-       for (TagElValueHT::iterator tag = tagHt.begin();
-                 tag != tagHt.end();
-                 ++tag){
+       for (TagElValueHT::iterator tag2 = tagHt.begin();
+                 tag2 != tagHt.end();
+                 ++tag2){
 
                // Question :
                // peut-on se passer des affectations?
                // - passer l'adresse du resultat d'une fonction (???)
                // - acceder au champ sans passer par un accesseur ?
                
-               gr =  tag->second->GetGroup();
-               el =  tag->second->GetElement();
-               lgr = tag->second->GetLength();
-               val = tag->second->GetValue().c_str();
-               vr =  tag->second->GetVR();
+               gr =  tag2->second->GetGroup();
+               el =  tag2->second->GetElement();
+               lgr = tag2->second->GetLength();
+               val = tag2->second->GetValue().c_str();
+               vr =  tag2->second->GetVR();
                        
                fwrite ( &gr,(size_t)2 ,(size_t)1 ,_fp);        //group
                fwrite ( &el,(size_t)2 ,(size_t)1 ,_fp);        //element