c++ - 工厂级实现问题

我根据以下可用的here实现了工厂类。

但是，我有一个问题，我认为这与编译器的优化有关。

我有一个类的层次结构，其中Node类(在Node.h / Node.cpp中)是基类，例如BuildingLot(在BuildingLot.h / BuildingLot.cpp中)是子类。

在两个声明为静态的源文件中，我都有一个Registrar类。

Node.cpp:

static Registrar<Node> _registrar(“Node”);

BuildingLot.cpp:

static Registrar<BuildingLot> _registrar(“BuildingLot”);

如果我尝试使用Factory，那么它可以完美地用于Node类。但是，我必须首先创建BuildingLot的实例才能在工厂中注册。

在单元测试中，我有:

NodePtr node = Factory::Instance()->Create(“Node”);
NodePtr bl = Factory::Instance()->Create(“BuildingLot”);

bl始终为nullptr。 Registrar构造函数从不执行，因此Factory类从不知道BuildingLot类。如果我这样做:

BuildingLot b;
NodePtr node = Factory::Instance()->Create(“Node”);
NodePtr bl = Factory::Instance()->Create(“BuildingLot”);

然后一切正常。调用注册服务商，然后工厂创建一个BuildingLot。

我认为，由于第一个示例未使用BuildingLot，因此编译器进行了优化，甚至没有编译BuildingLot.cpp。

这可能吗？我该如何解决？

编辑:
Registrar类实际上是模板类。我只是忘了插入模板参数。

这是我能做到的简单代码。我希望我没有遗漏任何东西。如果我取消注释main()的第一行，那么一切都可以。

NodeFactory.h:

class NodeFactory{
private:
    /// Map of factory functions
    std::map<std::string, std::function<std::shared_ptr<Node>(void)>> mFactoryFunctions;
public:
     /// Get Singleton
    static NodeFactory* Instance();

    /// Register Function.
    void Register(const std::string &name, std::function<std::shared_ptr<Node>(void)> factoryFunction);

    /// Factory Function.
    std::shared_ptr<Node> Create(const std::string &name);
};

NodeFactory.cpp:

/**
 * Get Singleton
 */
NodeFactory* NodeFactory::Instance(){
    static NodeFactory factory;
    return &factory;
}

void NodeFactory::Register(const std::string &name, std::function<std::shared_ptr<Node>(void)> factoryFunction){
    mFactoryFunctions[name] = factoryFunction;
}

std::shared_ptr<Node> NodeFactory::Create(const std::string &name){
    if(mFactoryFunctions.find(name) == mFactoryFunctions.end())
        return nullptr;

    std::shared_ptr<Node> n = mFactoryFunctions[name]();

    return n;
}

Registrar.h:

#define REGISTER_NODE_TYPE(NODE_TYPE) static NodeRegistrar<NODE_TYPE> _registrar(#NODE_TYPE);

template<class T>
class NodeRegistrar{
private:

public:

    NodeRegistrar(const std::string &name){
        NodeFactory::Instance()->Register(name,
                                          [](void) -> std::shared_ptr<T> { return std::make_shared<T>(); }
                                          );
    }
};

Node.h:

class Node{
private:
    /// The node ID.
    NodeID mID;

public:
    /* ****************************
     * Construction & Destruction *
     * ***************************/
    Node();
    Node(const NodeID &nodeID);
    virtual ~Node();
};

Node.cpp:

REGISTER_NODE_TYPE(Node);

Node::Node(){
    mID = -1;
}
Node::Node(const NodeID &nodeID){
    mID = nodeID;
}

Node::~Node(){

}

BuildingLot.h:

class BuildingLot : public Node{
public:

    BuildingLot();
    BuildingLot(const NodeID &nodeID);
    virtual ~BuildingLot();

};

BuildingLot.cpp:

REGISTER_NODE_TYPE(BuildingLot);

BuildingLot::BuildingLot(){

}

BuildingLot::BuildingLot(const NodeID &nodeID):Node(nodeID){

}

BuildingLot::~BuildingLot(){

}

main.cpp:

int main(int argc, const char * argv[]){

// BuildingLot bl; // if I uncomment this, then it works
std::shared_ptr<Node> node = NodeFactory::Instance()->Create("Node");
std::shared_ptr<Node> buildingLot = NodeFactory::Instance()->Create("BuildingLot");

if(node == nullptr)
    std::cout << "node is nullptr" << std::endl;
if(buildingLot == nullptr)
    std::cout << "buildingLot is nullptr" << std::endl;

return 0;

}

最佳答案

最后，我选择了在工厂类(class)手动注册lambda创建者。
如果创建和使用宏，并且代码行数完全相同，则非常简单。

关于c++ - 工厂级实现问题，我们在Stack Overflow上找到一个类似的问题：https://stackoverflow.com/questions/25566967/