Program: gdcm
Module: $RCSfile: TestCopyRescaleDicom.cxx,v $
Language: C++
- Date: $Date: 2004/12/07 16:55:56 $
- Version: $Revision: 1.1 $
+ Date: $Date: 2007/10/30 09:13:03 $
+ Version: $Revision: 1.23 $
Copyright (c) CREATIS (Centre de Recherche et d'Applications en Traitement de
l'Image). All rights reserved. See Doc/License.txt or
PURPOSE. See the above copyright notices for more information.
=========================================================================*/
-#include "gdcmHeader.h"
#include "gdcmFile.h"
-#include "gdcmValEntry.h"
-#include "gdcmBinEntry.h"
+#include "gdcmFileHelper.h"
+#include "gdcmDataEntry.h"
+
+#include <time.h>
+#include <sys/times.h>
+#include <iomanip> // for std::ios::left, ...
//Generated file:
#include "gdcmDataImages.h"
-bool FileExists(const char* filename);
+#ifndef _WIN32
+#include <unistd.h> //for access, unlink
+#else
+#include <io.h> //for _access on Win32
+#endif
-bool RemoveFile(const char* source);
+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;
+ }
+}
-int CopyRescaleDicom(std::string const & filename,
- std::string const & output )
+bool RemoveFile(const char *source)
+{
+#ifdef _MSC_VER
+#define _unlink unlink
+#endif
+ return unlink(source) != 0 ? false : true;
+}
+
+int CopyRescaleDicom(std::string const &filename,
+ std::string const &output )
{
std::cout << " Testing: " << filename << std::endl;
if( FileExists( output.c_str() ) )
return 1;
}
}
-
+
//////////////// Step 1:
std::cout << " 1...";
- gdcm::Header *originalH = new gdcm::Header( filename );
- gdcm::Header *copyH = new gdcm::Header( );
-
- //First of all copy the header field by field
-
- // Warning :Accessor gdcmElementSet::GetEntry() should not exist
- // It was commented out by Mathieu, that was a *good* idea
- // (the user does NOT have to know the way we implemented the Header !)
- // Waiting for a 'clean' solution, I keep the method ...JPRx
+ GDCM_NAME_SPACE::File *originalF = GDCM_NAME_SPACE::File::New( );
+ originalF->SetFileName( filename );
+ originalF->Load();
+
+ GDCM_NAME_SPACE::File *copyF = GDCM_NAME_SPACE::File::New( );
+ //First of all copy the file, field by field
//////////////// Step 2:
std::cout << "2...";
- originalH->Initialize();
- gdcm::DocEntry* d = originalH->GetNextEntry();
-
+ // Copy of the file content
+ GDCM_NAME_SPACE::DocEntry *d = originalF->GetFirstEntry();
while(d)
{
- if ( gdcm::ValEntry* v = dynamic_cast<gdcm::ValEntry*>(d) )
- {
- copyH->ReplaceOrCreateByNumber(
- v->GetValue(),
- v->GetGroup(),
- v->GetElement(),
- v->GetVR() );
+ if ( GDCM_NAME_SPACE::DataEntry *de = dynamic_cast<GDCM_NAME_SPACE::DataEntry *>(d) )
+ {
+ copyF->InsertEntryBinArea( de->GetBinArea(),de->GetLength(),
+ de->GetGroup(),de->GetElement(),
+ de->GetVR() );
+ }
+ else
+ {
+ // We skip pb of SQ recursive exploration
}
- d=originalH->GetNextEntry();
+ d = originalF->GetNextEntry();
}
- gdcm::File *original = new gdcm::File( originalH );
- gdcm::File *copy = new gdcm::File( copyH );
+ GDCM_NAME_SPACE::FileHelper *original = GDCM_NAME_SPACE::FileHelper::New( originalF );
+ GDCM_NAME_SPACE::FileHelper *copy = GDCM_NAME_SPACE::FileHelper::New( copyF );
size_t dataSize = original->GetImageDataSize();
- uint8_t* imageData = original->GetImageData();
- size_t rescaleSize = dataSize / 2;
-#ifdef VERSION_1
- uint8_t *rescaleImage = new uint8_t[dataSize];
-#endif
-
- //2 case now, if 16 bits input then try to downscale to 8bits just for
- //fun:
-
- const std::string & bitsStored = originalH->GetEntryByNumber(0x0028,0x0101);
-// std::cout << "BitsStored: " << bitsStored << std::endl;
+ size_t rescaleSize;
+ uint8_t *rescaleImage;
+ const std::string &bitsStored = originalF->GetEntryString(0x0028,0x0101);
if( bitsStored == "16" )
{
- uint16_t* imageData16 = (uint16_t*)original->GetImageData();
- copyH->ReplaceOrCreateByNumber( "8", 0x0028, 0x0100); // BitsAllocated
- copyH->ReplaceOrCreateByNumber( "8", 0x0028, 0x0101); // BitsStored
- copyH->ReplaceOrCreateByNumber( "7", 0x0028, 0x0102); // HighBit
- copyH->ReplaceOrCreateByNumber( "0", 0x0028, 0x0103); //Pixel Representation
-
+ std::cout << "Rescale...";
+ copyF->InsertEntryString( "8", 0x0028, 0x0100, "US"); // Bits Allocated
+ copyF->InsertEntryString( "8", 0x0028, 0x0101, "US"); // Bits Stored
+ copyF->InsertEntryString( "7", 0x0028, 0x0102, "US"); // High Bit
+ copyF->InsertEntryString( "0", 0x0028, 0x0103, "US"); // Pixel Representation
+
// We assume the value were from 0 to uint16_t max
+ rescaleSize = dataSize / 2;
+ rescaleImage = new uint8_t[dataSize];
+
+ uint16_t *imageData16 = (uint16_t*)original->GetImageData();
+ uint16_t *tmpImage = imageData16;
+ uint8_t *tmpRescale = rescaleImage;
for(unsigned int i=0; i<rescaleSize; i++)
{
-#ifdef VERSION_1
- rescaleImage[i] = imageData16[i]/256;
-#else
- imageData[i] = imageData16[i]/256;
-#endif
+ *tmpRescale = (uint8_t)( (*tmpImage)>>8 );
+ tmpImage++;
+ tmpRescale++;
}
-#ifdef VERSION_1
- copy->SetImageData(rescaleImage, rescaleSize);
-#endif
}
else
{
- copy->SetImageData(imageData, dataSize);
+ std::cout << "Same...";
+ rescaleSize = dataSize;
+ rescaleImage = new uint8_t[dataSize];
+ memcpy(rescaleImage,original->GetImageData(),dataSize);
}
+ copy->SetImageData(rescaleImage, rescaleSize);
+
//////////////// Step 3:
std::cout << "3...";
copy->SetWriteModeToRGB();
std::cout << " Failed" << std::endl
<< " " << output << " not written" << std::endl;
- delete original;
- delete copy;
- delete originalH;
- delete copyH;
+ delete[] rescaleImage;
return 1;
}
- delete copy;
- delete copyH;
-
//////////////// Step 4:
std::cout << "4...";
- copy = new gdcm::File( output );
+ GDCM_NAME_SPACE::FileHelper *copy2 = GDCM_NAME_SPACE::FileHelper::New( output );
//Is the file written still gdcm parsable ?
- if ( !copy->GetHeader()->IsReadable() )
+ if ( !copy2->GetFile()->IsReadable() )
{
std::cout << " Failed" << std::endl
<< " " << output << " not readable" << std::endl;
- delete original;
- delete originalH;
+ delete[] rescaleImage;
return 1;
}
//////////////// Step 5:
std::cout << "5...";
- size_t dataSizeWritten = copy->GetImageDataSize();
- uint8_t* imageDataWritten = copy->GetImageData();
+ size_t dataSizeWritten = copy2->GetImageDataSize();
+ uint8_t *imageDataWritten = copy2->GetImageData();
- if (dataSize != dataSizeWritten)
+ if (originalF->GetXSize() != copy2->GetFile()->GetXSize() ||
+ originalF->GetYSize() != copy2->GetFile()->GetYSize() ||
+ originalF->GetZSize() != copy2->GetFile()->GetZSize())
+ {
+ std::cout << "Failed" << std::endl
+ << " X Size differs: "
+ << "X: " << originalF->GetXSize() << " # "
+ << copy2->GetFile()->GetXSize() << " | "
+ << "Y: " << originalF->GetYSize() << " # "
+ << copy2->GetFile()->GetYSize() << " | "
+ << "Z: " << originalF->GetZSize() << " # "
+ << copy2->GetFile()->GetZSize() << std::endl;
+ delete[] rescaleImage;
+
+ return 1;
+ }
+
+ if (rescaleSize != dataSizeWritten)
{
std::cout << " Failed" << std::endl
<< " Pixel areas lengths differ: "
<< dataSize << " # " << dataSizeWritten << std::endl;
- delete original;
- delete copy;
- delete originalH;
+ delete[] rescaleImage;
return 1;
}
- if (int res = memcmp(imageData, imageDataWritten, dataSize) !=0)
+ if (int res = memcmp(rescaleImage, imageDataWritten, rescaleSize) !=0)
{
(void)res;
std::cout << " Failed" << std::endl
<< " Pixel differ (as expanded in memory)." << std::endl;
- delete original;
- delete copy;
- delete originalH;
+ delete[] rescaleImage;
return 1;
}
std::cout << "OK." << std::endl ;
- delete original;
- delete copy;
- delete originalH;
-#ifdef VERSION_1
delete[] rescaleImage;
-#endif
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 TestCopyRescaleDicom(int argc, char* argv[])
+int TestCopyRescaleDicom(int argc, char *argv[])
{
if ( argc == 3 )
{
<< std::endl;
std::cout << " apply the following to each filename.xxx: "
<< std::endl;
- std::cout << " step 1: parse the image (as gdcmHeader) and call"
+ 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 datas are set to the copy"
+ << " pixel data are set to the copy"
<< std::endl;
std::cout << " step 3: write the copy of the image"
<< std::endl;
<< " match (as expanded by gdcm)." << std::endl;
std::cout << std::endl;
+
+ clock_t r1,r2, r3,r4;
+ struct tms tms1,tms2, tms3,tms4;
+
+ r3 = times(&tms3);
int i =0;
int retVal = 0; //by default this is an error
while( gdcmDataImages[i] != 0 )
std::string output = "output.dcm";
+ r1 = times(&tms1);
if( CopyRescaleDicom( filename, output ) != 0 )
{
retVal++;
}
+ r2 = times(&tms2);
+
+ std::cout
+ << std::setw(150-strlen(gdcmDataImages[i]))
+ << gdcmDataImages[i] << " user time: "
+ << (long) ((tms2.tms_utime) - (tms1.tms_utime))
+ << " system time: "
+ << (long) ((tms2.tms_stime) - (tms1.tms_stime))
+ << "\t elapsed time: " << r2 - r1
+ << std::endl;
i++;
}
+ r4 = times(&tms4);
+
+ std::cout
+ << std::setw(150-strlen("Gross Total")) << " --> "
+ << "Gross Total" << " user time: "
+ << (long) ((tms4.tms_utime) - (tms3.tms_utime))
+ << " system time: "
+ << (long) ((tms4.tms_stime) - (tms3.tms_stime))
+ << "\t elapsed time: " << (long) (r4 - r3)
+ << std::endl;
return retVal;
}
git://git.creatis.insa-lyon.fr / gdcm.git / blobdiff