Changeset 96

Timestamp:

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

Author:

hozdoba

Message:

Suppression de nombreuses fuites mémoires (200kbytes)

Location:

XMLIO_V2/dev/dev_rv

Files:

• context.hpp (modified) (2 diffs)
• field.hpp (modified) (1 diff)
• field_group.hpp (modified) (1 diff)
• xmlio_group_template.hpp (modified) (1 diff)
• xmlio_object_template.hpp (modified) (1 diff)
• xmlio_xml_node.hpp (modified) (7 diffs)
• xmlio_xml_parser.hpp (modified) (2 diffs)

XMLIO_V2/dev/dev_rv/context.hpp

@@ -17 +17 @@
          {
             // On modifie le context courrant pour tout les ObjectTemplate
+            Context::SetContext(id);
             FieldGroup::SetContext(id);
             Field::SetContext(id);
@@ -84 +85 @@
                   
          ~Context()
-         {/* Ne rien faire de plus pour le moment */ }
+         { delete fieldDef; }
          
       protected:

XMLIO_V2/dev/dev_rv/field.hpp

@@ -43 +43 @@
          
          ~Field(void) 
-         {/* Ne rien faire de plus */}         
+         { /* Ne rien faire de plus */ }    
       
    }; // class Field 

XMLIO_V2/dev/dev_rv/field_group.hpp

@@ -101 +101 @@
          }
          
-         ~FieldGroup(void) 
+         virtual ~FieldGroup(void) 
          {/* Ne rien faire de plus */}   
          

XMLIO_V2/dev/dev_rv/xmlio_group_template.hpp

@@ -65 +65 @@
             
         
-         ~GroupTemplate()
-         {/* Ne rien faire de plus pour le moment */ }
+         virtual ~GroupTemplate()
+         {
+            for (unsigned int i = 0; i < childList.getVector().size(); i++)
+               delete childList.getVector()[i];
+            for (unsigned int i = 0; i < groupList.getVector().size(); i++)
+               delete groupList.getVector()[i];
+         }
          
       protected:

XMLIO_V2/dev/dev_rv/xmlio_object_template.hpp

@@ -41 +41 @@
          { return (ObjectTemplate<T>::AllListObj[CurrContext].hasMappedValue(_id)); }
          
-         static const StrHashMap<T> GetCurrentListObject(void) { return (AllListObj[CurrContext]); }
+         static const StrHashMap<T>& GetCurrentListObject(void) { return (AllListObj[CurrContext]); }
+         static HashMap<string, StrHashMap<T> >& GetAllListObject(void) { return (AllListObj); }
                 
-         static void SetContext(const string& id)
-         { ObjectTemplate<T>::CurrContext = id; } 
+         static void SetContext(const string& id){ ObjectTemplate<T>::CurrContext = id; } 
          
          static string& GetCurrentContextId(void) { return (CurrContext); } 

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;      
          

XMLIO_V2/dev/dev_rv/xmlio_xml_parser.hpp

@@ -18 +18 @@
             {
                THashAttributes attributes;
-               
+                              
                do {                
                  // Traitement de l'identifiant
@@ -36 +36 @@
                   
                } while (_node.goToNextElement());               
-            }    
+            } 
+            
+            static void FreeMemory(void)
+            {
+               HashMap<string, StrHashMap<Context> > &AllListContext = Context::GetAllListObject();
+               
+               for (HashMap<string, StrHashMap<Context> >::Iterator it = AllListContext.begin(); it != AllListContext.end(); it++)
+               {
+                  string& id = (*it).first;
+                  StrHashMap<Context>& sc = (*it).second;
+                  // BUG AllListContext contient une StrHashMap<Context> liée à l'identifiant "" d'où la vérification ci-dessous.
+                  if (id.size()>0) delete sc[id];
+               }
+            }
             
             static bool CLASS_TEST(const string& file = string("/local/testsuite/iodef_test.xml"), ostream& log = std::clog)
             {
                ifstream istr( file.c_str() , ifstream::in );
-               XMLNode node = XMLNode::CreateNode(istr);
+               XMLNode node("simulation");
+               XMLNode::CreateNode(node, istr);
                XMLParser::Parse(node);
-               log << "Nombre de Contexts listés : " << Context::GetCurrentListObject().getSize() << " contre 2 attendus."<< std::endl;
+               log << "Nombre de Contexts listés : " << Context::GetCurrentListObject().getSize() << " contre 1 attendus."<< std::endl;
                log << "Nombre de FieldGroups listés : " << FieldGroup::GetCurrentListObject().getSize() << " contre 5 attendus."<< std::endl;
                log << "Description du champs votkeavt : " << Field::GetObject("votkeavt").description << " contre \"Vertical Eddy Diffusivity\" attendus."<< std::endl;
                
                log << "Test  XMLParser ... ok !" << std::endl;
+               
+               FreeMemory(); // Pour supprimer la mémoire liée à l'allocation dynamique.
+               
                return (true);
             }