Changeset 457

Timestamp:

Dec 10, 2007, 8:34:48 PM (16 years ago)

Author:

landauf

Message:

changed the class-hierarchy creation: call Factory::createClassHierarchy() to create the hierarchy.
until now it was automatically created at the program-start, but that could have been a problem in the future when classes depend on ogre, because ogre isn't already initialized at that program-start, so i've changed it.

Location:

code/branches/objecthierarchy/src/orxonox

Files:

• core/ClassFactory.h (modified) (3 diffs)
• core/ConfigValueContainer.cc (modified) (1 diff)
• core/CoreIncludes.h (modified) (1 diff)
• core/Factory.cc (modified) (2 diffs)
• core/Factory.h (modified) (1 diff)
• core/Identifier.h (modified) (2 diffs)
• orxonox.cc (modified) (1 diff)

code/branches/objecthierarchy/src/orxonox/core/ClassFactory.h

@@ -10 +10 @@
 
 #include "Identifier.h"
-#include "Debug.h"
 
 namespace orxonox
@@ -22 +21 @@
     {
         public:
-            static bool create();
+            static bool create(const std::string& name);
             BaseObject* fabricate();
 
@@ -34 +33 @@
 
     /**
-        @brief Adds the ClassFactory to the Identifier of the same type and creates a new object to retrieve the parents.
+        @brief Adds the ClassFactory to the Identifier of the same type and the Identifier to the Factory.
         @return Always true (this is needed because the compiler only allows assignments before main())
     */
     template <class T>
-    bool ClassFactory<T>::create()
+    bool ClassFactory<T>::create(const std::string& name)
     {
-        // Add the ClassFactory to the Classidentifier of type T
         ClassIdentifier<T>::getIdentifier()->addFactory(new ClassFactory<T>);
-
-        // To create the new branch of the class-hierarchy, we create a new object and delete it afterwards.
-        ClassIdentifier<T>::getIdentifier()->startCreatingHierarchy();
-        COUT(4) << "*** Create Factory -> Create Class\n";
-        BaseObject* temp = ClassIdentifier<T>::getIdentifier()->fabricate();
-        delete temp;
-        ClassIdentifier<T>::getIdentifier()->stopCreatingHierarchy();
+        Factory::add(name, ClassIdentifier<T>::getIdentifier());
 
         return true;

code/branches/objecthierarchy/src/orxonox/core/ConfigValueContainer.cc

@@ -421 +421 @@
         if (!ConfigValueContainer::readConfigFile_s)
             ConfigValueContainer::readConfigFile(CONFIGFILEPATH);
-
 
         // The string of the section we're searching

code/branches/objecthierarchy/src/orxonox/core/CoreIncludes.h

@@ -81 +81 @@
 */
 #define CreateFactory(ClassName) \
-    bool bCreated##ClassName##Factory = ClassFactory<ClassName>::create()
+    bool bCreated##ClassName##Factory = ClassFactory<ClassName>::create(#ClassName)
 
 /**

code/branches/objecthierarchy/src/orxonox/core/Factory.cc

@@ -6 +6 @@
 #include "Factory.h"
 #include "Identifier.h"
+#include "Debug.h"
+#include "../objects/BaseObject.h"
 
 namespace orxonox
@@ -60 +62 @@
         pointer_s->identifierNetworkIDMap_[newID] = identifier;
     }
+
+    /**
+        @brief Creates the class-hierarchy by creating and destroying one object of each type.
+    */
+    void Factory::createClassHierarchy()
+    {
+        if (!pointer_s)
+            pointer_s = new Factory;
+
+        COUT(4) << "*** Factory -> Create class-hierarchy\n";
+        std::map<std::string, Identifier*>::iterator it;
+        it = pointer_s->identifierStringMap_.begin();
+        (*pointer_s->identifierStringMap_.begin()).second->startCreatingHierarchy();
+        for (it = pointer_s->identifierStringMap_.begin(); it != pointer_s->identifierStringMap_.end(); ++it)
+        {
+            // To create the new branch of the class-hierarchy, we create a new object and delete it afterwards.
+            BaseObject* temp = (*it).second->fabricate();
+            delete temp;
+        }
+        (*pointer_s->identifierStringMap_.begin()).second->stopCreatingHierarchy();
+        COUT(4) << "*** Factory -> Finished class-hierarchy creation\n";
+    }
 }

code/branches/objecthierarchy/src/orxonox/core/Factory.h

@@ -36 +36 @@
             static void add(const std::string& name, Identifier* identifier);
             static void changeNetworkID(Identifier* identifier, const unsigned int oldID, const unsigned int newID);
+            static void createClassHierarchy();
 
         private:

code/branches/objecthierarchy/src/orxonox/core/Identifier.h

@@ -64 +64 @@
         friend class SubclassIdentifier; // Forward declaration
 
-        template <class T>
-        friend class ClassFactory; // Forward declaration
+        friend class Factory; // Forward declaration
 
         public:
@@ -216 +215 @@
             COUT(4) << "*** Register Class in " << name << "-Singleton -> Initialize Singleton.\n";
             pointer_s->name_ = name;
-            Factory::add(name, pointer_s); // Add the Identifier to the Factory
+//            Factory::add(name, pointer_s); // Add the Identifier to the Factory
 
             if (bRootClass)

code/branches/objecthierarchy/src/orxonox/orxonox.cc

@@ -119 +119 @@
         setupCEGUI();
         createFrameListener();
-
+        Factory::createClassHierarchy();
 
         #define testandcout(code) \