]> Creatis software - gdcm.git/commitdiff
* src/gdcmSQItem[h|cxx] : rename some methods to have coherent names
authorregrain <regrain>
Thu, 20 Jan 2005 11:26:17 +0000 (11:26 +0000)
committerregrain <regrain>
Thu, 20 Jan 2005 11:26:17 +0000 (11:26 +0000)
   -- BeNours

ChangeLog
Example/TestPapyrus.cxx
src/gdcmDicomDir.cxx
src/gdcmDocument.cxx
src/gdcmSeqEntry.cxx
src/gdcmSeqEntry.h

index a05be787d2a9df1165d75032f1d2ca33332f18c2..bd622b9bdd1afaef1b9ad826d726c1adbcbcdfdc 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,6 @@
+2005-01-20 Benoit Regrain <Benoit.Regrain@creatis.insa-lyon.fr>
+   * src/gdcmSQItem[h|cxx] : rename some methods to have coherent names
+
 2005-01-20 Benoit Regrain <Benoit.Regrain@creatis.insa-lyon.fr>
    * src/gdcmDicomDir*.[h|cxx] : rename methods to be logik in their name.
      Remove all access to the hash-tables or lists in the objects.
index 893ab8cd1277d30a8dfad9df3b5de88a48592fd2..fbb05c69442b77ccb2386189367781f8a41a1c1e 100644 (file)
@@ -3,8 +3,8 @@
   Program:   gdcm
   Module:    $RCSfile: TestPapyrus.cxx,v $
   Language:  C++
-  Date:      $Date: 2005/01/19 17:49:42 $
-  Version:   $Revision: 1.1 $
+  Date:      $Date: 2005/01/20 11:26:17 $
+  Version:   $Revision: 1.2 $
                                                                                 
   Copyright (c) CREATIS (Centre de Recherche et d'Applications en Traitement de
   l'Image). All rights reserved. See Doc/License.txt or
@@ -100,7 +100,7 @@ int main(int argc, char* argv[])
       return 1;
    }
 
