Changeset 96 for XMLIO_V2/dev/dev_rv/xmlio_xml_node.hpp

Timestamp:

06/04/10 14:55:53 (14 years ago)

Author:

hozdoba

Message:

Suppression de nombreuses fuites mémoires (200kbytes)

File:

• XMLIO_V2/dev/dev_rv/xmlio_xml_node.hpp (modified) (7 diffs)

XMLIO_V2/dev/dev_rv/xmlio_xml_node.hpp

@@ -9 +9 @@
 #include <Poco/DOM/NamedNodeMap.h>
 
-//#include <Poco/DOM/AutoPtr.h>
+#include <Poco/DOM/AutoPtr.h>
 
 // Entêtes Poco SAX
@@ -38 +38 @@
 using Poco::HashMap;
 
+using Poco::XML::AutoPtr;
+
 namespace XMLIOSERVER
 {
@@ -46 +48 @@
          
       // TODO Mettre des auto_ptr ici car gestion de la mémoire lamentable sans.
-      typedef Document* PDocument;
-      typedef Node*     PNode;
+      typedef AutoPtr<Document> PDocument;
+      typedef Node*    PNode;
          
       class XMLNode
       {
-         protected :
-            
-            XMLNode(const string& _rootName) : rootName(_rootName) { this->cNode = NULL ; }
-            
          public :
             
-            static XMLNode CreateNode(istream& _istr, const string& _rootName = string("simulation"))
+            XMLNode(const string& _rootName) : rootName(_rootName) 
+            { /* Ne rien faire de plus */}          
+
+            static XMLNode& CreateNode(XMLNode& node, istream& _istr, const string& _rootName = string("simulation"))
             {
-               XMLNode node(_rootName);
-                  
                if ((_istr.good()))
                { // S'il est possible de lire le flux en entrée ...
@@ -68 +67 @@
                   // On parse la source XML et on vérifie que le premier noeud (racine) est du type "Element"
                   // ... et à pour valeur la chaîne rootName.
-                  PDocument pDoc = parser.parse(&src);
-                  if (!(pDoc->documentElement()->nodeName().compare(_rootName)))
+                  node.pDoc = parser.parse(&src);
+                  if (!(node.pDoc->documentElement()->nodeName().compare(_rootName)))
                   {
-                     node.cNode = pDoc->documentElement();
+                     node.setCNode(node.pDoc->documentElement());
                      node.goToChildElement();
                   }
@@ -77 +76 @@
                   {
                      ostringstream oss;
-                     oss << "L'élément racine doit avoir pour valeur <" << _rootName << "> (\"" <<  (pDoc->documentElement()->nodeName()) <<"\" lue)";
+                     oss << "L'élément racine doit avoir pour valeur <" << _rootName << "> (\"" <<  (node.pDoc->documentElement()->nodeName()) <<"\" lue)";
                      throw XMLParsingException(oss.str());
                   }    
@@ -87 +86 @@
             }
                
-            string getElementName(void) const {return (this->cNode->nodeName());}
+            string getElementName(void) const {return (this->getCNode()->nodeName());}
                 
-            bool goToNextElement(Node* nextElement = 0) 
+            bool goToNextElement(void) 
             {
-               nextElement = (!nextElement)? this->cNode->nextSibling() : nextElement;
+               PNode nextElement = this->getCNode()->nextSibling();
               
                // On parcourt la liste des "siblings" jusqu'à trouver un élément quelconque.
                for(; ; nextElement = nextElement->nextSibling())
-               if (nextElement == NULL) break;
-               else if (nextElement->nodeType() == 1)
-               {// Si l'un des noeuds est un élément...
-                this->cNode = nextElement ;
-                return (true);
-               } 
-              return (false);
-             }
+                  if (IsPtrNull(nextElement)) break;
+                  else if (nextElement->nodeType() == 1)
+                  {// Si l'un des noeuds est un élément...
+                     this->setCNode(nextElement) ;
+                     return (true);
+                  } 
+               return (false);
+            }
              
-             bool goToChildElement(void)
-             {
-              // On parcourt la liste des enfants jusqu'à trouver un élément quelconque.
-              if (this->cNode->firstChild())
-               if (this->goToNextElement(this->cNode->firstChild()))
-                return (true);
+            bool goToChildElement(void)
+            {
+               PNode nextElement = this->getCNode()->firstChild();
+               
+               // On parcourt la liste des enfants jusqu'à trouver un élément quelconque.
+               if (!IsPtrNull(nextElement))
+               {
+                  for(; ; nextElement = nextElement->nextSibling())
+                     if (IsPtrNull(nextElement)) break;
+                     else if (nextElement->nodeType() == 1)
+                     {// Si l'un des noeuds est un élément...
+                        this->setCNode(nextElement) ;
+                        return (true);
+                     }
+                  return (false);
+               }
                 
-              return (false);
-             }
+               return (false);
+            }
              
-             bool goToParentElement(void)
-             { 
-              // Pas de retour au parent si on est à la racine.
-              if (!(this->getElementName().compare(rootName))) return (false);
-              this->cNode = this->cNode->parentNode();
-              return (true);
-             }
+            bool goToParentElement(void)
+            { 
+               // Pas de retour au parent si on est à la racine.
+               if (!(this->getElementName().compare(rootName))) return (false);
+               this->setCNode(this->getCNode()->parentNode());
+               return (true);
+            }
              
-             bool getAttributes(THashAttributes& attributes) const
-             {
-              if(!this->cNode->hasAttributes()) return (false);
-              NamedNodeMap* map = this->cNode->attributes();
+            bool getAttributes(THashAttributes& attributes) const
+            {
+                
+               if(!this->getCNode()->hasAttributes()) return (false);
+               AutoPtr<NamedNodeMap> map = this->getCNode()->attributes();
               
               for(unsigned int i = 0; i< map->length(); i++)
@@ -131 +141 @@
 
               return (true);
-             }
+            }
             
             ~XMLNode() 
             { /* Ne rien faire de plus */ }
-
+            
+         protected :
+         
+            PNode getCNode(void) const {return (this->cNode); }
+            void setCNode(PNode other) { this->cNode = other; }
+            
+            static bool IsPtrNull(PNode ptr) {return (ptr==NULL);}
+            
          private :
-                  
+            PDocument pDoc; 
             PNode  cNode;
+            
             string rootName;