From fccdf8e6409f9cf8c2a66a658dcfdfb7a332cbb9 Mon Sep 17 00:00:00 2001
From: malaterre <malaterre>
Date: Fri, 14 Jan 2005 21:30:53 +0000
Subject: [PATCH] ENH: Simplify gdcmSerieHeader API. SerieHeader should focus
 only on sorting DICOM file instead of providing way too many methods

---
 src/gdcmSerieHeader.cxx | 23 ++++-------------------
 src/gdcmSerieHeader.h   | 10 +++-------
 2 files changed, 7 insertions(+), 26 deletions(-)

diff --git a/src/gdcmSerieHeader.cxx b/src/gdcmSerieHeader.cxx
index 6360e0a4..2c61c933 100644
--- a/src/gdcmSerieHeader.cxx
+++ b/src/gdcmSerieHeader.cxx
@@ -3,8 +3,8 @@
   Program:   gdcm
   Module:    $RCSfile: gdcmSerieHeader.cxx,v $
   Language:  C++
-  Date:      $Date: 2005/01/14 21:03:54 $
-  Version:   $Revision: 1.6 $
+  Date:      $Date: 2005/01/14 21:30:53 $
+  Version:   $Revision: 1.7 $
                                                                                 
   Copyright (c) CREATIS (Centre de Recherche et d'Applications en Traitement de
   l'Image). All rights reserved. See Doc/License.txt or
@@ -34,7 +34,7 @@ typedef std::vector<Header* > GdcmHeaderVector;
 SerieHeader::SerieHeader()
 {
    CoherentGdcmFileList.clear();
-   // Later will contains: 0020 000e UI REL Series Instance UID
+   // Later will contain: 0020 000e UI REL Series Instance UID
    CurrentSerieUID = "";
 }
 
@@ -88,28 +88,13 @@ void SerieHeader::AddFileName(std::string const &filename)
    }
 }
 
-/**
- * \brief add a File to the list
- * @param   file Header to add
- */
-void SerieHeader::AddGdcmFile(Header *file)
-{
-   if( file->IsReadable() )
-   {
-      CoherentGdcmFileList.push_back( file );
-   }
-   else
-   {
-      gdcmVerboseMacro("Could not add file: " << file->GetFileName() );
-   }
-}
-
 /**
  * \brief Sets the Directory
  * @param   dir Name of the directory to deal with
  */
 void SerieHeader::SetDirectory(std::string const &dir)
 {
+   CurrentSerieUID = ""; //Reset previous Serie Instance UID
    DirList filenames_list(dir);  //OS specific
   
    for( DirList::const_iterator it = filenames_list.begin(); 
diff --git a/src/gdcmSerieHeader.h b/src/gdcmSerieHeader.h
index 55893b9e..b6a8a76d 100644
--- a/src/gdcmSerieHeader.h
+++ b/src/gdcmSerieHeader.h
@@ -3,8 +3,8 @@
   Program:   gdcm
   Module:    $RCSfile: gdcmSerieHeader.h,v $
   Language:  C++
-  Date:      $Date: 2005/01/14 21:03:55 $
-  Version:   $Revision: 1.5 $
+  Date:      $Date: 2005/01/14 21:30:53 $
+  Version:   $Revision: 1.6 $
                                                                                 
   Copyright (c) CREATIS (Centre de Recherche et d'Applications en Traitement de
   l'Image). All rights reserved. See Doc/License.txt or
@@ -42,16 +42,12 @@ public:
 
    /// \todo should return bool or throw error ?
    void AddFileName(std::string const &filename);
-   void AddGdcmFile(Header *file);
    void SetDirectory(std::string const &dir);
    void OrderGdcmFileList();
    
-   /// \warning Assumes all elements in the list have the same global infos.
-   ///          Assumes the list is not empty.
-   Header *GetGdcmHeader() { return CoherentGdcmFileList.front(); }
-
    /// \brief Gets the *coherent* File List
    /// @return the *coherent* File List
+   /// Caller must call OrderGdcmFileList first
    const GdcmHeaderList &GetGdcmFileList() { return CoherentGdcmFileList; }
 
 private:
-- 
2.48.1