X-Git-Url: https://git.creatis.insa-lyon.fr/pubgit/?a=blobdiff_plain;ds=sidebyside;f=src%2FgdcmFile.h;h=6bcd20673ec6a344c3f8969291b580efcb0e2091;hb=3e78befc911eac2ecc70747854911d1f803797b5;hp=2afd67c701ecbeb493bb7a8993e5bfc9c435ef8f;hpb=565a47ec4e037897aaf770aeccc5d67d7ad7b478;p=gdcm.git diff --git a/src/gdcmFile.h b/src/gdcmFile.h index 2afd67c7..6bcd2067 100644 --- a/src/gdcmFile.h +++ b/src/gdcmFile.h @@ -10,79 +10,45 @@ // 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 + std::string OrigFileName; // To avoid file overwrite + void SwapZone(void* im, int swap, int lgr, int nb); + bool ReadPixelData(void * destination); +protected: + int WriteBase(std::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(std::string & filename); + gdcmFile(const char * filename); // For promotion (performs a deepcopy of pointed header object) // TODO Swig gdcmFile(gdcmHeader* header); // TODO Swig ~gdcmFile(); // On writing purposes. When instance was created through - // gdcmFile(string filename) then the filename argument MUST be different - // from the constructor's one (no overwriting allowed). - // TODO Swig int SetFileName(string filename); + // gdcmFile(std::string filename) then the filename argument MUST be + // different from the constructor's one (no overwriting allowed). + // TODO Swig int SetFileName(std::string filename); - // Allocates necessary memory, copies the data (image[s]/volume[s]) to - // newly allocated zone and return a pointer to it: - - void * GetImageData(); - - // Returns size (in bytes) of required memory to contain data - // represented in this file. - + void SetPixelDataSizeFromHeader(void); size_t GetImageDataSize(); - - // 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 :-( - // + void * GetImageData(); + 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 +59,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 (std::string nomFichier); + int WriteDcmImplVR(std::string nomFichier); + int WriteDcmImplVR(const char * nomFichier); + int WriteDcmExplVR(std::string nomFichier); + int WriteAcr (std::string nomFichier); }; #endif