X-Git-Url: https://git.creatis.insa-lyon.fr/pubgit/?a=blobdiff_plain;f=Testing%2FTestCopyDicom.cxx;h=698828b8cf20a010ca6abaf501af361d52f8e0a5;hb=1e66f7398c38a3d71b480e0297e29636ea6e1c1b;hp=0225453919dc50341f12ec1d33418a71f5663b89;hpb=4511c4982e71b3a90859cc7d7cb44e9a6cb588bc;p=gdcm.git diff --git a/Testing/TestCopyDicom.cxx b/Testing/TestCopyDicom.cxx index 02254539..698828b8 100644 --- a/Testing/TestCopyDicom.cxx +++ b/Testing/TestCopyDicom.cxx @@ -1,79 +1,222 @@ -#include "gdcmHeader.h" +/*========================================================================= + + Program: gdcm + Module: $RCSfile: TestCopyDicom.cxx,v $ + Language: C++ + Date: $Date: 2007/10/30 09:13:45 $ + Version: $Revision: 1.44 $ + + Copyright (c) CREATIS (Centre de Recherche et d'Applications en Traitement de + l'Image). All rights reserved. See Doc/License.txt or + http://www.creatis.insa-lyon.fr/Public/Gdcm/License.html for details. + + This software is distributed WITHOUT ANY WARRANTY; without even + the implied warranty of MERCHANTABILITY or FITNESS FOR A PARTICULAR + PURPOSE. See the above copyright notices for more information. + +=========================================================================*/ #include "gdcmFile.h" +#include "gdcmFileHelper.h" +#include "gdcmDataEntry.h" + +//Generated file: +#include "gdcmDataImages.h" // return true if the file exists -bool FileExists(const char* filename) -{ -#ifdef _MSC_VER -# define access _access -#endif -#ifndef R_OK -# define R_OK 04 -#endif - if ( access(filename, R_OK) != 0 ) - { - return false; - } - else - { - return true; - } -} +bool FileExists(const char *filename); +bool RemoveFile(const char *source); -bool RemoveFile(const char* source) +int CopyDicom(std::string const &filename, + std::string const &output ) { -#ifdef _MSC_VER -#define _unlink unlink -#endif - return unlink(source) != 0 ? false : true; -} + std::cout << " Testing: " << filename << std::endl; + if( FileExists( output.c_str() ) ) + { + if( !RemoveFile( output.c_str() ) ) + { + std::cout << "Ouch, the file exist, but I cannot remove it" << std::endl; + return 1; + } + } -// Here we load a gdcmFile and then try to create from scratch a copy of it, -// copying field by field the dicom image + //////////////// Step 1: + std::cout << " 1..."; + GDCM_NAME_SPACE::File *originalH = GDCM_NAME_SPACE::File::New(); + GDCM_NAME_SPACE::File *copyH = GDCM_NAME_SPACE::File::New(); -int TestCopyDicom(int argc, char* argv[]) -{ - if (argc < 3) - { - std::cerr << "Usage :" << std::endl << - argv[0] << " input_dicom output_dicom" << std::endl; - return 1; - } - if( FileExists( argv[2] ) ) - { - std::cerr << "Don't try to cheat, I am removing the file anyway" << std::endl; - if( !RemoveFile( argv[2] ) ) + originalH->SetFileName( filename ); + originalH->Load( ); + //First of all copy the file, field by field + + //////////////// Step 2: + std::cout << "2..."; + GDCM_NAME_SPACE::DocEntry *d=originalH->GetFirstEntry(); + while(d) { - std::cerr << "Ouch, the file exist, but I cannot remove it" << std::endl; + if ( GDCM_NAME_SPACE::DataEntry *de = dynamic_cast(d) ) + { + copyH->InsertEntryBinArea( de->GetBinArea(),de->GetLength(), + de->GetGroup(),de->GetElement(), + de->GetVR() ); + } + else + { + // We skip pb of SQ recursive exploration + } + + d=originalH->GetNextEntry(); + } + GDCM_NAME_SPACE::FileHelper *original = GDCM_NAME_SPACE::FileHelper::New(originalH); + GDCM_NAME_SPACE::FileHelper *copy = GDCM_NAME_SPACE::FileHelper::New(copyH); + + size_t dataSize = original->GetImageDataSize(); + uint8_t *imageData = original->GetImageData(); + + // Useless to set the image data, because it's already made when + // copying the corresponding DataEntry that contains the pixel data + + // --> FIXME : Why do we let the following line? + // to avoid compile time warnings? + copy->SetImageData(imageData, dataSize); + + //////////////// Step 3: + std::cout << "3..."; + copy->SetWriteModeToRGB(); + if( !copy->WriteDcmExplVR(output) ) + { + std::cout << " Failed" << std::endl + << " " << output << " not written" << std::endl; + + original->Delete(); + copy->Delete(); + originalH->Delete(); + copyH->Delete(); + return 1; } - } - gdcmFile *original = new gdcmFile( argv[1] ); - gdcmFile *copy = new gdcmFile( argv[2] ); - //First of all copy the header field by field - TagNameHT & nameHt = original->GetHeader()->GetPubDict()->GetEntriesByName(); - for (TagNameHT::iterator tag = nameHt.begin(); tag != nameHt.end(); ++tag) - { - std::cerr << "Reading: " << tag->second->GetVR() << std::endl; + copy->Delete(); + copyH->Delete(); - copy->GetHeader()->ReplaceOrCreateByNumber( tag->second->GetVR(), - tag->second->GetGroup(), tag->second->GetElement() ); - } + //////////////// Step 4: + std::cout << "4..."; +// copy = new GDCM_NAME_SPACE::FileHelper( output ); + copy = GDCM_NAME_SPACE::FileHelper::New(output); // ??? + //Is the file written still gdcm parsable ? + if ( !copy->GetFile()->IsReadable() ) + { + std::cout << " Failed" << std::endl + << " " << output << " not readable" << std::endl; - size_t dataSize = original->GetImageDataSize(); - void *imageData = original->GetImageData(); + original->Delete(); + originalH->Delete(); - copy->SetImageData(imageData, dataSize); - //original->GetHeader()->SetImageDataSize(dataSize); + return 1; + } + + //////////////// Step 5: + std::cout << "5..."; + size_t dataSizeWritten = copy->GetImageDataSize(); + uint8_t *imageDataWritten = copy->GetImageData(); - //copy->GetHeader()->PrintEntry(); + if (dataSize != dataSizeWritten) + { + std::cout << " Failed" << std::endl + << " Pixel areas lengths differ: " + << dataSize << " # " << dataSizeWritten << std::endl; - copy->WriteDcmExplVR( argv[2] ); + original->Delete(); + copy->Delete(); + originalH->Delete(); - return 0; + return 1; + } + + if (int res = memcmp(imageData, imageDataWritten, dataSize) !=0) + { + (void)res; + std::cout << " Failed" << std::endl + << " Pixel differ (as expanded in memory)." << std::endl; + + original->Delete(); + copy->Delete(); + originalH->Delete(); + + return 1; + } + std::cout << "OK." << std::endl ; + + original->Delete(); + copy->Delete(); + originalH->Delete(); + + return 0; } +// Here we load a gdcmFile and then try to create from scratch a copy of it, +// copying field by field the dicom image + +int TestCopyDicom(int argc, char *argv[]) +{ + if ( argc == 3 ) + { + // The test is specified a specific filename, use it instead of looping + // over all images + const std::string input = argv[1]; + const std::string reference = argv[2]; + return CopyDicom( input, reference ); + } + else if ( argc > 3 || argc == 2 ) + { + std::cout << " Usage: " << argv[0] + << " (no arguments needed)." << std::endl; + std::cout << "or Usage: " << argv[0] + << " filename.dcm reference.dcm" << std::endl; + return 1; + } + // else other cases: + + std::cout << " Description (Test::TestCopyDicom): " + << std::endl; + std::cout << " For all images in gdcmData (and not blacklisted in " + "Test/CMakeLists.txt)" + << std::endl; + std::cout << " apply the following to each filename.xxx: " + << std::endl; + std::cout << " step 1: parse the image (as gdcmFile) and call" + << " IsReadable(). After that, call GetImageData() and " + << "GetImageDataSize() " + << std::endl; + std::cout << " step 2: create a copy of the readed file and the new" + << " pixel data are set to the copy" + << std::endl; + std::cout << " step 3: write the copy of the image" + << std::endl; + std::cout << " step 4: read the copy and call IsReadable()" + << std::endl; + std::cout << " step 5: compare (in memory with memcmp) that the two " + << "images " << std::endl + << " match (as expanded by gdcm)." << std::endl; + std::cout << std::endl; + + int i =0; + int retVal = 0; //by default this is an error + while( gdcmDataImages[i] != 0 ) + { + std::string filename = GDCM_DATA_ROOT; + filename += "/"; //doh! + filename += gdcmDataImages[i]; + + std::string output = "output.dcm"; + if( CopyDicom( filename, output ) != 0 ) + { + retVal++; + } + + i++; + } + return retVal; +}