X-Git-Url: https://git.creatis.insa-lyon.fr/pubgit/?a=blobdiff_plain;ds=sidebyside;f=src%2FgdcmFile.h;h=b81587600ed500b90f84b91217bed357c4ca7bc8;hb=1988dee6507ec70166b38c7e0cac6c1e9c18351e;hp=2afd67c701ecbeb493bb7a8993e5bfc9c435ef8f;hpb=565a47ec4e037897aaf770aeccc5d67d7ad7b478;p=gdcm.git diff --git a/src/gdcmFile.h b/src/gdcmFile.h index 2afd67c7..b8158760 100644 --- a/src/gdcmFile.h +++ b/src/gdcmFile.h @@ -10,40 +10,21 @@ // In addition to Dicom header exploration, this class is designed // for accessing the image/volume content. One can also use it to // write Dicom files. -////// QUESTION: this looks still like an open question whether the -////// relationship between a gdcmFile and gdcmHeader is of -////// type IS_A or HAS_A ! class GDCM_EXPORT gdcmFile: public gdcmHeader { private: - // QUESTION : - // Data pointe sur quoi? - // sur les Pixels lus? - // --> j'ajoute un champ public : Pixels - // (il faudra que l'utilisateur puisse modifier les pixels ?) - - void* Data; + void* PixelData; + size_t lgrTotale; int Parsed; // weather allready parsed string OrigFileName; // To avoid file overwrite + void SwapZone(void* im, int swap, int lgr, int nb); + bool ReadPixelData(void * destination); +protected: + int WriteBase(string FileName, FileType type); public: - // je ne suis pas sur d'avoir compris *où* il serait légitime de ranger ca. - // on pourra tjs le deplacer, et mettre des accesseurs - void * Pixels; - size_t lgrTotale; - - // Constructor dedicated to writing a new DICOMV3 part10 compliant - // file (see SetFileName, SetDcmTag and Write) - // TODO Swig gdcmFile(); - // 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). - gdcmFile(string & filename); + gdcmFile(const char * filename); // For promotion (performs a deepcopy of pointed header object) // TODO Swig gdcmFile(gdcmHeader* header); @@ -54,35 +35,28 @@ public: // from the constructor's one (no overwriting allowed). // TODO Swig int SetFileName(string filename); - // Allocates necessary memory, copies the data (image[s]/volume[s]) to - // newly allocated zone and return a pointer to it: - - void * GetImageData(); - + void SetPixelDataSizeFromHeader(void); // Returns size (in bytes) of required memory to contain data // represented in this file. - size_t GetImageDataSize(); + // Allocates necessary memory, copies the data (image[s]/volume[s]) to + // newly allocated zone and return a pointer to it: + void * GetImageData(); + // Copies (at most MaxSize bytes) of data to caller's memory space. // Returns an error code on failure (if MaxSize is not big enough) - - int GetImageDataIntoVector(void* destination, size_t MaxSize ); - - // Question : - // - // GetImageData et GetImageDataIntoVector - // Get et Put pour 2 fonctions qui font presque la meme chose :-( - // + size_t GetImageDataIntoVector(void* destination, size_t MaxSize ); // Allocates ExpectedSize bytes of memory at this->Data and copies the - // pointed data to it. - - // Question : - // Pourquoi dupliquer les pixels, alors qu'on les a deja en mémoire, - // et que Data (dans le gdcmHeader) est un pointeur ? - - // TODO Swig int SetImageData(void * Data, size_t ExpectedSize); + // pointed data to it. Copying the image might look useless but + // the caller might destroy it's image (without knowing it: think + // of a complicated interface where display is done with a library + // e.g. VTK) before calling the Write + int SetImageData (void * Data, size_t ExpectedSize); + // When the caller is aware we simply point to the data: + // TODO int SetImageDataNoCopy (void * Data, size_t ExpectedSize); + void SetImageDataSize (size_t ExpectedSize); // Push to disk. // A NE PAS OUBLIER : que fait-on en cas de Transfert Syntax (dans l'entete) @@ -93,8 +67,11 @@ public: // Aucun test n'est fait sur l'"Endiannerie" du processeur. // Ca sera à l'utilisateur d'appeler son Reader correctement - int WriteRawData (string nomFichier); - int WriteDcm (string nomFichier); + int WriteRawData (string nomFichier); + int WriteDcmImplVR(string nomFichier); + int WriteDcmImplVR(const char * nomFichier); + int WriteDcmExplVR(string nomFichier); + int WriteAcr (string nomFichier); }; #endif