X-Git-Url: https://git.creatis.insa-lyon.fr/pubgit/?a=blobdiff_plain;f=src%2FgdcmFile.cxx;h=b26c57d831b59495ceb2d1c672dbccbe757b29c4;hb=9dbffd306a2b7981cf033ab8402c774f8f970475;hp=88986f539279d14b3a622de674fde247a40d9515;hpb=6a7c2fbcb5bfc240a3b9875ad4836f5dc058e069;p=gdcm.git diff --git a/src/gdcmFile.cxx b/src/gdcmFile.cxx index 88986f53..b26c57d8 100644 --- a/src/gdcmFile.cxx +++ b/src/gdcmFile.cxx @@ -3,8 +3,8 @@ Program: gdcm Module: $RCSfile: gdcmFile.cxx,v $ Language: C++ - Date: $Date: 2005/10/20 15:24:09 $ - Version: $Revision: 1.284 $ + Date: $Date: 2005/10/23 15:26:43 $ + Version: $Revision: 1.289 $ Copyright (c) CREATIS (Centre de Recherche et d'Applications en Traitement de l'Image). All rights reserved. See Doc/License.txt or @@ -467,6 +467,7 @@ float File::GetXSpacing() if( entry ) { nbValue = entry->GetValueCount(); + // Can't use IsValueCountValid because of the complex heuristic. if( nbValue >= 3 ) xspacing = (float)entry->GetValue(2); if( nbValue >= 2 ) @@ -483,25 +484,6 @@ float File::GetXSpacing() gdcmWarningMacro( "Unfound Imager Pixel Spacing (0018,1164)" ); } -/* const std::string &strImagerPixelSpacing = GetEntryString(0x0018,0x1164); - if ( strImagerPixelSpacing != GDCM_UNFOUND ) - { - if ( ( nbValues = sscanf( strImagerPixelSpacing.c_str(), - "%f\\%f", &yspacing, &xspacing)) != 2 ) - { - // if no values, xspacing is set to 1.0 - if ( nbValues == 0 ) - xspacing = 1.0; - // if single value is found, xspacing is defaulted to yspacing - if ( nbValues == 1 ) - xspacing = yspacing; - - if ( xspacing == 0.0 ) - xspacing = 1.0; - } - return xspacing; - }*/ - entry = GetDataEntry(0x0028,0x0030); if( entry ) { @@ -522,41 +504,6 @@ float File::GetXSpacing() gdcmWarningMacro( "Unfound Pixel Spacing (0028,0030)" ); } -/* const std::string &strSpacing = GetEntryString(0x0028,0x0030); - if ( strSpacing == GDCM_UNFOUND ) - { - gdcmWarningMacro( "Unfound Pixel Spacing (0028,0030)" ); - return 1.; - } - - if ( ( nbValues = sscanf( strSpacing.c_str(), - "%f \\%f ", &yspacing, &xspacing)) != 2 ) - { - // if no values, xspacing is set to 1.0 - if ( nbValues == 0 ) - xspacing = 1.0; - // if single value is found, xspacing is defaulted to yspacing - if ( nbValues == 1 ) - xspacing = yspacing; - - if ( xspacing == 0.0 ) - xspacing = 1.0; - return xspacing; - - }*/ - - // to avoid troubles with David Clunie's-like images (at least one) -/* if ( xspacing == 0.0 && yspacing == 0.0) - return 1.0; - - if ( xspacing == 0.0) - { - gdcmWarningMacro("gdcmData/CT-MONO2-8-abdo.dcm-like problem"); - // seems to be a bug in the header ... - nbValues = sscanf( strSpacing.c_str(), "%f \\0\\%f ", &yspacing, &xspacing); - gdcmAssertMacro( nbValues == 2 ); - } -*/ return xspacing; } @@ -584,20 +531,6 @@ float File::GetYSpacing() { gdcmWarningMacro( "Unfound Imager Pixel Spacing (0018,1164)" ); } -/* const std::string &strImagerPixelSpacing = GetEntryString(0x0018,0x1164); - if ( strImagerPixelSpacing != GDCM_UNFOUND ) - { - nbValues = sscanf( strImagerPixelSpacing.c_str(), "%f", &yspacing); - - // if sscanf cannot read any float value, it won't affect yspacing - if ( nbValues == 0 ) - yspacing = 1.0; - - if ( yspacing == 0.0 ) - yspacing = 1.0; - - return yspacing; - }*/ entry = GetDataEntry(0x0028,0x0030); if( entry ) @@ -612,22 +545,6 @@ float File::GetYSpacing() { gdcmWarningMacro( "Unfound Pixel Spacing (0028,0030)" ); } -/* std::string strSpacing = GetEntryString(0x0028,0x0030); - if ( strSpacing == GDCM_UNFOUND ) - { - gdcmWarningMacro("Unfound Pixel Spacing (0028,0030)"); - return 1.; - } - - // if sscanf cannot read any float value, it won't affect yspacing - nbValues = sscanf( strSpacing.c_str(), "%f", &yspacing); - - // if no values, yspacing is set to 1.0 - if ( nbValues == 0 ) - yspacing = 1.0; - - if ( yspacing == 0.0 ) - yspacing = 1.0;*/ return yspacing; } @@ -689,31 +606,6 @@ float File::GetZSpacing() gdcmWarningMacro("Unfound Slice Thickness (0018,0050)"); return zspacing; - -/* const std::string &strSpacingBSlices = GetEntryString(0x0018,0x0088); - if ( strSpacingBSlices == GDCM_UNFOUND ) - { - gdcmWarningMacro("Unfound Spacing Between Slices (0018,0088)"); - const std::string &strSliceThickness = GetEntryString(0x0018,0x0050); - if ( strSliceThickness == GDCM_UNFOUND ) - { - gdcmWarningMacro("Unfound Slice Thickness (0018,0050)"); - return 1.0; - } - else - { - // if no 'Spacing Between Slices' is found, - // we assume slices join together - // (no overlapping, no interslice gap) - // if they don't, we're fucked up - return (float)atof( strSliceThickness.c_str() ); - } - } - - float zsp = (float)atof( strSpacingBSlices.c_str()); - if (zsp == 0.0) // last change not to break further computations ... - zsp = 1.0; - return zsp;*/ } /** @@ -737,27 +629,11 @@ float File::GetXOrigin() } if( entry->GetValueCount() == 3 ) - return (float)entry->GetValue(0); - return 0.0f; - -/* std::string strImPos = GetEntryString(0x0020,0x0032); - if ( strImPos == GDCM_UNFOUND ) { - gdcmWarningMacro( "Unfound Image Position Patient (0020,0032)"); - strImPos = GetEntryString(0x0020,0x0030); // For ACR-NEMA images - if ( strImPos == GDCM_UNFOUND ) - { - gdcmWarningMacro( "Unfound Image Position (RET) (0020,0030)"); - return 0.0; - } - } - - if ( sscanf( strImPos.c_str(), "%f \\%f \\%f ", &xImPos, &yImPos, &zImPos) != 3 ) - { - return 0.0; + gdcmAssertMacro( entry->IsValueCountValid() ); + return (float)entry->GetValue(0); } - - return xImPos;*/ + return 0.0f; } /** @@ -781,28 +657,11 @@ float File::GetYOrigin() } if( entry->GetValueCount() == 3 ) - return (float)entry->GetValue(1); - return 0.0f; -/* float xImPos, yImPos, zImPos; - std::string strImPos = GetEntryString(0x0020,0x0032); - - if ( strImPos == GDCM_UNFOUND) - { - gdcmWarningMacro( "Unfound Image Position Patient (0020,0032)"); - strImPos = GetEntryString(0x0020,0x0030); // For ACR-NEMA images - if ( strImPos == GDCM_UNFOUND ) - { - gdcmWarningMacro( "Unfound Image Position (RET) (0020,0030)"); - return 0.; - } - } - - if ( sscanf( strImPos.c_str(), "%f \\%f \\%f ", &xImPos, &yImPos, &zImPos) != 3 ) { - return 0.; + gdcmAssertMacro( entry->IsValueCountValid() ); + return (float)entry->GetValue(1); } - - return yImPos;*/ + return 0.0f; } /** @@ -819,97 +678,50 @@ float File::GetZOrigin() if( entry ) { if( entry->GetValueCount() == 3 ) - return (float)entry->GetValue(0); + { + gdcmAssertMacro( entry->IsValueCountValid() ); + return (float)entry->GetValue(2); + } gdcmWarningMacro( "Wrong Image Position Patient (0020,0032)"); return 0.0f; } -/* float xImPos, yImPos, zImPos; - std::string strImPos = GetEntryString(0x0020,0x0032); - - if ( strImPos != GDCM_UNFOUND ) - { - if ( sscanf( strImPos.c_str(), "%f \\%f \\%f ", &xImPos, &yImPos, &zImPos) != 3) - { - gdcmWarningMacro( "Wrong Image Position Patient (0020,0032)"); - return 0.; // bug in the element 0x0020,0x0032 - } - else - { - return zImPos; - } - }*/ entry = GetDataEntry(0x0020,0x0030); if( entry ) { if( entry->GetValueCount() == 3 ) - return (float)entry->GetValue(0); + { + gdcmAssertMacro( entry->IsValueCountValid() ); + return (float)entry->GetValue(2); + } gdcmWarningMacro( "Wrong Image Position (RET) (0020,0030)"); return 0.0f; } -/* strImPos = GetEntryString(0x0020,0x0030); // For ACR-NEMA images - if ( strImPos != GDCM_UNFOUND ) - { - if ( sscanf( strImPos.c_str(), - "%f \\%f \\%f ", &xImPos, &yImPos, &zImPos ) != 3 ) - { - gdcmWarningMacro( "Wrong Image Position (RET) (0020,0030)"); - return 0.; // bug in the element 0x0020,0x0032 - } - else - { - return zImPos; - } - }*/ // for *very* old ACR-NEMA images entry = GetDataEntry(0x0020,0x1041); if( entry ) { if( entry->GetValueCount() == 1 ) - return (float)entry->GetValue(0); - gdcmWarningMacro( "Wrong Slice Location (0020,1041)"); - return 0.0f; - } -/* std::string strSliceLocation = GetEntryString(0x0020,0x1041); - if ( strSliceLocation != GDCM_UNFOUND ) - { - if ( sscanf( strSliceLocation.c_str(), "%f ", &zImPos) != 1) - { - gdcmWarningMacro( "Wrong Slice Location (0020,1041)"); - return 0.; // bug in the element 0x0020,0x1041 - } - else { - return zImPos; + gdcmAssertMacro( entry->IsValueCountValid() ); + return (float)entry->GetValue(0); // VM=1 ! } + gdcmWarningMacro( "Wrong Slice Location (0020,1041)"); + return 0.0f; } - gdcmWarningMacro( "Unfound Slice Location (0020,1041)");*/ entry = GetDataEntry(0x0020,0x0050); if( entry ) { - gdcmWarningMacro( "Unfound Location (0020,0050)"); if( entry->GetValueCount() == 1 ) + { + gdcmAssertMacro( entry->IsValueCountValid() ); return (float)entry->GetValue(0); + } gdcmWarningMacro( "Wrong Location (0020,0050)"); return 0.0f; } -/* std::string strLocation = GetEntryString(0x0020,0x0050); - if ( strLocation != GDCM_UNFOUND ) - { - if ( sscanf( strLocation.c_str(), "%f ", &zImPos) != 1 ) - { - gdcmWarningMacro( "Wrong Location (0020,0050)"); - return 0.; // bug in the element 0x0020,0x0050 - } - else - { - return zImPos; - } - } - gdcmWarningMacro( "Unfound Location (0020,0050)");*/ - return 0.; // Hopeless } @@ -953,8 +765,6 @@ bool File::GetImageOrientationPatient( float iop[6] ) return true; } - - /** * \brief Retrieve the number of Bits Stored (actually used) * (as opposed to number of Bits Allocated) @@ -970,15 +780,6 @@ int File::GetBitsStored() return 0; } return (int)entry->GetValue(0); - -/* std::string strSize = GetEntryString( 0x0028, 0x0101 ); - if ( strSize == GDCM_UNFOUND ) - { - gdcmWarningMacro("(0028,0101) is supposed to be mandatory"); - return 0; // It's supposed to be mandatory - // the caller will have to check - } - return atoi( strSize.c_str() );*/ } /** @@ -996,15 +797,6 @@ int File::GetBitsAllocated() return 0; } return (int)entry->GetValue(0); - -/* std::string strSize = GetEntryString(0x0028,0x0100); - if ( strSize == GDCM_UNFOUND ) - { - gdcmWarningMacro( "(0028,0100) is supposed to be mandatory"); - return 0; // It's supposed to be mandatory - // the caller will have to check - } - return atoi( strSize.c_str() );*/ } /** @@ -1022,14 +814,6 @@ int File::GetHighBitPosition() return 0; } return (int)entry->GetValue(0); - -/* std::string strSize = GetEntryString( 0x0028, 0x0102 ); - if ( strSize == GDCM_UNFOUND ) - { - gdcmWarningMacro( "(0028,0102) is supposed to be mandatory"); - return 0; - } - return atoi( strSize.c_str() );*/ } /** @@ -1048,15 +832,6 @@ int File::GetSamplesPerPixel() // but sometimes it's missing : *we* assume Gray pixels } return (int)entry->GetValue(0); - -/* const std::string &strSize = GetEntryString(0x0028,0x0002); - if ( strSize == GDCM_UNFOUND ) - { - gdcmWarningMacro( "(0028,0002) is supposed to be mandatory"); - return 1; // Well, it's supposed to be mandatory ... - // but sometimes it's missing : *we* assume Gray pixels - } - return atoi( strSize.c_str() );*/ } /** @@ -1073,14 +848,6 @@ int File::GetPlanarConfiguration() return 0; } return (int)entry->GetValue(0); - -/* std::string strSize = GetEntryString(0x0028,0x0006); - if ( strSize == GDCM_UNFOUND ) - { - gdcmWarningMacro( "Not found : Planar Configuration (0028,0006)"); - return 0; - } - return atoi( strSize.c_str() );*/ } /** @@ -1163,44 +930,6 @@ std::string File::GetPixelType() { sign = "U"; } - -/* std::string bitsAlloc = GetEntryString(0x0028, 0x0100); // Bits Allocated - if ( bitsAlloc == GDCM_UNFOUND ) - { - gdcmWarningMacro( "Missing Bits Allocated (0028,0100)"); - bitsAlloc = "16"; // default and arbitrary value, not to polute the output - } - - if ( bitsAlloc == "64" ) - { - return "FD"; - } - else if ( bitsAlloc == "12" ) - { - // It will be unpacked - bitsAlloc = "16"; - } - else if ( bitsAlloc == "24" ) - { - // (in order no to be messed up by old RGB images) - bitsAlloc = "8"; - } - - std::string sign = GetEntryString(0x0028, 0x0103);//"Pixel Representation" - - if (sign == GDCM_UNFOUND ) - { - gdcmWarningMacro( "Missing Pixel Representation (0028,0103)"); - sign = "U"; // default and arbitrary value, not to polute the output - } - else if ( sign == "0" ) - { - sign = "U"; - } - else - { - sign = "S"; - }*/ return bitsAlloc + sign; } @@ -1221,18 +950,6 @@ bool File::IsSignedPixelData() return false; } return entry->GetValue(0) != 0; -/* std::string strSign = GetEntryString( 0x0028, 0x0103 ); - if ( strSign == GDCM_UNFOUND ) - { - gdcmWarningMacro( "(0028,0103) is supposed to be mandatory"); - return false; - } - int sign = atoi( strSign.c_str() ); - if ( sign == 0 ) - { - return false; - } - return true;*/ } /** @@ -1406,19 +1123,6 @@ float File::GetRescaleIntercept() } return (float)entry->GetValue(0); -/* float resInter = 0.; - /// 0028 1052 DS IMG Rescale Intercept - const std::string &strRescInter = GetEntryString(0x0028,0x1052); - if ( strRescInter != GDCM_UNFOUND ) - { - if ( sscanf( strRescInter.c_str(), "%f ", &resInter) != 1 ) - { - // bug in the element 0x0028,0x1052 - gdcmWarningMacro( "Rescale Intercept (0028,1052) is empty." ); - } - } - - return resInter;*/ } /** @@ -1435,19 +1139,6 @@ float File::GetRescaleSlope() return 1.0f; } return (float)entry->GetValue(0); -/* float resSlope = 1.; - //0028 1053 DS IMG Rescale Slope - std::string strRescSlope = GetEntryString(0x0028,0x1053); - if ( strRescSlope != GDCM_UNFOUND ) - { - if ( sscanf( strRescSlope.c_str(), "%f ", &resSlope) != 1 ) - { - // bug in the element 0x0028,0x1053 - gdcmWarningMacro( "Rescale Slope (0028,1053) is empty."); - } - } - - return resSlope;*/ } /** @@ -2094,10 +1785,10 @@ void File::ReadEncapsulatedBasicOffsetTable() // These are the deprecated method that one day should be removed (after the next release) #ifndef GDCM_LEGACY_REMOVE -/** - * \brief Constructor (DEPRECATED : temporaryly kept not to break the API) - * @param filename name of the file whose header we want to analyze - * @deprecated do not use any longer +/* + * brief Constructor (DEPRECATED : temporaryly kept not to break the API) + * param filename name of the file whose header we want to analyze + * deprecated do not use any longer */ File::File( std::string const &filename ) :Document( ) @@ -2109,12 +1800,12 @@ File::File( std::string const &filename ) Load( ); // gdcm::Document is first Loaded, then the 'File part' } -/** - * \brief Loader. (DEPRECATED : temporaryly kept not to break the API) - * @param fileName file to be open for parsing - * @return false if file cannot be open or no swap info was found, +/* + * \ brief Loader. (DEPRECATED : temporaryly kept not to break the API) + * @ param fileName file to be open for parsing + * @ return false if file cannot be open or no swap info was found, * or no tag was found. - * @deprecated Use the Load() [ + SetLoadMode() ] + SetFileName() functions instead + * @ deprecated Use the Load() [ + SetLoadMode() ] + SetFileName() functions instead */ bool File::Load( std::string const &fileName ) {