-#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 "gdcmDocument.h"
-#include "gdcmValEntry.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;
-}
-
-// 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 , char* [])
-{
- 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::cerr << "Filename: " << filename << std::endl;
-
- std::string output = "../Testing/Temporary/output.dcm";
-
+ std::cout << " Testing: " << filename << std::endl;
if( FileExists( output.c_str() ) )
{
- std::cerr << "Don't try to cheat, I am removing the file anyway" << std::endl;
if( !RemoveFile( output.c_str() ) )
{
- std::cerr << "Ouch, the file exist, but I cannot remove it" << std::endl;
+ std::cout << "Ouch, the file exist, but I cannot remove it" << std::endl;
return 1;
}
}
- gdcmFile *original = new gdcmFile( filename );
- gdcmFile *copy = new gdcmFile( output );
+ //////////////// 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();
+
- //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)
+ 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)
{
- if (tag->second->GetVR() == "SQ") //to skip pb of SQ recursive exploration
- continue;
-
- //According to JPR I should also skip those:
-// if (tag->second->GetVR() == "unkn") //to skip pb of SQ recursive exploration
-// continue;
-
- //no clue what to do with this one
- //std::cerr << "Reading: " << tag->second->GetVR() << std::endl;
- std::string value = ((gdcmValEntry*)(tag->second))->GetValue();
-// if( value.find( "gdcm::NotLoaded" ) == 0 )
-// continue;
-
- //no clue what to do with this one
- if( value.find( "gdcm::Loaded" ) == 0 )
- continue;
-
- copy->GetHeader()->ReplaceOrCreateByNumber(
- value,
- tag->second->GetGroup(),
- tag->second->GetElement() );
+ if ( GDCM_NAME_SPACE::DataEntry *de = dynamic_cast<GDCM_NAME_SPACE::DataEntry *>(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();
- void *imageData = original->GetImageData();
+ 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);
- //original->GetHeader()->SetImageDataSize(dataSize);
- //copy->GetHeader()->PrintEntry();
+ //////////////// 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();
- //copy->WriteDcmExplVR( output );
-
+ return 1;
+ }
+
+ copy->Delete();
+ copyH->Delete();
+
+ //////////////// 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 ?
- gdcmFile check( output );
- retVal += !check.GetHeader()->IsReadable();
- }
+ if ( !copy->GetFile()->IsReadable() )
+ {
+ std::cout << " Failed" << std::endl
+ << " " << output << " not readable" << std::endl;
- return retVal;
+ original->Delete();
+ originalH->Delete();
+
+ return 1;
+ }
+
+ //////////////// Step 5:
+ std::cout << "5...";
+ size_t dataSizeWritten = copy->GetImageDataSize();
+ uint8_t *imageDataWritten = copy->GetImageData();
+
+ if (dataSize != dataSizeWritten)
+ {
+ std::cout << " Failed" << std::endl
+ << " Pixel areas lengths differ: "
+ << dataSize << " # " << dataSizeWritten << std::endl;
+
+ original->Delete();
+ copy->Delete();
+ originalH->Delete();
+
+ 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;
+}
git://git.creatis.insa-lyon.fr / gdcm.git / blobdiff