Question

我根据以下文章here实现了一个工厂类。

然而，我有一个问题，我认为这与编译器优化有关。

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

在两个声明为static的源文件中，我都有一个注册器类。

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。注册器构造函数永远不会执行，因此，Factory类永远不会知道BuildingLot类。如果我这样做：

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

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

我认为，由于第一个例子中没有使用BuildingLot，编译器已经优化，甚至没有编译BuildingLot.cpp。

这可能吗？我该如何解决这个问题？

编辑：事实上，注册商类是模板类。我只是忘了插入模板参数。

这里我的代码尽可能简单。我希望我没有留下任何东西。如果我取消注释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;

}

Answer 1

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

工厂类实现问题

1 个答案: