Changeset 690

Timestamp:

Dec 26, 2007, 11:28:31 PM (16 years ago)

Author:

landauf

Message:

some changes in the ClassIdentifier (inspired by retos suggestion)

Location:

code/branches/FICN/src/orxonox/core

Files:

• ClassFactory.h (modified) (1 diff)
• Identifier.cc (modified) (1 diff)
• Identifier.h (modified) (5 diffs)

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

@@ -73 +73 @@
     {
         COUT(4) << "*** Create entry for " << name << " in Factory." << std::endl;
+        ClassIdentifier<T>::createIdentifier(name);
         ClassIdentifier<T>::getIdentifier()->addFactory(new ClassFactory<T>);
         Factory::add(name, ClassIdentifier<T>::getIdentifier());

code/branches/FICN/src/orxonox/core/Identifier.cc

@@ -40 +40 @@
     int Identifier::hierarchyCreatingCounter_s = 0; // Set the static member variable hierarchyCreatingCounter_s to zero
     unsigned int Identifier::classIDcounter_s = 0;  // Set the static member variable classIDcounter_s to zero
+    std::map<std::string, Identifier*> Identifier::identifierMap_s;
 
     /**

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

@@ -172 +172 @@
             unsigned int classID_;                                      //!< The network ID to identify a class through the network
             std::map<std::string, ConfigValueContainer*> configValues_; //!< A map to link the string of configurable variables with their ConfigValueContainer
+            static std::map<std::string, Identifier*> identifierMap_s;  //!< A map, containing all existing ClassIdentifiers
     };
 
@@ -189 +190 @@
         public:
             static ClassIdentifier<T>* registerClass(const IdentifierList* parents, const std::string& name, bool bRootClass);
-            static ClassIdentifier<T>* getIdentifier();
+            static void createIdentifier(const std::string& name);
             static void addObject(T* object);
 
+            /** @returns the Identifier itself (createIdentifier() has to be called first). */
+            inline static ClassIdentifier<T>* getIdentifier() { return pointer_s; }
+
         private:
-            ClassIdentifier();
+            ClassIdentifier(const std::string& name);
             ClassIdentifier(const ClassIdentifier<T>& identifier) {} // don't copy
             ~ClassIdentifier();
@@ -206 +210 @@
     /**
         @brief Constructor: Creates the ObjectList.
-    */
-    template <class T>
-    ClassIdentifier<T>::ClassIdentifier()
+        @param name The name of the Class
+    */
+    template <class T>
+    ClassIdentifier<T>::ClassIdentifier(const std::string& name)
     {
         this->objects_ = new ObjectList<T>;
+        this->name_ = name;
+        this->identifierMap_s[name] = this;
     }
 
@@ -236 +243 @@
 
         // It's a singleton, so maybe we have to create it first
-        if (!pointer_s)
-        {
-            COUT(4) << "*** Register Class in " << name << "-Singleton -> Create Singleton." << std::endl;
-            pointer_s = new ClassIdentifier();
-        }
+        ClassIdentifier<T>::createIdentifier(name);
 
         // Check if at least one object of the given type was created
@@ -258 +261 @@
 
     /**
-        @returns the Identifier itself.
-    */
-    template <class T>
-    ClassIdentifier<T>* ClassIdentifier<T>::getIdentifier()
-    {
-        if (!pointer_s)
+        @brief Creates the only instance of this class for the template class T.
+        @param name The name of the Class
+    */
+    template <class T>
+    void ClassIdentifier<T>::createIdentifier(const std::string& name)
+    {
+        static bool bFirstCeateIdentifierFunctionCall = true;
+        static ClassIdentifier<T> classIdentifierObject = ClassIdentifier<T>(name);
+
+        if (bFirstCeateIdentifierFunctionCall)
         {
-            COUT(4) << "*** Create Singleton." << std::endl;
-            pointer_s = new ClassIdentifier();
+            COUT(4) << "*** Create Identifier Singleton for " << name << "." << std::endl;
+            pointer_s = &classIdentifierObject;
+            bFirstCeateIdentifierFunctionCall = false;
         }
-
-        return pointer_s;
     }