Program: gdcm
Module: $RCSfile: gdcmSerieHelper.cxx,v $
Language: C++
- Date: $Date: 2005/10/17 09:52:41 $
- Version: $Revision: 1.21 $
+ Date: $Date: 2005/11/08 16:31:21 $
+ Version: $Revision: 1.34 $
Copyright (c) CREATIS (Centre de Recherche et d'Applications en Traitement de
l'Image). All rights reserved. See Doc/License.txt or
#include <vector>
#include <map>
#include <algorithm>
+#include <stdio.h> //for sscanf
namespace gdcm
{
}
/**
- * \brief - Preventively, clear everything at constructor time.
- * - use it at destructor time.
+ * \brief Preventively, clear everything at constructor time.
+ * ( use it at destructor time.)
*/
void SerieHelper::ClearAll()
{
it != l->end();
++it)
{
- delete *it; // remove each entry
+ (*it)->Delete(); // remove each entry
}
l->clear();
delete l; // remove the container
void SerieHelper::AddFileName(std::string const &filename)
{
// Create a DICOM file
- File *header = new File ();
+ File *header = File::New();
header->SetLoadMode(LoadMode);
header->SetFileName( filename );
header->Load();
++it2)
{
const ExRule &r = *it2;
- s = header->GetEntryValue( r.group, r.elem );
+ s = header->GetEntryString( r.group, r.elem );
if ( !Util::CompareDicomString(s, r.value.c_str(), r.op) )
{
// Argh ! This rule is unmatched; let's just quit
+
allrules = 0;
break;
}
// Let's add it!
// 0020 000e UI REL Series Instance UID
- const std::string &uid = header->GetEntryValue (0x0020, 0x000e);
+ const std::string &uid = header->GetEntryString(0x0020, 0x000e);
// if uid == GDCM_UNFOUND then consistently we should find GDCM_UNFOUND
// no need here to do anything special
+
if ( SingleSerieUIDFileSetHT.count(uid) == 0 )
{
gdcmDebugMacro(" New Serie UID :[" << uid << "]");
else
{
// at least one rule was unmatched we need to deallocate the file:
- delete header;
+ header->Delete();
}
}
else
{
gdcmWarningMacro("Could not read file: " << filename );
- delete header;
+ header->Delete();
}
}
* of the gdcm::SerieHelper.
* \warning : this method should be used by aware users only!
* Passing a gdcm::File* has the same effect than passing a file name!
- * \TODO : decide which one is wrong (the method, or the commentary)!
+ * \todo : decide which one is wrong (the method, or the commentary)!
* the following comment doesn't match the method :-(
* User is supposed to know the files he want to deal with
* and consider them they belong to the same Serie
* @param value value to be checked to exclude File
* @param op operator we want to use to check
*/
-void SerieHelper::AddRestriction(uint16_t group, uint16_t elem,
+void SerieHelper::AddRestriction(TagKey const &key,
std::string const &value, int op)
{
ExRule r;
- r.group = group;
- r.elem = elem;
+ r.group = key[0];
+ r.elem = key[1];
r.value = value;
r.op = op;
ExRestrictions.push_back( r );
}
#ifndef GDCM_LEGACY_REMOVE
-/**
- * \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
- * @param group Group number of the target tag.
- * @param elem Element number of the target tag.
- * @param value value to be checked to exclude File
- * @deprecated use : AddRestriction(uint16_t group, uint16_t elem,
- * std::string const &value, int op);
- */
void SerieHelper::AddRestriction(TagKey const &key, std::string const &value)
{
Rule r;
FileList::const_iterator it = fileSet->begin();
- int nX = (*it)->GetXSize();
- int nY = (*it)->GetYSize();
- int pixelSize = (*it)->GetPixelSize();
-
+ int nX = (*it)->GetXSize();
+ int nY = (*it)->GetYSize();
+ int pixelSize = (*it)->GetPixelSize();
+ bool signedPixelData = (*it)->IsSignedPixelData();
it ++;
for ( ;
it != fileSet->end();
return false;
if ( (*it)->GetPixelSize() != pixelSize )
return false;
- // probabely more is to be checked (?)
+ if ( (*it)->IsSignedPixelData() != signedPixelData )
+ return false;
+ // probabely more is to be checked (?)
}
return true;
}
#ifndef GDCM_LEGACY_REMOVE
-/**
- * \brief accessor (DEPRECATED : use GetFirstSingleSerieUIDFileSet )
- * Warning : 'coherent' means here they have the same Serie UID
- * @return The first FileList if found, otherwhise NULL
- */
+
FileList *SerieHelper::GetFirstCoherentFileList()
{
ItFileSetHt = SingleSerieUIDFileSetHT.begin();
return NULL;
}
-/**
- * \brief accessor (DEPRECATED : use GetNextSingleSerieUIDFileSet )
- * Warning : 'coherent' means here they have the same Serie UID
- * \note : meaningfull only if GetFirstCoherentFileList() already called
- * @return The next FileList if found, otherwhise NULL
- */
+
FileList *SerieHelper::GetNextCoherentFileList()
{
gdcmAssertMacro (ItFileSetHt != SingleSerieUIDFileSetHT.end());
return NULL;
}
-/**
- * \brief accessor (DEPRECATED : use GetSingleSerieUIDFileSet )
- * Warning : 'coherent' means here they have the same Serie UID
- * @param SerieUID SerieUID
- * \return pointer to the FileList if found, otherwhise NULL
- */
+
FileList *SerieHelper::GetCoherentFileList(std::string SerieUID)
{
if ( SingleSerieUIDFileSetHT.count(SerieUID) == 0 )
if (nb == 0 )
return CoherentFileSet;
float iop[6];
- std::ostringstream ossOrient;
+
std::string strOrient;
-
+ std::ostringstream ossOrient;
FileList::const_iterator it = fileSet->begin();
it ++;
for ( ;
// Let's build again the 'cosines' string, to be sure of it's format
(*it)->GetImageOrientationPatient(iop);
+
ossOrient << iop[0];
for (int i = 1; i < 6; i++)
{
ossOrient << iop[i];
}
strOrient = ossOrient.str();
-
+ ossOrient.str("");
+ // FIXME : is it a 'cleaner' way to initialize an ostringstream?
+
if ( CoherentFileSet.count(strOrient) == 0 )
{
gdcmDebugMacro(" New Orientation :[" << strOrient << "]");
}
// Current Orientation and DICOM header match; add the file:
CoherentFileSet[strOrient]->push_back( (*it) );
- }
+ }
return CoherentFileSet;
}
/**
- * \brief Splits a Single SerieUID Fileset according to the Positions
+ * \brief Splits a 'Single SerieUID' Fileset according to the Positions
* @param fileSet File Set to be splitted
* \return std::map of 'Xcoherent' File sets
*/
// 0020,0032 : Image Position Patient
// 0020,0030 : Image Position (RET)
- std::string strImPos = (*it)->GetEntryValue(0x0020,0x0032);
+ strImPos = (*it)->GetEntryString(0x0020,0x0032);
if ( strImPos == GDCM_UNFOUND)
{
gdcmWarningMacro( "Unfound Image Position Patient (0020,0032)");
- strImPos = (*it)->GetEntryValue(0x0020,0x0030); // For ACR-NEMA images
+ strImPos = (*it)->GetEntryString(0x0020,0x0030); // For ACR-NEMA images
if ( strImPos == GDCM_UNFOUND )
{
gdcmWarningMacro( "Unfound Image Position (RET) (0020,0030)");
ossPosition << pos[i];
}
strPosition = ossPosition.str();
-
+ ossPosition.str("");
+
if ( CoherentFileSet.count(strPosition) == 0 )
{
gdcmDebugMacro(" New Position :[" << strPosition << "]");
}
/**
- * \brief Splits a SingleSerieUID File set Coherent according to the
+ * \brief Splits a 'Single SerieUID' File set Coherent according to the
* value of a given Tag
* @param fileSet File Set to be splitted
+ * @param group group number of the target Element
+ * @param elem element number of the target Element
* \return std::map of 'Xcoherent' File sets
*/
XCoherentFileSetmap SerieHelper::SplitOnTagValue(FileList *fileSet,
- uint16_t group, uint16_t element)
+ uint16_t group, uint16_t elem)
{
XCoherentFileSetmap CoherentFileSet;
// 0020,0032 : Image Position Patient
// 0020,0030 : Image Position (RET)
- std::string strTagValue = (*it)->GetEntryValue(group,element);
+ strTagValue = (*it)->GetEntryString(group,elem);
if ( CoherentFileSet.count(strTagValue) == 0 )
{
}
// Current Tag value and DICOM header match; add the file:
CoherentFileSet[strTagValue]->push_back( (*it) );
- }
+ }
return CoherentFileSet;
}
* We may order, considering :
* -# Image Position Patient
* -# Image Number
+ * -# File Name
* -# More to come :-)
- * WARNING : FileList = std::vector<File* >
+ * \note : FileList = std::vector<File* >
* @param fileList Coherent File list (same Serie UID) to sort
* @return false only if the header is bugged !
*/
* \note Works only on bona fide files (i.e image number is a character string
* corresponding to an integer)
* within a bona fide serie (i.e image numbers are consecutive)
- * @param fileList Coherent File list (same Serie UID) to sort
+ * @param fileList File set (same Serie UID) to sort
* @return false if non bona fide stuff encountered
*/
bool SerieHelper::ImageNumberOrdering(FileList *fileList)