* file (see SetFileName, SetDcmTag and Write)
* Opens (in read only and when possible) an existing file and checks
* for DICOM compliance. Returns NULL on failure.
- * \Note the in-memory representation of all available tags found in
+ * \note the in-memory representation of all available tags found in
* the DICOM header is post-poned to first header information access.
* This avoid a double parsing of public part of the header when
* one sets an a posteriori shadow dictionary (efficiency can be
* seen as a side effect).
- *
* @param header file to be opened for reading datas
- *
* @return
*/
* file (see SetFileName, SetDcmTag and Write)
* Opens (in read only and when possible) an existing file and checks
* for DICOM compliance. Returns NULL on failure.
- * \Note the in-memory representation of all available tags found in
+ * \note the in-memory representation of all available tags found in
* the DICOM header is post-poned to first header information access.
* This avoid a double parsing of public part of the header when
* one sets an a posteriori shadow dictionary (efficiency can be
* seen as a side effect).
* @param filename file to be opened for parsing
- * @return
*/
gdcmFile::gdcmFile(std::string & filename) {
Header=new gdcmHeader(filename.c_str());
SetPixelDataSizeFromHeader();
}
-gdcmFile::gdcmFile(const char * filename) {
+/**
+ * \ingroup gdcmFile
+ * \brief Constructor dedicated to writing a new DICOMV3 part10 compliant
+ * file (see SetFileName, SetDcmTag and Write)
+ * Opens (in read only and when possible) an existing file and checks
+ * for DICOM compliance. Returns NULL on failure.
+ * \note the in-memory representation of all available tags found in
+ * the DICOM header is post-poned to first header information access.
+ * This avoid a double parsing of public part of the header when
+ * one sets an a posteriori shadow dictionary (efficiency can be
+ * seen as a side effect).
+ * @param filename file to be opened for parsing
+ */
+ gdcmFile::gdcmFile(const char * filename) {
Header=new gdcmHeader(filename);
SelfHeader=true;
PixelRead=-1; // no ImageData read yet.
/**
* \ingroup gdcmFile
* \brief Returns the size (in bytes) of required memory to hold
- * \ the pixel data represented in this file, when user DOESN'T want
- * \ to get RGB pixels image when it's stored as a PALETTE COLOR image
- * \ - the (vtk) user is supposed to know how deal with LUTs -
+ * the pixel data represented in this file, when user DOESN'T want
+ * to get RGB pixels image when it's stored as a PALETTE COLOR image
+ * -the (vtk) user is supposed to know how deal with LUTs-
* \warning to be used with GetImagePixelsRaw()
* @return The size of pixel data in bytes.
*/
* Transforms 3 planes R, G, B into a single RGB Plane
* Transforms single Grey plane + 3 Palettes into a RGB Plane
* @return Pointer to newly allocated pixel data.
- * \ NULL if alloc fails
+ * NULL if alloc fails
*/
void * gdcmFile::GetImageData (void) {
PixelData = (void *) malloc(lgrTotale);
* \ingroup gdcmFile
* \brief TODO JPR
* \warning doit-etre etre publique ?
- * TODO : y a-t-il un inconvenient à fusioner ces 2 fonctions
+ * \toto : y a-t-il un inconvenient à fusioner ces 2 fonctions
*
* @param inData
* @param ExpectedSize
* \ingroup gdcmFile
* \brief Writes on disk A SINGLE Dicom file
* NO test is performed on processor "Endiannity".
- * Ca sera à l'utilisateur d'appeler son Reader correctement
- * (Equivalent a IdImaWriteRawFile)
- *
+ * It's up to the user to call his Reader properly
* @param fileName name of the file to be created
* (any already existing file is over written)
* @return false if write fails
FILE * fp1;
fp1 = fopen(fileName.c_str(),"wb");
if (fp1 == NULL) {
- printf("Echec ouverture (ecriture) Fichier [%s] \n",fileName.c_str());
+ printf("Fail to open (write) file [%s] \n",fileName.c_str());
return (false);
}
fwrite (PixelData,lgrTotale, 1, fp1);
* \ingroup gdcmFile
* \brief Writes on disk A SINGLE Dicom file
* NO test is performed on processor "Endiannity".
- *
* @param fileName name of the file to be created
* (any already existing file is overwritten)
* @return false if write fails
/**
* \ingroup gdcmFile
*
-* @param fileName name of the file to be created
+ * @param fileName name of the file to be created
* (any already existing file is overwritten)
* @param type file type (ExplicitVR, ImplicitVR, ...)
* @return false if write fails
bool gdcmFile::WriteBase (std::string fileName, FileType type) {
FILE * fp1;
+
+ if (PixelRead==-1 && type != DICOMDIR) {
+/* std::cout << "U never Read the pixels; U cannot write the file"
+ << std::endl;*/
+ return false;
+ }
+
fp1 = fopen(fileName.c_str(),"wb");
if (fp1 == NULL) {
printf("Failed to open (write) File [%s] \n",fileName.c_str());
// if recognition code tells us we dealt with a LibIDO image
// we reproduce on disk the switch between lineNumber and columnNumber
// just before writting ...
+
+ // TODO : the best trick would be *change* the recognition code
+ // but pb expected if user deals with, e.g. COMPLEX images
std::string rows, columns;
if ( Header->GetFileType() == ACR_LIBIDO){
}
// ----------------- End of Special Patch ----------------
- // TODO : get the grPixel, numPixel values
- guint16 grPixel =0x7fe0;
- guint16 numPixel=0x0010;
+ // TODO : get the grPixel, numPixel values (for some ACR-NEMA images only)
- IterHT p;
+ guint16 grPixel =Header->GetGrPixel();
+ guint16 numPixel=Header->GetNumPixel();;
+
+ // Update Pixel Data Length
+ // the *last* of the (GrPixel, NumPixel), if many.
+
TagKey key = gdcmDictEntry::TranslateToKey(grPixel, numPixel);
- gdcmHeaderEntry * a;
TagHeaderEntryHT::iterator p2;
+ gdcmHeaderEntry * PixelElement;
- //IterHT it = GetHeaderEntrySameNumber(grPixel,numPixel);
- // Update Pixel Data Length
- // the *last* of the 7fe0,0010, if many.
-
-/*
- // good looking, but it doesn't work
- p= Header->GetEntry().equal_range(key); // get a pair of iterators first-last synonym
- p2=p.second; // iterator on the last synonym
- a=p2->second; // H Table target column (2-nd col)
- //a->SetLength(lgrTotale);
- a->SetPrintLevel(2);
- a->Print();
- // use the old -wrong if many 7fe0,0010- way
- */
+ IterHT it= Header->GetEntry().equal_range(key); // get a pair of iterators first-last synonym
+
+ if (Header->GetEntry().count(key) == 1) // only the first is significant
+ p2=it.first; // iterator on the first (unique) synonym
+ else
+ p2=it.second;// iterator on the last synonym
+
+ PixelElement=p2->second; // H Table target column (2-nd col)
+ // PixelElement->SetPrintLevel(2);
+ // PixelElement->Print();
+
if (PixelRead==1)
- Header->SetEntryLengthByNumber(lgrTotaleRaw,grPixel, numPixel);
+ PixelElement->SetLength(lgrTotaleRaw);
else if (PixelRead==0)
- Header->SetEntryLengthByNumber(lgrTotale, grPixel, numPixel);
- // if == -1 : no Pixel Data was read : abort the method
+ PixelElement->SetLength(lgrTotale);
+ //PixelElement->SetPrintLevel(2);
+ //PixelElement->Print();
+
Header->Write(fp1, type);
// --------------------------------------------------------------