X-Git-Url: https://git.creatis.insa-lyon.fr/pubgit/?a=blobdiff_plain;f=src%2FgdcmDicomDir.cxx;h=cc41e96dd951fbe7bb27058dcc6448bd4ef11642;hb=3842922fd2263cea5f679c9d283b3b4afead35d2;hp=28160d5c3c4aedb045a16bde682cb4e8ed19a4ad;hpb=bac2910466f502dec4c1842527aa81a8f83f04b8;p=gdcm.git diff --git a/src/gdcmDicomDir.cxx b/src/gdcmDicomDir.cxx index 28160d5c..cc41e96d 100644 --- a/src/gdcmDicomDir.cxx +++ b/src/gdcmDicomDir.cxx @@ -3,8 +3,8 @@ Program: gdcm Module: $RCSfile: gdcmDicomDir.cxx,v $ Language: C++ - Date: $Date: 2004/11/16 16:20:23 $ - Version: $Revision: 1.81 $ + Date: $Date: 2005/01/19 08:57:14 $ + Version: $Revision: 1.106 $ Copyright (c) CREATIS (Centre de Recherche et d'Applications en Traitement de l'Image). All rights reserved. See Doc/License.txt or @@ -20,6 +20,9 @@ #include "gdcmDicomDirStudy.h" #include "gdcmDicomDirSerie.h" #include "gdcmDicomDirImage.h" +#include "gdcmDicomDirPatient.h" +#include "gdcmDicomDirMeta.h" +#include "gdcmDicomDirElement.h" #include "gdcmDirList.h" #include "gdcmUtil.h" #include "gdcmDebug.h" @@ -34,11 +37,16 @@ #include #include -#if defined( _MSC_VER) || defined(__BORLANDC__) - #include +#ifdef _MSC_VER +# define getcwd _getcwd +#endif + +#if defined(_MSC_VER) || defined(__BORLANDC__) +# include #else - #include +# include #endif + namespace gdcm { @@ -49,14 +57,12 @@ namespace gdcm // Constructor / Destructor /** - * \ingroup DicomDir * \brief Constructor : creates an empty DicomDir */ DicomDir::DicomDir() :Document( ) { Initialize(); // sets all private fields to NULL - std::string pathBidon = "Bidon"; // Sorry, NULL not allowed ... MetaElems = NewMeta(); } @@ -72,21 +78,25 @@ DicomDir::DicomDir() * - false if user passed an already built DICOMDIR file * and wants to use it */ -DicomDir::DicomDir(std::string const & fileName, bool parseDir ): +DicomDir::DicomDir(std::string const &fileName, bool parseDir ): Document( fileName ) { - // Whatever user passed (a root directory or a DICOMDIR) - // and whatever the value of parseDir was, - // Document is already executed + // At this step, Document constructor is already executed, + // whatever user passed (a root directory or a DICOMDIR) + // and whatever the value of parseDir was. + // (nothing is cheked in Document constructor, to avoid overhead) + Initialize(); // sets all private fields to NULL // if user passed a root directory, sure we didn't get anything - if ( TagHT.begin() == TagHT.end() ) // when user passed a Directory to parse { - dbg.Verbose(0, "DicomDir::DicomDir : entry HT empty"); + if (!parseDir) + gdcmVerboseMacro( "Entry HT empty for file: "<SetPrintLevel( PrintLevel ); - (*cc)->Print( os ); + (*cc)->SetPrintLevel(PrintLevel); + (*cc)->Print(os); } } @@ -181,16 +190,19 @@ void DicomDir::Print(std::ostream &os) */ bool DicomDir::IsReadable() { - if( !Document::IsReadable() ) + if( Filetype == Unknown) { + gdcmVerboseMacro( "Wrong filetype"); return false; } if( !MetaElems ) { + gdcmVerboseMacro( "Meta Elements missing in DicomDir"); return false; } if( Patients.size() <= 0 ) { + gdcmVerboseMacro( "NO Patient in DicomDir"); return false; } @@ -216,12 +228,11 @@ void DicomDir::Initialize() Progress = 0.0; Abort = false; - MetaElems = 0; + MetaElems = NULL; } /** - * \ingroup DicomDir * \brief fills the whole structure, starting from a root Directory */ void DicomDir::ParseDirectory() @@ -231,7 +242,6 @@ void DicomDir::ParseDirectory() } /** - * \ingroup DicomDir * \brief Set the start method to call when the parsing of the * directory starts. * @param method Method to call @@ -239,8 +249,8 @@ void DicomDir::ParseDirectory() * @param argDelete Argument * \warning In python : the arg parameter isn't considered */ -void DicomDir::SetStartMethod( DicomDir::Method* method, void* arg, - DicomDir::Method* argDelete ) +void DicomDir::SetStartMethod( DicomDir::Method *method, void *arg, + DicomDir::Method *argDelete ) { if( StartArg && StartMethodArgDelete ) { @@ -253,19 +263,17 @@ void DicomDir::SetStartMethod( DicomDir::Method* method, void* arg, } /** - * \ingroup DicomDir * \brief Set the method to delete the argument * The argument is destroyed when the method is changed or when the * class is destroyed * @param method Method to call to delete the argument */ -void DicomDir::SetStartMethodArgDelete( DicomDir::Method* method ) +void DicomDir::SetStartMethodArgDelete( DicomDir::Method *method ) { StartMethodArgDelete = method; } /** - * \ingroup DicomDir * \brief Set the progress method to call when the parsing of the * directory progress * @param method Method to call @@ -273,8 +281,8 @@ void DicomDir::SetStartMethodArgDelete( DicomDir::Method* method ) * @param argDelete Argument * \warning In python : the arg parameter isn't considered */ -void DicomDir::SetProgressMethod( DicomDir::Method* method, void* arg, - DicomDir::Method* argDelete ) +void DicomDir::SetProgressMethod( DicomDir::Method *method, void *arg, + DicomDir::Method *argDelete ) { if( ProgressArg && ProgressMethodArgDelete ) { @@ -287,27 +295,25 @@ void DicomDir::SetProgressMethod( DicomDir::Method* method, void* arg, } /** - * \ingroup DicomDir * \brief Set the method to delete the argument * The argument is destroyed when the method is changed or when the * class is destroyed * @param method Method to call to delete the argument */ -void DicomDir::SetProgressMethodArgDelete( DicomDir::Method* method ) +void DicomDir::SetProgressMethodArgDelete( DicomDir::Method *method ) { ProgressMethodArgDelete = method; } /** - * \ingroup DicomDir * \brief Set the end method to call when the parsing of the directory ends * @param method Method to call * @param arg Argument to pass to the method * @param argDelete Argument * \warning In python : the arg parameter isn't considered */ -void DicomDir::SetEndMethod( DicomDir::Method* method, void* arg, - DicomDir::Method* argDelete ) +void DicomDir::SetEndMethod( DicomDir::Method *method, void *arg, + DicomDir::Method *argDelete ) { if( EndArg && EndMethodArgDelete ) { @@ -320,19 +326,17 @@ void DicomDir::SetEndMethod( DicomDir::Method* method, void* arg, } /** - * \ingroup DicomDir * \brief Set the method to delete the argument * The argument is destroyed when the method is changed or when * the class is destroyed * @param method Method to call to delete the argument */ -void DicomDir::SetEndMethodArgDelete( DicomDir::Method* method ) +void DicomDir::SetEndMethodArgDelete( DicomDir::Method *method ) { EndMethodArgDelete = method; } /** - * \ingroup DicomDir * \brief writes on disc a DICOMDIR * \ warning does NOT add the missing elements in the header : * it's up to the user doing it ! @@ -343,17 +347,17 @@ void DicomDir::SetEndMethodArgDelete( DicomDir::Method* method ) * @return false only when fail to open */ -bool DicomDir::WriteDicomDir(std::string const& fileName) +bool DicomDir::WriteDicomDir(std::string const &fileName) { int i; uint16_t sq[4] = { 0x0004, 0x1220, 0xffff, 0xffff }; uint16_t sqt[4]= { 0xfffe, 0xe0dd, 0xffff, 0xffff }; - std::ofstream* fp = new std::ofstream(fileName.c_str(), + std::ofstream *fp = new std::ofstream(fileName.c_str(), std::ios::out | std::ios::binary); if( !fp ) { - dbg.Verbose(2, "Failed to open(write) File: ", fileName.c_str()); + gdcmVerboseMacro("Failed to open(write) File: " << fileName.c_str()); return false; } @@ -363,7 +367,7 @@ bool DicomDir::WriteDicomDir(std::string const& fileName) binary_write( *fp, "DICM"); DicomDirMeta *ptrMeta = GetDicomDirMeta(); - ptrMeta->Write(fp, ExplicitVR); + ptrMeta->WriteContent(fp, ExplicitVR); // force writing 0004|1220 [SQ ], that CANNOT exist within DicomDirMeta for(i=0;i<4;++i) @@ -375,7 +379,7 @@ bool DicomDir::WriteDicomDir(std::string const& fileName) cc != Patients.end(); ++cc ) { - (*cc)->Write( fp, ExplicitVR ); + (*cc)->WriteContent( fp, ExplicitVR ); } // force writing Sequence Delimitation Item @@ -385,6 +389,8 @@ bool DicomDir::WriteDicomDir(std::string const& fileName) } fp->close(); + delete fp; + return true; } @@ -392,19 +398,20 @@ bool DicomDir::WriteDicomDir(std::string const& fileName) // Protected /** - * \ingroup DicomDir * \brief create a Document-like chained list from a root Directory * @param path entry point of the tree-like structure */ void DicomDir::CreateDicomDirChainedList(std::string const & path) { CallStartMethod(); - DirList fileList(path,1); // gets recursively the file list + DirList dirList(path,1); // gets recursively the file list unsigned int count = 0; VectDocument list; Header *header; - for( DirList::iterator it = fileList.begin(); + DirListType fileList = dirList.GetFilenames(); + + for( DirListType::iterator it = fileList.begin(); it != fileList.end(); ++it ) { @@ -418,20 +425,15 @@ void DicomDir::CreateDicomDirChainedList(std::string const & path) header = new Header( it->c_str() ); if( !header ) { - dbg.Verbose( 1, - "DicomDir::CreateDicomDirChainedList: " - "failure in new Header ", - it->c_str() ); + gdcmVerboseMacro( "Failure in new Header " << it->c_str() ); + continue; } if( header->IsReadable() ) { // Add the file header to the chained list: list.push_back(header); - dbg.Verbose( 1, - "DicomDir::CreateDicomDirChainedList: readable ", - it->c_str() ); - + gdcmVerboseMacro( "Readable " << it->c_str() ); } else { @@ -442,20 +444,26 @@ void DicomDir::CreateDicomDirChainedList(std::string const & path) // sorts Patient/Study/Serie/ std::sort(list.begin(), list.end(), DicomDir::HeaderLessThan ); - std::string tmp = fileList.GetDirName(); + std::string tmp = dirList.GetDirName(); //for each Header of the chained list, add/update the Patient/Study/Serie/Image info SetElements(tmp, list); CallEndMethod(); + + for(VectDocument::iterator itDoc=list.begin(); + itDoc!=list.end(); + ++itDoc) + { + delete dynamic_cast
(*itDoc); + } } /** - * \ingroup DicomDir * \brief adds *the* Meta to a partially created DICOMDIR */ DicomDirMeta * DicomDir::NewMeta() { - DicomDirMeta *m = new DicomDirMeta( &TagHT ); + DicomDirMeta *m = new DicomDirMeta(); if ( TagHT.begin() != TagHT.end() ) // after Document Parsing { @@ -467,12 +475,12 @@ DicomDirMeta * DicomDir::NewMeta() cc != lastOneButSequence; ++cc) { - m->AddDocEntry( cc->second ); + m->AddEntry( cc->second ); } } else // after root directory parsing { - ListDicomDirMetaElem const & elemList = + ListDicomDirMetaElem const &elemList = Global::GetDicomDirElements()->GetDicomDirMetaElements(); m->FillObject(elemList); } @@ -483,7 +491,7 @@ DicomDirMeta * DicomDir::NewMeta() /** * \brief adds a new Patient (with the basic elements) to a partially created DICOMDIR */ -DicomDirPatient * DicomDir::NewPatient() +DicomDirPatient *DicomDir::NewPatient() { ListDicomDirPatientElem::const_iterator it; uint16_t tmpGr,tmpEl; @@ -492,44 +500,21 @@ DicomDirPatient * DicomDir::NewPatient() ListDicomDirPatientElem const & elemList = Global::GetDicomDirElements()->GetDicomDirPatientElements(); - SQItem *s = new SQItem(0); + DicomDirPatient *p = new DicomDirPatient(); // for all the DicomDirPatient Elements for( it = elemList.begin(); it != elemList.end(); ++it ) { tmpGr = it->Group; tmpEl = it->Elem; - dictEntry = GetPubDict()->GetDictEntryByNumber(tmpGr, tmpEl); + dictEntry = GetPubDict()->GetDictEntry(tmpGr, tmpEl); entry = new ValEntry( dictEntry ); entry->SetOffset(0); // just to avoid further missprinting entry->SetValue( it->Value ); - // dealing with value length ... - - if( dictEntry->GetGroup() == 0xfffe) - { - entry->SetLength(entry->GetValue().length()); - } - else if( dictEntry->GetVR() == "UL" || dictEntry->GetVR() == "SL" ) - { - entry->SetLength( 4 ); - } - else if( dictEntry->GetVR() == "US" || dictEntry->GetVR() == "SS" ) - { - entry->SetLength(2); - } - else if( dictEntry->GetVR() == "SQ" ) - { - entry->SetLength( 0xffffffff ); - } - else - { - entry->SetLength( entry->GetValue().length() ); - } - s->AddDocEntry( entry ); + p->AddEntry( entry ); } - DicomDirPatient *p = new DicomDirPatient(s, &TagHT); Patients.push_front( p ); return p; @@ -543,7 +528,7 @@ DicomDirPatient * DicomDir::NewPatient() * GDCM_DICOMDIR_STUDY, GDCM_DICOMDIR_SERIE ...) * @param header Header of the current file */ -void DicomDir::SetElement(std::string const & path, DicomDirType type, +void DicomDir::SetElement(std::string const &path, DicomDirType type, Document *header) { ListDicomDirElem elemList; //FIXME this is going to be a by copy operation @@ -552,30 +537,52 @@ void DicomDir::SetElement(std::string const & path, DicomDirType type, DictEntry *dictEntry; ValEntry *entry; std::string val; - SQItem *si = new SQItem(0); // all the items will be at level 1 + SQItem *si; switch( type ) { case GDCM_DICOMDIR_IMAGE: elemList = Global::GetDicomDirElements()->GetDicomDirImageElements(); + si = new DicomDirImage(); + if( !AddDicomDirImageToEnd(static_cast(si)) ) + { + gdcmVerboseMacro( "Add DicomDirImageToEnd failed"); + } break; - case GDCM_DICOMDIR_SERIE: elemList = Global::GetDicomDirElements()->GetDicomDirSerieElements(); + si = new DicomDirSerie(); + if( !AddDicomDirSerieToEnd(static_cast(si)) ) + { + gdcmVerboseMacro( "Add DicomDirSerieToEnd failed"); + } break; - case GDCM_DICOMDIR_STUDY: elemList = Global::GetDicomDirElements()->GetDicomDirStudyElements(); + si = new DicomDirStudy(); + if( !AddDicomDirStudyToEnd(static_cast(si)) ) + { + gdcmVerboseMacro( "Add DicomDirStudyToEnd failed"); + } break; - case GDCM_DICOMDIR_PATIENT: elemList = Global::GetDicomDirElements()->GetDicomDirPatientElements(); + si = new DicomDirPatient(); + if( !AddDicomDirPatientToEnd(static_cast(si)) ) + { + gdcmVerboseMacro( "Add DicomDirPatientToEnd failed"); + } break; - case GDCM_DICOMDIR_META: elemList = Global::GetDicomDirElements()->GetDicomDirMetaElements(); + si = new DicomDirMeta(); + if( MetaElems ) + { + gdcmVerboseMacro( "MetaElements already exist, they will be destroyed"); + delete MetaElems; + } + MetaElems = static_cast(si); break; - default: return; } @@ -593,18 +600,17 @@ void DicomDir::SetElement(std::string const & path, DicomDirType type, { tmpGr = it->Group; tmpEl = it->Elem; - dictEntry = GetPubDict()->GetDictEntryByNumber(tmpGr, tmpEl); + dictEntry = GetPubDict()->GetDictEntry(tmpGr, tmpEl); entry = new ValEntry( dictEntry ); // Be sure it's never a BinEntry ! entry->SetOffset(0); // just to avoid further missprinting - entry->SetLength(0); // just to avoid further missprinting if( header ) { // NULL when we Build Up (ex nihilo) a DICOMDIR // or when we add the META elems - val = header->GetEntryByNumber(tmpGr, tmpEl); + val = header->GetEntry(tmpGr, tmpEl); } else { @@ -622,8 +628,7 @@ void DicomDir::SetElement(std::string const & path, DicomDirType type, { if( header->GetFileName().substr(0, path.length()) != path ) { - dbg.Verbose(0, "DicomDir::SetElement : the base path" - " of file name is incorrect"); + gdcmVerboseMacro( "The base path of file name is incorrect"); val = header->GetFileName(); } else @@ -638,38 +643,12 @@ void DicomDir::SetElement(std::string const & path, DicomDirType type, } else { - if ( header->GetEntryLengthByNumber(tmpGr,tmpEl) == 0 ) + if ( header->GetEntryLength(tmpGr,tmpEl) == 0 ) val = it->Value; } - // GDCM_UNFOUND or not ! - entry->SetValue( val ); // troubles expected when vr=SQ ... - if( dictEntry ) - { - if( dictEntry->GetGroup() == 0xfffe ) - { - entry->SetLength( entry->GetValue().length() ); // FIXME - } - else if( dictEntry->GetVR() == "UL" || dictEntry->GetVR() == "SL" ) - { - entry->SetLength(4); - } - else if( dictEntry->GetVR() == "US" || dictEntry->GetVR() == "SS" ) - { - entry->SetLength(2); - } - else if( dictEntry->GetVR() == "SQ" ) - { - entry->SetLength( 0xffffffff ); - } - else - { - entry->SetLength( entry->GetValue().length() ); - } - } - if ( type == GDCM_DICOMDIR_META ) // fusible : should never print ! { std::cout << "GDCM_DICOMDIR_META ?!? should never print that" @@ -677,30 +656,6 @@ void DicomDir::SetElement(std::string const & path, DicomDirType type, } si->AddEntry(entry); } - switch( type ) - { - case GDCM_DICOMDIR_IMAGE: - AddDicomDirImageToEnd(si); - break; - - case GDCM_DICOMDIR_SERIE: - AddDicomDirSerieToEnd(si); - break; - - case GDCM_DICOMDIR_STUDY: - AddDicomDirStudyToEnd(si); - break; - - case GDCM_DICOMDIR_PATIENT: - AddDicomDirPatientToEnd(si); - break; - - default: - return; - } - //int count=1; // find a trick to increment - //s->AddEntry(si, count); // Seg Faults - } //----------------------------------------------------------------------------- @@ -719,7 +674,6 @@ void DicomDir::CallStartMethod() //----------------------------------------------------------------------------- /** - * \ingroup DicomDir * \brief CallProgressMethod */ void DicomDir::CallProgressMethod() @@ -732,7 +686,6 @@ void DicomDir::CallProgressMethod() //----------------------------------------------------------------------------- /** - * \ingroup DicomDir * \brief CallEndMethod */ void DicomDir::CallEndMethod() @@ -747,7 +700,6 @@ void DicomDir::CallEndMethod() //----------------------------------------------------------------------------- // Private /** - * \ingroup DicomDir * \brief create a 'DicomDir' from a DICOMDIR Header */ void DicomDir::CreateDicomDir() @@ -761,63 +713,62 @@ void DicomDir::CreateDicomDir() // + loop to 1 - // Directory record sequence - DocEntry *e = GetDocEntryByNumber(0x0004, 0x1220); + DocEntry *e = GetDocEntry(0x0004, 0x1220); if ( !e ) { - dbg.Verbose(0, "DicomDir::DicomDir : NO Directory record" - " sequence (0x0004,0x1220)"); + gdcmVerboseMacro( "NO Directory record sequence (0x0004,0x1220)"); /// \todo FIXME: what to do when the parsed file IS NOT a DICOMDIR file ? return; } - SeqEntry* s = dynamic_cast(e); + SeqEntry *s = dynamic_cast(e); if ( !s ) { - dbg.Verbose(0, "DicomDir::CreateDicomDir: no SeqEntry present"); + gdcmVerboseMacro( "No SeqEntry present"); // useless : (0x0004,0x1220) IS a Sequence ! return; } - DicomDirType type; // = DicomDir::GDCM_DICOMDIR_META; MetaElems = NewMeta(); ListSQItem listItems = s->GetSQItems(); - DocEntry * d; + DocEntry *d; std::string v; + SQItem *si; for( ListSQItem::iterator i = listItems.begin(); i !=listItems.end(); ++i ) { - d = (*i)->GetDocEntryByNumber(0x0004, 0x1430); // Directory Record Type - if ( ValEntry* valEntry = dynamic_cast< ValEntry* >(d) ) + d = (*i)->GetDocEntry(0x0004, 0x1430); // Directory Record Type + if ( ValEntry* valEntry = dynamic_cast(d) ) { v = valEntry->GetValue(); } else { - dbg.Verbose(0, "DicomDir::CreateDicomDir: not a ValEntry."); + gdcmVerboseMacro( "Not a ValEntry."); continue; } if( v == "PATIENT " ) { - AddDicomDirPatientToEnd( *i ); - type = DicomDir::GDCM_DICOMDIR_PATIENT; + si = new DicomDirPatient(); + AddDicomDirPatientToEnd( static_cast(si) ); } else if( v == "STUDY " ) { - AddDicomDirStudyToEnd( *i ); - type = DicomDir::GDCM_DICOMDIR_STUDY; + si = new DicomDirStudy(); + AddDicomDirStudyToEnd( static_cast(si) ); } else if( v == "SERIES" ) { - AddDicomDirSerieToEnd( *i ); - type = DicomDir::GDCM_DICOMDIR_SERIE; + si = new DicomDirSerie(); + AddDicomDirSerieToEnd( static_cast(si) ); } else if( v == "IMAGE " ) { - AddDicomDirImageToEnd( *i ); - type = DicomDir::GDCM_DICOMDIR_IMAGE; + si = new DicomDirImage(); + AddDicomDirImageToEnd( static_cast(si) ); } else { @@ -825,53 +776,55 @@ void DicomDir::CreateDicomDir() // neither an 'IMAGE' SQItem. Skip to next item. continue; } + MoveSQItem(si,*i); } + TagHT.clear(); } /** - * \ingroup DicomDir * \brief Well ... there is only one occurence */ -void DicomDir::AddDicomDirMeta() +bool DicomDir::AddDicomDirMeta() { if( MetaElems ) { delete MetaElems; } - MetaElems = new DicomDirMeta( &TagHT ); + MetaElems = new DicomDirMeta(); + return true; } /** - * \ingroup DicomDir * \brief AddDicomDirPatientToEnd - * @param s SQ Item to enqueue to the DicomPatient chained List + * @param dd SQ Item to enqueue to the DicomPatient chained List */ -void DicomDir::AddDicomDirPatientToEnd(SQItem *s) +bool DicomDir::AddDicomDirPatientToEnd(DicomDirPatient *dd) { - Patients.push_back(new DicomDirPatient(s, &TagHT)); + Patients.push_back(dd); + return true; } /** - * \ingroup DicomDir * \brief AddDicomDirStudyToEnd - * @param s SQ Item to enqueue to the DicomDirStudy chained List + * @param dd SQ Item to enqueue to the DicomDirStudy chained List */ - void DicomDir::AddDicomDirStudyToEnd(SQItem *s) +bool DicomDir::AddDicomDirStudyToEnd(DicomDirStudy *dd) { if( Patients.size() > 0 ) { ListDicomDirPatient::iterator itp = Patients.end(); itp--; - (*itp)->AddDicomDirStudy(new DicomDirStudy(s, &TagHT)); + (*itp)->AddDicomDirStudy(dd); + return true; } + return false; } /** - * \ingroup DicomDir * \brief AddDicomDirSerieToEnd - * @param s SQ Item to enqueue to the DicomDirSerie chained List + * @param dd SQ Item to enqueue to the DicomDirSerie chained List */ -void DicomDir::AddDicomDirSerieToEnd(SQItem *s) +bool DicomDir::AddDicomDirSerieToEnd(DicomDirSerie *dd) { if( Patients.size() > 0 ) { @@ -883,17 +836,18 @@ void DicomDir::AddDicomDirSerieToEnd(SQItem *s) ListDicomDirStudy::const_iterator itst = (*itp)->GetDicomDirStudies().end(); itst--; - (*itst)->AddDicomDirSerie(new DicomDirSerie(s, &TagHT)); + (*itst)->AddDicomDirSerie(dd); + return true; } } + return false; } /** - * \ingroup DicomDir * \brief AddDicomDirImageToEnd - * @param s SQ Item to enqueue to the DicomDirImage chained List + * @param dd SQ Item to enqueue to the DicomDirImage chained List */ - void DicomDir::AddDicomDirImageToEnd(SQItem *s) +bool DicomDir::AddDicomDirImageToEnd(DicomDirImage *dd) { if( Patients.size() > 0 ) { @@ -910,14 +864,15 @@ void DicomDir::AddDicomDirSerieToEnd(SQItem *s) { ListDicomDirSerie::const_iterator its = (*itst)->GetDicomDirSeries().end(); its--; - (*its)->AddDicomDirImage(new DicomDirImage(s, &TagHT)); + (*its)->AddDicomDirImage(dd); + return true; } } } + return false; } /** - * \ingroup DicomDir * \brief for each Header of the chained list, add/update the Patient/Study/Serie/Image info * @param path path of the root directory * @param list chained list of Headers @@ -935,32 +890,38 @@ void DicomDir::SetElements(std::string const & path, VectDocument const &list) std::string studCurInstanceUID, studCurID; std::string serCurInstanceUID, serCurID; + bool first = true; for( VectDocument::const_iterator it = list.begin(); it != list.end(); ++it ) { // get the current file characteristics - patCurName = (*it)->GetEntryByNumber(0x0010,0x0010); - patCurID = (*it)->GetEntryByNumber(0x0010,0x0011); - studCurInstanceUID = (*it)->GetEntryByNumber(0x0020,0x000d); - studCurID = (*it)->GetEntryByNumber(0x0020,0x0010); - serCurInstanceUID = (*it)->GetEntryByNumber(0x0020,0x000e); - serCurID = (*it)->GetEntryByNumber(0x0020,0x0011); - - if( patCurName != patPrevName || patCurID != patPrevID) + patCurName = (*it)->GetEntry(0x0010,0x0010); + patCurID = (*it)->GetEntry(0x0010,0x0011); + studCurInstanceUID = (*it)->GetEntry(0x0020,0x000d); + studCurID = (*it)->GetEntry(0x0020,0x0010); + serCurInstanceUID = (*it)->GetEntry(0x0020,0x000e); + serCurID = (*it)->GetEntry(0x0020,0x0011); + + if( patCurName != patPrevName || patCurID != patPrevID || first ) { SetElement(path, GDCM_DICOMDIR_PATIENT, *it); + first = true; } // if new Study Deal with 'STUDY' Elements - if( studCurInstanceUID != studPrevInstanceUID || studCurID != studPrevID ) + if( studCurInstanceUID != studPrevInstanceUID || studCurID != studPrevID + || first ) { SetElement(path, GDCM_DICOMDIR_STUDY, *it); + first = true; } // if new Serie Deal with 'SERIE' Elements - if( serCurInstanceUID != serPrevInstanceUID || serCurID != serPrevID ) + if( serCurInstanceUID != serPrevInstanceUID || serCurID != serPrevID + || first ) { SetElement(path, GDCM_DICOMDIR_SERIE, *it); + first = true; } // Always Deal with 'IMAGE' Elements @@ -972,17 +933,67 @@ void DicomDir::SetElements(std::string const & path, VectDocument const &list) studPrevID = studCurID; serPrevInstanceUID = serCurInstanceUID; serPrevID = serCurID; + first = false; + } +} + +/** + * \brief Move the content of the source SQItem to the destination SQItem + * Only DocEntry's are moved + * @param dst destination SQItem + * @param src source SQItem + */ +void DicomDir::MoveSQItem(SQItem *dst,SQItem *src) +{ + DocEntry *entry; + + entry = src->GetFirstEntry(); + while(entry) + { + src->RemoveEntryNoDestroy(entry); + dst->AddEntry(entry); + entry = src->GetNextEntry(); } } /** - * \ingroup DicomDir * \brief compares two dgcmHeaders */ bool DicomDir::HeaderLessThan(Document *header1, Document *header2) { return *header1 < *header2; } + + +/** + * \brief Get the first entry while visiting the DicomDirPatients + * \return The first DicomDirPatient if found, otherwhise NULL + */ +DicomDirPatient *DicomDir::GetFirstEntry() +{ + ItDicomDirPatient = Patients.begin(); + if ( ItDicomDirPatient != Patients.end() ) + return *ItDicomDirPatient; + return NULL; +} + +/** + * \brief Get the next entry while visiting the DicomDirPatients + * \note : meaningfull only if GetFirstEntry already called + * \return The next DicomDirPatient if found, otherwhise NULL + */ +DicomDirPatient *DicomDir::GetNextEntry() +{ + gdcmAssertMacro (ItDicomDirPatient != Patients.end()); + { + ++ItDicomDirPatient; + if ( ItDicomDirPatient != Patients.end() ) + return *ItDicomDirPatient; + } + return NULL; +} + + } // end namespace gdcm //-----------------------------------------------------------------------------