-   gdcm::SQItem *sqi = seqPapyrus->GetFirstEntry();
+   gdcm::SQItem *sqi = seqPapyrus->GetFirstSQItem();
    if (sqi == 0)
    {
       std::cout << "NO SQItem found within private Papyrus Sequence"
@@ -163,7 +163,7 @@ int main(int argc, char* argv[])
    while (sqi)
    {
       nbImages++;
-      sqi =  seqPapyrus->GetNextEntry();
+      sqi =  seqPapyrus->GetNextSQItem();
    }
    std::cout <<"Number of frames :" << nbImages << std::endl;  
 
@@ -189,7 +189,7 @@ int main(int argc, char* argv[])
 
    uint32_t offset;
    std::string previousRows = Rows;
-   sqi = seqPapyrus->GetFirstEntry();
+   sqi = seqPapyrus->GetFirstSQItem();
    while (sqi)
    {
       std::cout << "One more image read. Keep waiting" << std::endl;
@@ -210,7 +210,7 @@ int main(int argc, char* argv[])
       currentPosition +=lgrImage;
 
       std::string previousRowNb = Rows;
-      sqi =  seqPapyrus->GetNextEntry();
+      sqi =  seqPapyrus->GetNextSQItem();
    }
 
    // build up a new File, with file info + images info + global pixel area.
index 9f05141bcf99c667e3405ce41cbc190659e9ffa4..4d2973c9d95b9d61bd2a1ee8f128401ff57f6168 100644 (file)
@@ -3,8 +3,8 @@
   Program:   gdcm
   Module:    $RCSfile: gdcmDicomDir.cxx,v $
   Language:  C++
-  Date:      $Date: 2005/01/20 11:09:23 $
-  Version:   $Revision: 1.108 $
+  Date:      $Date: 2005/01/20 11:26:17 $
+  Version:   $Revision: 1.109 $
   
   Copyright (c) CREATIS (Centre de Recherche et d'Applications en Traitement de
   l'Image). All rights reserved. See Doc/License.txt or
@@ -762,15 +762,16 @@ void DicomDir::CreateDicomDir()
 
    NewMeta();
 
-   ListSQItem listItems = s->GetSQItems();
+   //ListSQItem listItems = s->GetSQItems();
    
    DocEntry *d;
    std::string v;
    SQItem *si;
-   for( ListSQItem::iterator i = listItems.begin(); 
-                             i !=listItems.end(); ++i ) 
+
+   SQItem *tmpSI=s->GetFirstSQItem();
+   while(tmpSI)
    {
-      d = (*i)->GetDocEntry(0x0004, 0x1430); // Directory Record Type
+      d = tmpSI->GetDocEntry(0x0004, 0x1430); // Directory Record Type
       if ( ValEntry* valEntry = dynamic_cast<ValEntry *>(d) )
       {
          v = valEntry->GetValue();
@@ -829,7 +830,9 @@ void DicomDir::CreateDicomDir()
       }
 
       if( si )
-         MoveSQItem(si,*i);
+         MoveSQItem(si,tmpSI);
+
+      tmpSI=s->GetNextSQItem();
    }
    TagHT.clear();
 }
index 6e3d862b03ade9e44e2ea34db24dc338eaeedaea..369dc0de68e9fc34118918c08056a17613dc37ae 100644 (file)
@@ -3,8 +3,8 @@
   Program:   gdcm
   Module:    $RCSfile: gdcmDocument.cxx,v $
   Language:  C++
-  Date:      $Date: 2005/01/20 11:07:07 $
-  Version:   $Revision: 1.201 $
+  Date:      $Date: 2005/01/20 11:26:17 $
+  Version:   $Revision: 1.202 $
                                                                                 
   Copyright (c) CREATIS (Centre de Recherche et d'Applications en Traitement de
   l'Image). All rights reserved. See Doc/License.txt or
@@ -1313,7 +1313,7 @@ void Document::ParseSQ( SeqEntry *seqEntry,
       {
          if ( newDocEntry->IsSequenceDelimitor() )
          {
-            seqEntry->SetSequenceDelimitationItem( newDocEntry ); 
+            seqEntry->SetDelimitationItem( newDocEntry ); 
             break;
          }
       }
@@ -1356,7 +1356,7 @@ void Document::ParseSQ( SeqEntry *seqEntry,
       offsetStartCurrentSQItem = Fp->tellg();
       // end try -----------------
  
-      seqEntry->AddEntry( itemSQ, SQItemNumber ); 
+      seqEntry->AddSQItem( itemSQ, SQItemNumber ); 
       SQItemNumber++;
       if ( !delim_mode && ((long)(Fp->tellg())-offset ) >= l_max )
       {
index 243ae59cf7474991c58c8fb40d28fa9fa5da7496..ec1a6d3de42e8d3b5cfd981fbcd8e3a957c62538 100644 (file)
@@ -3,8 +3,8 @@
   Program:   gdcm
   Module:    $RCSfile: gdcmSeqEntry.cxx,v $
   Language:  C++
-  Date:      $Date: 2005/01/18 14:28:32 $
-  Version:   $Revision: 1.47 $
+  Date:      $Date: 2005/01/20 11:26:18 $
+  Version:   $Revision: 1.48 $
                                                                                 
   Copyright (c) CREATIS (Centre de Recherche et d'Applications en Traitement de
   l'Image). All rights reserved. See Doc/License.txt or
@@ -78,6 +78,8 @@ SeqEntry::~SeqEntry()
    }
 }
 
+//-----------------------------------------------------------------------------
+// Print
 /**
  * \brief   canonical Printer
  */
@@ -119,6 +121,8 @@ void SeqEntry::Print( std::ostream &os, std::string const & )
    }                    
 }
 
+//-----------------------------------------------------------------------------
+// Public
 /*
  * \brief   canonical Writer
  */
@@ -146,23 +150,42 @@ void SeqEntry::WriteContent(std::ofstream *fp, FileType filetype)
    binary_write(*fp, seq_term_lg);
 }
 
-//-----------------------------------------------------------------------------
-// Public
+/**
+ * \brief   Get the first entry while visiting the SeqEntry
+ * \return  The first SQItem if found, otherwhise NULL
+ */ 
+SQItem *SeqEntry::GetFirstSQItem()
+{
+   ItSQItem = Items.begin();
+   if (ItSQItem != Items.end())
+      return *ItSQItem;
+   return NULL;
+} 
 
-/// \brief   adds the passed ITEM to the ITEM chained List for this SeQuence.
-void SeqEntry::AddEntry(SQItem *sqItem, int itemNumber)
+/**
+ * \brief   Get the next SQItem while visiting the SeqEntry
+ * \note : meaningfull only if GetFirstEntry already called
+ * \return  The next SQItem if found, otherwhise NULL
+ */
+
+SQItem *SeqEntry::GetNextSQItem()
 {
-   sqItem->SetSQItemNumber(itemNumber);
-   Items.push_back(sqItem);
+   gdcmAssertMacro (ItSQItem != Items.end())
+   {
+      ++ItSQItem;
+      if (ItSQItem != Items.end())
+         return *ItSQItem;
+   }
+   return NULL;
 }
-
 /**
  * \brief return a pointer to the SQItem referenced by its ordinal number.
  *        Returns the first item when argument is negative.
  *        Returns the last item when argument is bigger than the total
  *        item number.
  */
-SQItem *SeqEntry::GetSQItemByOrdinalNumber(int nb)
+SQItem *SeqEntry::GetSQItem(int nb)
 {
    if (nb<0)
    {
@@ -181,35 +204,18 @@ SQItem *SeqEntry::GetSQItemByOrdinalNumber(int nb)
    return *(Items.end()); // Euhhhhh ?!? Is this the last one . FIXME
 }
 
-/**
- * \brief   Get the first entry while visiting the SeqEntry
- * \return  The first SQItem if found, otherwhise NULL
- */ 
-SQItem *SeqEntry::GetFirstEntry()
+unsigned int SeqEntry::GetNumberOfSQItems()
 {
-   ItSQItem = Items.begin();
-   if (ItSQItem != Items.end())
-      return *ItSQItem;
-   return NULL;
-} 
-
-/**
- * \brief   Get the next SQItem while visiting the SeqEntry
- * \note : meaningfull only if GetFirstEntry already called
- * \return  The next SQItem if found, otherwhise NULL
- */
+   return Items.size();
+}
 
-SQItem *SeqEntry::GetNextEntry()
+/// \brief   adds the passed ITEM to the ITEM chained List for this SeQuence.
+void SeqEntry::AddSQItem(SQItem *sqItem, int itemNumber)
 {
-   gdcmAssertMacro (ItSQItem != Items.end())
-   {
-      ++ItSQItem;
-      if (ItSQItem != Items.end())
-         return *ItSQItem;
-   }
-   return NULL;
+   sqItem->SetSQItemNumber(itemNumber);
+   Items.push_back(sqItem);
 }
+
 //-----------------------------------------------------------------------------
 // Protected
 
index e8ce9c7fd2a1592c1d318d2302fc351b4279321a..6676c754abb40bafe4f26058a35c207aaad8cb1e 100644 (file)
@@ -3,8 +3,8 @@
   Program:   gdcm
   Module:    $RCSfile: gdcmSeqEntry.h,v $
   Language:  C++
-  Date:      $Date: 2005/01/18 12:16:10 $
-  Version:   $Revision: 1.29 $
+  Date:      $Date: 2005/01/20 11:26:18 $
+  Version:   $Revision: 1.30 $
                                                                                 
   Copyright (c) CREATIS (Centre de Recherche et d'Applications en Traitement de
   l'Image). All rights reserved. See Doc/License.txt or
@@ -47,18 +47,20 @@ public:
    void WriteContent(std::ofstream *fp, FileType filetype);
 
    /// returns the SQITEM chained List for this SeQuence.
-   ListSQItem const &GetSQItems() const { return Items; }
-   SQItem *GetFirstEntry();
-   SQItem *GetNextEntry();
+   //ListSQItem const &GetSQItems() const { return Items; }
+   SQItem *GetFirstSQItem();
+   SQItem *GetNextSQItem();
+   SQItem *GetSQItem(int itemNumber);
+   unsigned int GetNumberOfSQItems();
       
    /// Sets the delimitor mode
    void SetDelimitorMode(bool dm) { DelimitorMode = dm; }
 
    /// Sets the Sequence Delimitation Item
-   void SetSequenceDelimitationItem(DocEntry *e) { SeqTerm = e;}
+   void SetDelimitationItem(DocEntry *e) { SeqTerm = e;}
+   DocEntry *GetDelimitationItem()       { return SeqTerm;}
 
-   void AddEntry(SQItem *it, int itemNumber);
-   SQItem *GetSQItemByOrdinalNumber(int itemNumber);
+   void AddSQItem(SQItem *it, int itemNumber);
 
    /// Gets the depth level
    int GetDepthLevel() const { return SQDepthLevel; }