X-Git-Url: https://git.creatis.insa-lyon.fr/pubgit/?a=blobdiff_plain;f=src%2FgdcmFile.cxx;h=b21463b038b616c13f9afe77a69684ef461a12db;hb=3705811f0b24769b2222f7d5ad2b14f8fd932104;hp=2d76f3f0a5e4a38d9ad4734b02a6bd4192e8ff32;hpb=8e5be16bcfd2ba5a93e2692d39add1bc7bfedaf8;p=gdcm.git diff --git a/src/gdcmFile.cxx b/src/gdcmFile.cxx index 2d76f3f0..b21463b0 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/25 14:36:30 $ - Version: $Revision: 1.294 $ + Date: $Date: 2005/11/05 13:24:39 $ + Version: $Revision: 1.302 $ Copyright (c) CREATIS (Centre de Recherche et d'Applications en Traitement de l'Image). All rights reserved. See Doc/License.txt or @@ -19,21 +19,21 @@ // // -------------- Remember ! ---------------------------------- // -// Image Position Patient (0020,0032): +// Image Position (Patient) (0020,0032): // If not found (ACR_NEMA) we try Image Position (0020,0030) // If not found (ACR-NEMA), we consider Slice Location (0020,1041) // or Location (0020,0050) // as the Z coordinate, // 0. for all the coordinates if nothing is found // -// Image Position (Patient) (0020,0032) VM=3 What is it used for? +// Image Position (Patient) (0020,0032) VM=3 // --> // The attribute Patient Orientation (0020,0020) from the General Image Module // is of type 2C and has the condition Required if image does not require // Image Orientation (0020,0037) and Image Position (0020,0032). // However, if the image does require the attributes // - Image Orientation (Patient) (0020,0037), VM=6 -// - Image Position Patient (0020,0032), VM=3 +// - Image Position (Patient) (0020,0032), VM=3 // then attribute Patient Orientation (0020,0020) should not be present // in the images. // @@ -65,7 +65,7 @@ // FFDR = Feet First-Decubitus Right // FFDL = Feet First-Decubitus Left -// we can also find +// we can also find (non standard!) // SEMIERECT // SUPINE @@ -469,9 +469,13 @@ float File::GetXSpacing() { nbValue = entry->GetValueCount(); // Can't use IsValueCountValid because of the complex heuristic. + if( nbValue !=2 ) + gdcmWarningMacro("ImagerPixelSpacing (0x0018,0x1164) " + << "has a wrong number of values :" << nbValue); + if( nbValue >= 3 ) xspacing = (float)entry->GetValue(2); - if( nbValue >= 2 ) + else if( nbValue >= 2 ) xspacing = (float)entry->GetValue(1); else xspacing = (float)entry->GetValue(0); @@ -489,6 +493,10 @@ float File::GetXSpacing() if( entry ) { nbValue = entry->GetValueCount(); + if( nbValue !=2 ) + gdcmWarningMacro("PixelSpacing (0x0018,0x0030) " + << "has a wrong number of values :" << nbValue); + if( nbValue >= 3 ) xspacing = (float)entry->GetValue(2); else if( nbValue >= 2 ) @@ -631,7 +639,10 @@ float File::GetXOrigin() if( entry->GetValueCount() == 3 ) { - gdcmErrorMacro( entry->IsValueCountValid() ); + if (!entry->IsValueCountValid() ) + { + gdcmErrorMacro( "Invalid Value Count" ); + } return (float)entry->GetValue(0); } return 0.0f; @@ -659,7 +670,10 @@ float File::GetYOrigin() if( entry->GetValueCount() == 3 ) { - gdcmErrorMacro( entry->IsValueCountValid() ); + if (!entry->IsValueCountValid() ) + { + gdcmErrorMacro( "Invalid Value Count" ); + } return (float)entry->GetValue(1); } return 0.0f; @@ -680,7 +694,10 @@ float File::GetZOrigin() { if( entry->GetValueCount() == 3 ) { - gdcmErrorMacro( entry->IsValueCountValid() ); + if (!entry->IsValueCountValid() ) + { + gdcmErrorMacro( "Invalid Value Count" ); + } return (float)entry->GetValue(2); } gdcmWarningMacro( "Wrong Image Position Patient (0020,0032)"); @@ -692,7 +709,10 @@ float File::GetZOrigin() { if( entry->GetValueCount() == 3 ) { - gdcmErrorMacro( entry->IsValueCountValid() ); + if (!entry->IsValueCountValid() ) + { + gdcmErrorMacro( "Invalid Value Count" ); + } return (float)entry->GetValue(2); } gdcmWarningMacro( "Wrong Image Position (RET) (0020,0030)"); @@ -705,7 +725,10 @@ float File::GetZOrigin() { if( entry->GetValueCount() == 1 ) { - gdcmErrorMacro( entry->IsValueCountValid() ); + if (!entry->IsValueCountValid() ) + { + gdcmErrorMacro( "Invalid Value Count" ); + } return (float)entry->GetValue(0); // VM=1 ! } gdcmWarningMacro( "Wrong Slice Location (0020,1041)"); @@ -717,7 +740,10 @@ float File::GetZOrigin() { if( entry->GetValueCount() == 1 ) { - gdcmErrorMacro( entry->IsValueCountValid() ); + if (!entry->IsValueCountValid() ) + { + gdcmErrorMacro( "Invalid Value Count" ); + } return (float)entry->GetValue(0); } gdcmWarningMacro( "Wrong Location (0020,0050)"); @@ -845,7 +871,6 @@ int File::GetPlanarConfiguration() DataEntry *entry = GetDataEntry(0x0028,0x0006); if( !entry ) { - gdcmWarningMacro( "Not found : Planar Configuration (0028,0006)"); return 0; } return (int)entry->GetValue(0); @@ -1454,19 +1479,18 @@ bool File::Write(std::string fileName, FileType writetype) e0000->SetString(sLen.str()); } - // Derma?.dcm does not have it...let's remove it FIXME FIXME + // FIXME : Derma?.dcm does not have it...let's remove it ?!? JPRx if( writetype != JPEG ) - { - int i_lgPix = GetEntryLength(GrPixel, NumPixel); - if (i_lgPix != -2) - { - // no (GrPixel, NumPixel) element - std::string s_lgPix = Util::Format("%d", i_lgPix+12); - s_lgPix = Util::DicomString( s_lgPix.c_str() ); - InsertEntryString(s_lgPix,GrPixel, 0x0000); - } - } - + { + int i_lgPix = GetEntryLength(GrPixel, NumPixel); + if (i_lgPix != -2) + { + // no (GrPixel, NumPixel) element + std::string s_lgPix = Util::Format("%d", i_lgPix+12); + s_lgPix = Util::DicomString( s_lgPix.c_str() ); + InsertEntryString(s_lgPix,GrPixel, 0x0000); + } + } Document::WriteContent(fp, writetype); fp->close(); @@ -1777,9 +1801,9 @@ void File::ReadEncapsulatedBasicOffsetTable() uint32_t val = BasicOffsetTableItemValue[i]; BasicOffsetTableItemValue[i] = ( (val<<24) | ((val<<8) & 0x00ff0000) | - ((val>>8) & 0x0000ff00) | (val>>24) ); + ( (val>>8) & 0x0000ff00) | (val>>24) ); #endif - gdcmWarningMacro( "Read one length for: " << + gdcmDebugMacro( "Read one length for: " << std::hex << BasicOffsetTableItemValue[i] ); } @@ -1790,21 +1814,6 @@ 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 - */ -File::File( std::string const &filename ) - :Document( ) -{ - RLEInfo = new RLEFramesInfo; - JPEGInfo = new JPEGFragmentsInfo; - - SetFileName( 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