X-Git-Url: https://git.creatis.insa-lyon.fr/pubgit/?a=blobdiff_plain;f=src%2FgdcmSerieHelper.cxx;h=9ac3369398b74da8fdcf28759746825f0eea2d3d;hb=6237b242dba31eddbd81ba4c4b167bcf82a51c37;hp=f9fd686322cf17cdca35266b186bb823425662b7;hpb=eda9bfda6d03f728b261824ca3b04f596485e288;p=gdcm.git diff --git a/src/gdcmSerieHelper.cxx b/src/gdcmSerieHelper.cxx index f9fd6863..9ac33693 100644 --- a/src/gdcmSerieHelper.cxx +++ b/src/gdcmSerieHelper.cxx @@ -3,8 +3,8 @@ Program: gdcm Module: $RCSfile: gdcmSerieHelper.cxx,v $ Language: C++ - Date: $Date: 2005/02/05 01:25:03 $ - Version: $Revision: 1.3 $ + Date: $Date: 2005/06/24 10:55:59 $ + Version: $Revision: 1.10 $ 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,7 @@ #include "gdcmDirList.h" #include "gdcmFile.h" #include "gdcmDebug.h" +#include "gdcmUtil.h" #include #include @@ -36,13 +37,13 @@ namespace gdcm SerieHelper::SerieHelper() { // For all the File lists of the gdcm::Serie - GdcmFileList *l = GetFirstCoherentFileList(); + FileList *l = GetFirstCoherentFileList(); while (l) { // For all the files of a File list - for (GdcmFileList::iterator it = l->begin(); - it != l->end(); - ++it) + for (FileList::iterator it = l->begin(); + it != l->end(); + ++it) { delete *it; } @@ -58,13 +59,13 @@ SerieHelper::SerieHelper() SerieHelper::~SerieHelper() { // For all the Coherent File lists of the gdcm::Serie - GdcmFileList *l = GetFirstCoherentFileList(); + FileList *l = GetFirstCoherentFileList(); while (l) { // For all the files of a Coherent File list - for (GdcmFileList::iterator it = l->begin(); - it != l->end(); - ++it) + for (FileList::iterator it = l->begin(); + it != l->end(); + ++it) { delete *it; } @@ -85,30 +86,71 @@ SerieHelper::~SerieHelper() */ void SerieHelper::AddFileName(std::string const &filename) { - //directly use string and not const char*: + // Create a DICOM file File *header = new File( filename ); - if( header->IsReadable() ) + if ( header->IsReadable() ) { - // 0020 000e UI REL Series Instance UID - std::string uid = header->GetEntryValue (0x0020, 0x000e); - // if uid == GDCM_UNFOUND then consistently we should find GDCM_UNFOUND - // no need here to do anything special + int allrules = 1; + // First step the user has defined s set of rules for the DICOM he is looking for + // make sure the file correspond to his set of rules: + for(SerieRestrictions::iterator it = Restrictions.begin(); + it != Restrictions.end(); + ++it) + { + const Rule &r = *it; + const std::string s;// = header->GetEntryValue( r.first ); + if ( !Util::DicomStringEqual(s, r.second.c_str()) ) + { + // Argh ! This rule is unmatch let's just quit + allrules = 0; + break; + } + } + if ( allrules ) // all rules are respected: + { + // Alright ! we have a found a DICOM that match the user expectation. + // Let's add it ! + + // 0020 000e UI REL Series Instance UID + const std::string &uid = header->GetEntryValue (0x0020, 0x000e); + // if uid == GDCM_UNFOUND then consistently we should find GDCM_UNFOUND + // no need here to do anything special - if ( CoherentGdcmFileListHT.count(uid) == 0 ) + if ( CoherentFileListHT.count(uid) == 0 ) + { + gdcmDebugMacro(" New Serie UID :[" << uid << "]"); + // create a std::list in 'uid' position + CoherentFileListHT[uid] = new FileList; + } + // Current Serie UID and DICOM header seems to match add the file: + CoherentFileListHT[uid]->push_back( header ); + } + else { - gdcmVerboseMacro(" New Serie UID :[" << uid << "]"); - // create a std::list in 'uid' position - CoherentGdcmFileListHT[uid] = new GdcmFileList; + // at least one rule was unmatch we need to deallocate the file: + delete header; } - // Current Serie UID and DICOM header seems to match add the file: - CoherentGdcmFileListHT[uid]->push_back( header ); } else { - gdcmVerboseMacro("Could not read file: " << filename ); + gdcmWarningMacro("Could not read file: " << filename ); delete header; } } +/** + * \brief add a rules for restricting a DICOM file to be in the serie we are + * trying to find. For example you can select only the DICOM file from a + * directory which would have a particular EchoTime==4.0. + * This method is a user level, value is not required to be formatted as a DICOM + * string + */ +void SerieHelper::AddRestriction(TagKey const &key, std::string const &value) +{ + Rule r; + r.first = key; + r.second = value; + Restrictions.push_back( r ); +} /** * \brief Sets the root Directory @@ -133,30 +175,30 @@ void SerieHelper::SetDirectory(std::string const &dir, bool recursive) * But as I don't know how to do it, I leave it this way * BTW, this is also a Strategy, I don't know this is the best approach :) */ -void SerieHelper::OrderGdcmFileList(GdcmFileList *CoherentGdcmFileList) +void SerieHelper::OrderFileList(FileList *coherentFileList) { - if( ImagePositionPatientOrdering( CoherentGdcmFileList ) ) + if ( ImagePositionPatientOrdering( coherentFileList ) ) { return ; } - else if( ImageNumberOrdering(CoherentGdcmFileList ) ) + else if ( ImageNumberOrdering(coherentFileList ) ) { return ; } else { - FileNameOrdering(CoherentGdcmFileList ); + FileNameOrdering(coherentFileList ); } } /** * \brief Get the first List while visiting the CoherentFileListHT - * @return The first GdcmFileList if found, otherwhise NULL + * @return The first FileList if found, otherwhise NULL */ -GdcmFileList *SerieHelper::GetFirstCoherentFileList() +FileList *SerieHelper::GetFirstCoherentFileList() { - ItListHt = CoherentGdcmFileListHT.begin(); - if( ItListHt != CoherentGdcmFileListHT.end() ) + ItListHt = CoherentFileListHT.begin(); + if ( ItListHt != CoherentFileListHT.end() ) return ItListHt->second; return NULL; } @@ -164,14 +206,14 @@ GdcmFileList *SerieHelper::GetFirstCoherentFileList() /** * \brief Get the next List while visiting the CoherentFileListHT * \note : meaningfull only if GetFirstCoherentFileList already called - * @return The next GdcmFileList if found, otherwhise NULL + * @return The next FileList if found, otherwhise NULL */ -GdcmFileList *SerieHelper::GetNextCoherentFileList() +FileList *SerieHelper::GetNextCoherentFileList() { - gdcmAssertMacro (ItListHt != CoherentGdcmFileListHT.end()); + gdcmAssertMacro (ItListHt != CoherentFileListHT.end()); ++ItListHt; - if ( ItListHt != CoherentGdcmFileListHT.end() ) + if ( ItListHt != CoherentFileListHT.end() ) return ItListHt->second; return NULL; } @@ -179,13 +221,13 @@ GdcmFileList *SerieHelper::GetNextCoherentFileList() /** * \brief Get the Coherent Files list according to its Serie UID * @param SerieUID SerieUID - * \return pointer to the Coherent Filseslist if found, otherwhise NULL + * \return pointer to the Coherent Files list if found, otherwhise NULL */ -GdcmFileList *SerieHelper::GetCoherentFileList(std::string SerieUID) +FileList *SerieHelper::GetCoherentFileList(std::string SerieUID) { - if ( CoherentGdcmFileListHT.count(SerieUID) == 0 ) + if ( CoherentFileListHT.count(SerieUID) == 0 ) return 0; - return CoherentGdcmFileListHT[SerieUID]; + return CoherentFileListHT[SerieUID]; } //----------------------------------------------------------------------------- @@ -199,10 +241,11 @@ GdcmFileList *SerieHelper::GetCoherentFileList(std::string SerieUID) * -# Image Position Patient * -# Image Number * -# More to come :-) + * WARNING : FileList = std::vector * @param fileList Coherent File list (same Serie UID) to sort * @return false only if the header is bugged ! */ -bool SerieHelper::ImagePositionPatientOrdering( GdcmFileList *fileList ) +bool SerieHelper::ImagePositionPatientOrdering( FileList *fileList ) //based on Jolinda's algorithm { //iop is calculated based on the file file @@ -216,11 +259,11 @@ bool SerieHelper::ImagePositionPatientOrdering( GdcmFileList *fileList ) std::vector distlist; //!\todo rewrite this for loop. - for ( GdcmFileList::const_iterator + for ( FileList::const_iterator it = fileList->begin(); it != fileList->end(); ++it ) { - if( first ) + if ( first ) { (*it)->GetImageOrientationPatient( cosines ); @@ -242,11 +285,6 @@ bool SerieHelper::ImagePositionPatientOrdering( GdcmFileList *fileList ) dist += normal[i]*ipp[i]; } - if( dist == 0 ) - { - return false; - } - distlist.push_back( dist ); max = min = dist; @@ -264,11 +302,6 @@ bool SerieHelper::ImagePositionPatientOrdering( GdcmFileList *fileList ) dist += normal[i]*ipp[i]; } - if( dist == 0 ) - { - return false; - } - distlist.push_back( dist ); min = (min < dist) ? min : dist; @@ -280,17 +313,24 @@ bool SerieHelper::ImagePositionPatientOrdering( GdcmFileList *fileList ) // Then I order the slices according to the value "dist". Finally, once // I've read in all the slices, I calculate the z-spacing as the difference // between the "dist" values for the first two slices. - GdcmFileVector CoherentGdcmFileVector(n); - // CoherentGdcmFileVector.reserve( n ); - CoherentGdcmFileVector.resize( n ); - // gdcmAssertMacro( CoherentGdcmFileVector.capacity() >= n ); + FileVector CoherentFileVector(n); + // CoherentFileVector.reserve( n ); + CoherentFileVector.resize( n ); + // gdcmAssertMacro( CoherentFileVector.capacity() >= n ); + + // Find out if min/max are coherent + if ( min == max ) + { + gdcmWarningMacro( "Looks like all images have the exact same image position..."); + return false; + } float step = (max - min)/(n - 1); int pos; n = 0; //VC++ don't understand what scope is !! it -> it2 - for (GdcmFileList::const_iterator it2 = fileList->begin(); + for (FileList::const_iterator it2 = fileList->begin(); it2 != fileList->end(); ++it2, ++n) { //2*n sort algo !! @@ -301,11 +341,11 @@ bool SerieHelper::ImagePositionPatientOrdering( GdcmFileList *fileList ) // and images may have differents directions // -> More than one may have the same 'pos' // Sorting has then NO meaning ! - if (CoherentGdcmFileVector[pos]==NULL) - CoherentGdcmFileVector[pos] = *it2; + if (CoherentFileVector[pos]==NULL) + CoherentFileVector[pos] = *it2; else { - gdcmVerboseMacro( "2 files same position"); + gdcmWarningMacro( "2 files same position"); return false; } } @@ -313,14 +353,14 @@ bool SerieHelper::ImagePositionPatientOrdering( GdcmFileList *fileList ) fileList->clear(); // doesn't delete list elements, only node //VC++ don't understand what scope is !! it -> it3 - for (GdcmFileVector::const_iterator it3 = CoherentGdcmFileVector.begin(); - it3 != CoherentGdcmFileVector.end(); ++it3) + for (FileVector::const_iterator it3 = CoherentFileVector.begin(); + it3 != CoherentFileVector.end(); ++it3) { fileList->push_back( *it3 ); } distlist.clear(); - CoherentGdcmFileVector.clear(); + CoherentFileVector.clear(); return true; } @@ -338,12 +378,12 @@ bool SerieHelper::ImageNumberLessThan(File *file1, File *file2) * @param fileList Coherent File list (same Serie UID) to sort * @return false if non nona fide stuff encountered */ -bool SerieHelper::ImageNumberOrdering(GdcmFileList *fileList) +bool SerieHelper::ImageNumberOrdering(FileList *fileList) { int min, max, pos; int n = fileList->size(); - GdcmFileList::const_iterator it = fileList->begin(); + FileList::const_iterator it = fileList->begin(); min = max = (*it)->GetImageNumber(); for (; it != fileList->end(); ++it, ++n) @@ -354,7 +394,7 @@ bool SerieHelper::ImageNumberOrdering(GdcmFileList *fileList) } // Find out if image numbers are coherent (consecutive) - if( min == max || max == 0 || max >= (n+min)) + if ( min == max || max == 0 || max >= (n+min) ) return false; std::sort(fileList->begin(), fileList->end(), SerieHelper::ImageNumberLessThan ); @@ -372,7 +412,7 @@ bool SerieHelper::FileNameLessThan(File *file1, File *file2) * @param fileList Coherent File list (same Serie UID) to sort * @return false only if the header is bugged ! */ -bool SerieHelper::FileNameOrdering(GdcmFileList *fileList) +bool SerieHelper::FileNameOrdering(FileList *fileList) { std::sort(fileList->begin(), fileList->end(), SerieHelper::FileNameLessThan); return true; @@ -383,21 +423,21 @@ bool SerieHelper::FileNameOrdering(GdcmFileList *fileList) /** * \brief Canonical printer. */ -void SerieHelper::Print(std::ostream &os, std::string const & indent) +void SerieHelper::Print(std::ostream &os, std::string const &indent) { // For all the Coherent File lists of the gdcm::Serie - CoherentFileListmap::iterator itl = CoherentGdcmFileListHT.begin(); - if ( itl == CoherentGdcmFileListHT.end() ) + CoherentFileListmap::iterator itl = CoherentFileListHT.begin(); + if ( itl == CoherentFileListHT.end() ) { - gdcmVerboseMacro( "No Coherent File list found" ); + gdcmWarningMacro( "No Coherent File list found" ); return; } - while (itl != CoherentGdcmFileListHT.end()) + while (itl != CoherentFileListHT.end()) { os << "Serie UID :[" << itl->first << "]" << std::endl; // For all the files of a Coherent File list - for (GdcmFileList::iterator it = (itl->second)->begin(); + for (FileList::iterator it = (itl->second)->begin(); it != (itl->second)->end(); ++it) {