Changeset 9635 for code/branches/core6

Timestamp:

Aug 11, 2013, 2:38:03 PM (11 years ago)

Author:

landauf

Message:

added Factory for classes that don't require a Context as constructor argument

Location:

code/branches/core6

Files:

• src/libraries/core/CoreIncludes.h (modified) (1 diff)
• src/libraries/core/CorePrereqs.h (modified) (1 diff)
• src/libraries/core/object/ClassFactory.h (modified) (3 diffs)
• test/core/object/ClassFactoryTest.cc (modified) (1 diff)

code/branches/core6/src/libraries/core/CoreIncludes.h

@@ -113 +113 @@
 
 /**
-    @brief Creates the Factory.
+    @brief Creates and registers the Factory.
     @param ClassName The name of the class
 */
 #define CreateFactory(ClassName) \
-    Factory* _##ClassName##Factory = new orxonox::ClassFactory<ClassName>(#ClassName, true)
+    RegisterFactory(ClassName, new orxonox::ClassFactoryWithContext<ClassName>(#ClassName, true))
 
 /**
-    @brief Creates the Factory for classes which should not be loaded through XML.
+    @brief Creates and registers the Factory for classes which should not be loaded through XML.
     @param ClassName The name of the class
 */
 #define CreateUnloadableFactory(ClassName) \
-    Factory* _##ClassName##Factory = new orxonox::ClassFactory<ClassName>(#ClassName, false)
+    RegisterFactory(ClassName, new orxonox::ClassFactoryWithContext<ClassName>(#ClassName, false))
+
+/**
+    @brief Registers a given Factory.
+    @param ClassName The name of the class
+*/
+#define RegisterFactory(ClassName, FactoryInstance) \
+    Factory* _##ClassName##Factory = FactoryInstance
 
 /**

code/branches/core6/src/libraries/core/CorePrereqs.h

@@ -126 +126 @@
     template <class T>
     class ClassFactory;
+    template <class T>
+    class ClassFactoryWithContext;
     template <class T>
     class ClassIdentifier;

code/branches/core6/src/libraries/core/object/ClassFactory.h

@@ -50 +50 @@
     // ###       Factory       ###
     // ###########################
-    /// Base-class of ClassFactory.
+    /// Base-class of all factories.
     class _CoreExport Factory
     {
@@ -61 +61 @@
     // ###      ClassFactory       ###
     // ###############################
-    /// The ClassFactory is able to create new objects of a specific class.
+    /// The ClassFactory is able to create new objects of a specific class that require no constructor arguments.
     template <class T>
     class ClassFactory : public Factory
@@ -84 +84 @@
             inline Identifiable* fabricate(Context* context)
             {
+                return static_cast<Identifiable*>(new T());
+            }
+    };
+
+    // ###############################
+    // ### ClassFactoryWithContext ###
+    // ###############################
+    /// The ClassFactoryWithContext is able to create new objects of a specific class that require a context as constructor argument
+    template <class T>
+    class ClassFactoryWithContext : public Factory
+    {
+        public:
+            /**
+                @brief Constructor: Adds the ClassFactory to the Identifier of the same type.
+                @param name The name of the class
+                @param bLoadable True if the class can be loaded through XML
+            */
+            ClassFactoryWithContext(const std::string& name, bool bLoadable = true)
+            {
+                orxout(verbose, context::misc::factory) << "Create entry for " << name << " in Factory." << endl;
+                ClassIdentifier<T>::getIdentifier(name)->setFactory(this);
+                ClassIdentifier<T>::getIdentifier()->setLoadable(bLoadable);
+            }
+
+            /**
+                @brief Creates and returns a new object of class T.
+                @return The new object
+            */
+            inline Identifiable* fabricate(Context* context)
+            {
                 return static_cast<Identifiable*>(new T(context));
             }

code/branches/core6/test/core/object/ClassFactoryTest.cc

@@ -7 +7 @@
     TEST(ClassFactoryTest, CanFabricateObject)
     {
-        Factory* factory = new ClassFactory<BaseObject>("BaseObject");
+        Factory* factory = new ClassFactoryWithContext<BaseObject>("BaseObject");
         Identifiable* object = factory->fabricate(NULL);
         ASSERT_TRUE(object != NULL);