1

在网上搜索了工厂模式的不同方法后，我实现了自己的版本，我对此非常满意。Register成员函数为模板类型创建一个函数指针，使用索引作为键将其存储在std::map中。下面的代码可以轻松编译和运行（Windows 7 64 位，带 GCC 的 Code::Blocks 10.05）。

#ifndef OBJECT_FACTORY_HPP
#define OBJECT_FACTORY_HPP

#include <map>

namespace internal {
    template<typename BaseType, typename ObjectType>
    BaseType* CreateFunction() {
        return new ObjectType;
    }
}

template<typename BaseType, typename IndexType>
class ObjectFactory {
    public:
        ObjectFactory();

        template<typename ObjectType>
        bool Register(const IndexType& index);

        bool Unregister(const IndexType& index);

        BaseType* Create(const IndexType& index);

    private:
        typedef BaseType* (*creationCallback)();
        typedef std::map<IndexType, creationCallback> Registry;
        Registry registry;

//    private:
//        template<typename ObjectType>
//        BaseType* CreateFunction();
};

template<typename BaseType, typename IndexType>
ObjectFactory<BaseType, IndexType>::ObjectFactory() {
    registry.clear();
}

template<typename BaseType, typename IndexType>
template<typename ObjectType>
bool ObjectFactory<BaseType, IndexType>::Register(const IndexType& index) {
    if (registry.find(index) != registry.end())
        return false;

    registry[index] = &internal::CreateFunction<BaseType, ObjectType>;
    // registry[index] = &CreateFunction<ObjectType>; <-- FAILS!
    return true;
}

template<typename BaseType, typename IndexType>
bool ObjectFactory<BaseType, IndexType>::Unregister(const IndexType& type) {
    if (registry.find(type) == registry.end())
        return false;

    return (registry.erase(type) == 1);
}

template<typename BaseType, typename IndexType>
BaseType* ObjectFactory<BaseType, IndexType>::Create(const IndexType& index) {
    if (registry.find(index) == registry.end())
        return NULL;

    return registry[index]();
}

//template<typename BaseType, typename IndexType>
//template<typename ObjectType>
//BaseType* ObjectFactory<BaseType, IndexType>::CreateFunction() {
//    return new ObjectType();
//}

#endif

我最初的方法是让CreateFunction作为私有成员以对用户隐藏它（查看注释部分，注意非成员函数需要一个额外的模板参数）。但是，这会失败并显示以下错误消息，所有错误消息都指向我在Register成员函数中存储函数指针的行：

In member function 'bool ObjectFactory<BaseType, IndexType>::Register(const IndexType&) [with ObjectType = Triangle, BaseType = Shape, IndexType = std::basic_string<char, std::char_traits<char>, std::allocator<char> >]':|
instantiated from here
error: no matches converting function 'CreateFunction' to type 'class Shape* (*)()'|
error: candidates are: template<class ObjectType> BaseType* ObjectFactory::CreateFunction() [with ObjectType = ObjectType, BaseType = Shape, IndexType = std::basic_string<char, std::char_traits<char>, std::allocator<char> >]|

我写了一个小测试客户端进行测试：

#include "ObjectFactory.hpp"
#include <iostream>

class Shape {
    public:
        Shape() {}
        virtual void print() { std::cout << "At heart, I'm a shape"; }
};

class Triangle : public Shape {
    public:
        Triangle() {}
        void print() { Shape::print(); std::cout << ", but I'm truly a triangle" << std::endl; }
};

int main(int argc, char* argv[]) {
    ObjectFactory<Shape, std::string> objectFactory;

    objectFactory.Register<Triangle>("triangle");

    Shape* triangle = objectFactory.Create("triangle");
    triangle->print();

    delete triangle;

    return 0;
}

这甚至可能吗？我觉得应该是，而且我知道我以某种方式调用了不正确的成员函数，但我不明白为什么。在相关说明中，由于有人可能会提到它，我计划使用 *boost::shared_ptr* 进行分配，而不是普通的 new 运算符；）也欢迎任何其他关于实施的建议或建议。

4

2 回答 2

1

在我们的文件中，我们匿名化我们的命名空间：

// namespace internal {  
   namespace          {
    template<typename BaseType, typename ObjectType> 
    BaseType* CreateFunction() { ... }  
}

现在可以按最初编写的方式调用非静态函数，而无需命名空间限定： 

// registry[index] = &internal::CreateFunction<ObjectType>;  
   registry[index] =           &CreateFunction<ObjectType>;

文件范围的CreateFunction函数对翻译单元之外的代码是不可见的，并且仅由 class 调用ObjectFactory。

这很好地近似了问题中提出的（来自）的private访问说明符。CreateFunctionObjectFactory

于 2012-05-09T01:41:54.750 回答
0

创建函数应该是静态成员：

template<typename ObjectType> static BaseType* CreateFunction();

（上面的代码，没有static，在你的例子中被注释掉了）。

现在您的其他注释掉的代码是有效的：

registry[index] = &CreateFunction<ObjectType>;

编辑：清除一些混乱：

这里的问题主要在于语法。模板private在这里并不重要，所以让我们简化一下情况：

struct ObjectFactory {
    void CreateFunction() {std::cout << "message";}
    void AnotherFunction()
    {
        ... = &CreateFunction; // what's that?
    }
};

ifCreateFunction是非静态成员函数，语法&CreateFunction不正确；我想 gcc 就是在抱怨这一点。然而，更糟糕的是，MS Visual Studio 试图通过在内部“更正”语法&ObjectFactory::CreateFunction并尝试使用它来“帮助”您；但它会因无法理解的错误消息而失败（您拥有模板的事实会使它们更加严重）。

我不确定 Visual Studio 部分（我记得几年前 MSVC 6 遇到过这样的问题；较新版本的 Visual Studio 没有这个问题）。

于 2012-05-08T19:02:14.970 回答