X-Git-Url: https://git.creatis.insa-lyon.fr/pubgit/?a=blobdiff_plain;f=src%2FgdcmElValSet.cxx;h=ed07d4f1d77c0778fdffdba304e3d6f05af77024;hb=4a0c2407ec81e041c2e43781acec4c78c83fc487;hp=184513937ae20f2cb8680e81c5c217d3c05ae638;hpb=f41fb331c46df02fd4e284960f0473d7155573a2;p=gdcm.git diff --git a/src/gdcmElValSet.cxx b/src/gdcmElValSet.cxx index 18451393..ed07d4f1 100644 --- a/src/gdcmElValSet.cxx +++ b/src/gdcmElValSet.cxx @@ -14,26 +14,6 @@ void gdcmElValSet::Add(gdcmElValue * newElValue) { NameHt[newElValue->GetName()] = newElValue; } -// TODO : faire un gdcmElValSet::ReplaceOrCreate qui remplace si ça existe, qui cree sinon - -void gdcmElValSet::ReplaceOrCreate(gdcmElValue * newElValue) { - - TagKey key = newElValue->GetKey(); - - if (tagHt.count(key) > 1) - dbg.Verbose(0, "gdcmElValSet::GetElValueByNumber", - "multiple entries for this key (FIXME) !"); - - if (tagHt.count(key)) { - tagHt.erase(key); - tagHt.erase(newElValue->GetName()); - } - - tagHt [key] = newElValue; - NameHt[newElValue->GetName()] = newElValue; -} - - void gdcmElValSet::Print(ostream & os) { for (TagElValueHT::iterator tag = tagHt.begin(); tag != tagHt.end(); @@ -43,7 +23,7 @@ void gdcmElValSet::Print(ostream & os) { os << "[" << tag->second->GetName() << "]"; os << "[" << tag->second->GetVR() << "]"; os << " lgr : " << tag->second->GetLength(); - os << endl; + os << endl; } } @@ -58,88 +38,59 @@ void gdcmElValSet::PrintByName(ostream & os) { } } -gdcmElValue* gdcmElValSet::GetElementByNumber(guint32 group, guint32 element) { +gdcmElValue* gdcmElValSet::GetElementByNumber(guint16 group, guint16 element) { TagKey key = gdcmDictEntry::TranslateToKey(group, element); if ( ! tagHt.count(key)) return (gdcmElValue*)0; - if (tagHt.count(key) > 1) - dbg.Verbose(0, "gdcmElValSet::GetElementByNumber", - "multiple entries for this key (FIXME) !"); return tagHt.find(key)->second; } gdcmElValue* gdcmElValSet::GetElementByName(string TagName) { if ( ! NameHt.count(TagName)) return (gdcmElValue*)0; - if (NameHt.count(TagName) > 1) - dbg.Verbose(0, "gdcmElValSet::GetElement", - "multipe entries for this key (FIXME) !"); return NameHt.find(TagName)->second; } -string gdcmElValSet::GetElValueByNumber(guint32 group, guint32 element) { +string gdcmElValSet::GetElValueByNumber(guint16 group, guint16 element) { TagKey key = gdcmDictEntry::TranslateToKey(group, element); if ( ! tagHt.count(key)) return "gdcm::Unfound"; - if (tagHt.count(key) > 1) - dbg.Verbose(0, "gdcmElValSet::GetElValueByNumber", - "multiple entries for this key (FIXME) !"); return tagHt.find(key)->second->GetValue(); } string gdcmElValSet::GetElValueByName(string TagName) { if ( ! NameHt.count(TagName)) return "gdcm::Unfound"; - if (NameHt.count(TagName) > 1) - dbg.Verbose(0, "gdcmElValSet::GetElValue", - "multipe entries for this key (FIXME) !"); return NameHt.find(TagName)->second->GetValue(); } + int gdcmElValSet::SetElValueByNumber(string content, - guint32 group, guint32 element) { + guint16 group, guint16 element) { TagKey key = gdcmDictEntry::TranslateToKey(group, element); if ( ! tagHt.count(key)) return 0; - if (tagHt.count(key) > 1) { - dbg.Verbose(0, "gdcmElValSet::SetElValueByNumber", - "multiple entries for this key (FIXME) !"); - return (0); - } tagHt[key]->SetValue(content); - - // Question : m à j LgrElem ? - tagHt[key]->SetLength(strlen(content.c_str())); - - // FIXME should we really update the element length ? tagHt[key]->SetLength(content.length()); - - return(1); + return 1; } int gdcmElValSet::SetElValueByName(string content, string TagName) { if ( ! NameHt.count(TagName)) return 0; - if (NameHt.count(TagName) > 1) { - dbg.Verbose(0, "gdcmElValSet::SetElValueByName", - "multipe entries for this key (FIXME) !"); - return 0; - } - NameHt.find(TagName)->second->SetValue(content); - NameHt.find(TagName)->second->SetLength(strlen(content.c_str())); - return(1); + NameHt[TagName]->SetValue(content); + NameHt[TagName]->SetLength(content.length()); + return 1; } /** - * \ingroup gdcmElValSet - * \brief Generate a free TagKey i.e. a TagKey that is not present - * in the TagHt dictionary. One of the potential usage is - * to add gdcm generated additional informartion to the ElValSet - * (see gdcmHeader::AddAndDefaultElements). - * @param group The generated tag must belong to this group. - * @return The element of tag with given group which is fee. + * \ingroup gdcmElValSet + * \brief Generate a free TagKey i.e. a TagKey that is not present + * in the TagHt dictionary. + * @param group The generated tag must belong to this group. + * @return The element of tag with given group which is fee. */ -guint32 gdcmElValSet::GenerateFreeTagKeyInGroup(guint32 group) { +guint32 gdcmElValSet::GenerateFreeTagKeyInGroup(guint16 group) { for (guint32 elem = 0; elem < UINT32_MAX; elem++) { TagKey key = gdcmDictEntry::TranslateToKey(group, elem); if (tagHt.count(key) == 0) @@ -148,362 +99,215 @@ guint32 gdcmElValSet::GenerateFreeTagKeyInGroup(guint32 group) { return UINT32_MAX; } -int gdcmElValSet::SetElValueLengthByNumber(guint32 l, - guint32 group, guint32 element) { +int gdcmElValSet::SetElValueLengthByNumber(guint32 length, + guint16 group, guint16 element) { TagKey key = gdcmDictEntry::TranslateToKey(group, element); if ( ! tagHt.count(key)) return 0; - if (tagHt.count(key) > 1) { - dbg.Verbose(0, "gdcmElValSet::SetElValueLengthByNumber", - "multiple entries for this key (FIXME) !"); - return (0); - } - tagHt[key]->SetLength(l); - return(1); + tagHt[key]->SetLength(length); + return 1 ; } -int gdcmElValSet::SetElValueLengthByName(guint32 l, string TagName) { +int gdcmElValSet::SetElValueLengthByName(guint32 length, string TagName) { if ( ! NameHt.count(TagName)) return 0; - if (NameHt.count(TagName) > 1) { - dbg.Verbose(0, "gdcmElValSet::SetElValueByName", - "multipe entries for this key (FIXME) !"); - return 0; - } - NameHt.find(TagName)->second->SetLength(l); - return(1); + NameHt.find(TagName)->second->SetLength(length); + return 1 ; } -// Sorry for the DEBUG's, but tomorow is gonna be hoter than today -#define DEBUG 0 -int gdcmElValSet::Write(FILE * _fp) { -// ATTENTION : fonction non terminée (commitée a titre de precaution) +void gdcmElValSet::UpdateGroupLength(bool SkipSequence, FileType type) { + guint16 gr, el; + string vr; + + 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){ + + 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 : pb CERTAIN + // si on est descendu 'a l'interieur' des SQ + // + // --> 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(); + } + } + + if(0) + for (GroupHT::iterator g = groupHt.begin(); + g != groupHt.end(); + ++g){ + printf("groupKey %s : %d\n",g->first.c_str(),g->second); + } + + + unsigned short int gr_bid; + + for (GroupHT::iterator g = groupHt.begin(); + g != groupHt.end(); + ++g){ + + sscanf(g->first.c_str(),"%x",&gr_bid); + tk = g->first + "|0000"; + + 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); + } + sprintf(trash ,"%d",g->second); + str_trash=trash; + elemZ->SetValue(str_trash); + } +} - guint16 gr, el; - guint32 lgr; - const char * val; - string vr; - guint32 val_uint32; - gint32 val_int32; - guint16 val_uint16; - gint16 val_int16;; - - vector tokens; - - void *ptr; - char str_lgrCalcGroupe[10]; +void gdcmElValSet::WriteElements(FileType type, FILE * _fp) { + guint16 gr, el; + guint32 lgr; + const char * val; + string vr; + guint32 val_uint32; + guint16 val_uint16; + + vector tokens; + + void *ptr; + + // 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){ + + gr = tag2->second->GetGroup(); + el = tag2->second->GetElement(); + lgr = tag2->second->GetLength(); + val = tag2->second->GetValue().c_str(); + vr = tag2->second->GetVR(); + + if ( type == ACR ) { + if (gr < 0x0008) continue; + if (gr %2) continue; + if (vr == "SQ" ) continue; + } + + fwrite ( &gr,(size_t)2 ,(size_t)1 ,_fp); //group + fwrite ( &el,(size_t)2 ,(size_t)1 ,_fp); //element + + if ( (type == ExplicitVR) && (gr <= 0x0002) ) { + // On est en EXPLICIT VR + guint16 z=0, shortLgr; + fwrite (vr.c_str(),(size_t)2 ,(size_t)1 ,_fp); + + if ( (vr == "OB") || (vr == "OW") || (vr == "SQ") ) { + fwrite ( &z, (size_t)2 ,(size_t)1 ,_fp); + fwrite ( &lgr,(size_t)4 ,(size_t)1 ,_fp); + + } else { + shortLgr=lgr; + fwrite ( &shortLgr,(size_t)2 ,(size_t)1 ,_fp); + } + } else { + fwrite ( &lgr,(size_t)4 ,(size_t)1 ,_fp); + } + + tokens.erase(tokens.begin(),tokens.end()); + Tokenize (tag2->second->GetValue(), tokens, "\\"); + + if (vr == "US" || vr == "SS") { + for (unsigned int i=0; isecond; - 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(); - - 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 elmeZ\n"); - } - - ostringstream fock; - fock << lgrCalcGroupe; - //sprintf(str_lgrCalcGroupe,"%d",lgrCalcGroupe); - elemZPrec->SetValue(fock.str()); - if(DEBUG)printf("ecriture lgr (%d, %s) pour gr %04x\n",lgrCalcGroupe, fock.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); - } - } - -} // fin if (1) - - - // restent à tester les echecs en écriture (apres chaque fwrite) - - for (TagElValueHT::iterator tag2 = tagHt.begin(); - tag2 != tagHt.end(); - ++tag2){ - - gr = tag2->second->GetGroup(); - el = tag2->second->GetElement(); - lgr = tag2->second->GetLength(); - val = tag2->second->GetValue().c_str(); - vr = tag2->second->GetVR(); - if(DEBUG)printf ("%04x %04x [%s] : [%s]\n",gr, el, vr.c_str(), val); - - fwrite ( &gr,(size_t)2 ,(size_t)1 ,_fp); //group - fwrite ( &el,(size_t)2 ,(size_t)1 ,_fp); //element - - //fwrite ( vr,(size_t)2 ,(size_t)1 ,_fp); //VR - - // si on n'est pas en IMPLICIT VR voir pb (lgr + VR) - - fwrite ( &lgr,(size_t)4 ,(size_t)1 ,_fp); //lgr - - tokens.erase(tokens.begin(),tokens.end()); - Tokenize (tag2->second->GetValue(), tokens, "\\"); - - //if (tokens.size() > 1) { printf ("size : %d\n",tokens.size());} - - if (vr == "US" || vr == "SS") { - for (unsigned int i=0; i tokens; - - void *ptr; - char str_lgrCalcGroupe[10]; - - //string implicitVRTransfertSyntax = "1.2.840.10008.1.2"; // supprime par rapport à Write - - // Utilisées pour le calcul Group Length - int deja = 0; - guint32 lgrCalcGroupe=0; - gdcmElValue *elem, *elemZ, *elemZPrec; - guint16 grCourant = 0; - - // Question : - // Comment pourrait-on tester si on est TrueDicom ou non , - // (FileType est un champ de gdcmHeader ...) - // + if ( (type == ImplicitVR) || (type == ExplicitVR) ) + UpdateGroupLength(false,type); + if ( type == ACR) + UpdateGroupLength(true,ACR); - // 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 - + WriteElements(type, _fp); - - 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(); - - 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 elmeZ\n"); - } - - ostringstream fock; - fock << lgrCalcGroupe; - //sprintf(str_lgrCalcGroupe,"%d",lgrCalcGroupe); - elemZPrec->SetValue(fock.str()); - if(DEBUG)printf("ecriture lgr (%d, %s) pour gr %04x\n",lgrCalcGroupe, fock.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); - } - } - - // Si on fait de l'implicit VR little Endian - // (pour moins se fairche sur processeur INTEL) - // penser a forcer le TRANSFERT SYNTAX UID - - // supprime par rapport à Write - //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 tag2 = tagHt.begin(); - tag2 != tagHt.end(); - ++tag2){ - - gr = tag2->second->GetGroup(); - // saut des groupes speciaux DICOM V3 - if (gr < 0x0008) continue; // ajouté par rapport à Write - // saut des groupes impairs - if (gr %2) continue; // ajouté par rapport à Write - - 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 - - // si on n'est pas en IMPLICIT VR voir pb (lgr + VR) - - fwrite ( &lgr,(size_t)4 ,(size_t)1 ,_fp); //lgr - - tokens.erase(tokens.begin(),tokens.end()); - Tokenize (tag2->second->GetValue(), tokens, "\\"); - - if (vr == "US" || vr == "SS") { - - for (unsigned int i=0; i