Question

我有这个问题，如果我包含某个标题，我无法编译。

标题的代码：

//#include "tinyxml2.h"

#pragma once

#include "Camera.h"
#include "Controller.h"
#include "Lights.h"
#include "Mesh.h"

namespace ActorFactory {

    //using namespace tinyxml2;

    template<class T>  
    Actor* createInstance() { 
        return new T; 
    }

    typedef std::map<std::string, Actor*(*)()> class_map;
    class_map test = {
        { "Camera", &createInstance<Camera> }
        //{ "Mesh", &createInstance<Mesh> }
    };
}

来自visual studio的完整错误消息：

1>------ Build started: Project: ogl_inf251_ca2, Configuration: Debug Win32     ------
1>  Main.cpp
1>d:\development\inf251\ogl_inf251_ca2\model_obj.h(26): warning C4005:     '_CRT_SECURE_NO_WARNINGS' : macro redefinition
1>          command-line arguments :  see previous definition of     '_CRT_SECURE_NO_WARNINGS'
1>  Generating Code...
1>  Compiling...
1>  Scene.cpp
1>d:\development\inf251\ogl_inf251_ca2\model_obj.h(26): warning C4005:     '_CRT_SECURE_NO_WARNINGS' : macro redefinition
1>          command-line arguments :  see previous definition of     '_CRT_SECURE_NO_WARNINGS'
1>  Generating Code...
1>Scene.obj : error LNK2005: "class std::map<class     std::basic_string<char,struct std::char_traits<char>,class std::allocator<char>     >,class Actor * (__cdecl*)(void),struct std::less<class     std::basic_string<char,struct std::char_traits<char>,class std::allocator<char>     > >,class std::allocator<struct std::pair<class std::basic_string<char,struct     std::char_traits<char>,class std::allocator<char> > const ,class Actor *     (__cdecl*)(void)> > > ActorFactory::test" (?test@ActorFactory@@3V?$map@V?    $basic_string@DU?$char_traits@D@std@@V?$allocator@D@2@@std@@P6APAVActor@@XZU?    $less@V?$basic_string@DU?$char_traits@D@std@@V?$allocator@D@2@@std@@@2@V?    $allocator@U?$pair@$$CBV?$basic_string@DU?$char_traits@D@std@@V?    $allocator@D@2@@std@@P6APAVActor@@XZ@std@@@2@@std@@A) already defined in     Main.obj
1>D:\development\inf251\Debug\ogl_inf251_ca2.exe : fatal error LNK1169: one     or more multiply defined symbols found
========== Build: 0 succeeded, 1 failed, 0 up-to-date, 0 skipped ==========

遗憾的是，错误消息非常长。我不知道如何解决这个问题甚至错误。还有一种更好的方法来按名称实例化类吗？

感谢您提前提供任何帮助：）

Answer 1

问题在于您宣布test的方式。

您可以声明static或const（在这种情况下暗示static）。但是，对于包含此标头的每个源文件，您将拥有单独的副本test，浪费内存并可能导致其他问题。

因此，您应该在标题extern class_map test;中仅在一个源文件中声明您的测试，放置ActorFactory::class_map ActorFactory::test = {...};（如果您将其设为const，也将extern放在源文件位中）。

Answer 2

您的test对象违反了单一定义规则。

#pragma once使定义不会在 相同的 编译单元中多次出现（读取：相同的* .cpp文件），但不会出现在 不同的 编译单元中（读取：不同的* .cpp文件）。

解决此问题的一种方法是将对象转换为函数中的静态本地对象，并通过引用使其可访问：

<强> header.h：

// ...

namespace ActorFactory {

    // ...

    typedef std::map<std::string, Actor*(*)()> class_map;
    class_map& test();
}

<强> TEST.CPP：

#include "header.h"

namespace ActorFactory {

    // ...

    class_map& test() {
        static class_map test = {
            { "Camera", &createInstance<Camera> }
            { "Mesh", &createInstance<Mesh> };
        return test;
    }
}

这也是针对所谓的static initialization order fiasco的自动防范保证，并且为您提供了懒惰的初始化（并不是说这个小对象很重要）。

通过使用new分配对象并永不删除它，您可以使整个事情更安全（面对破坏顺序问题），但该函数仍应返回引用< /强>：

namespace ActorFactory { // ... class_map& test() { static class_map* test = new class_map { { "Camera", &createInstance<Camera> } { "Mesh", &createInstance<Mesh> }; return *test; } }

阅读我上面链接的常见问题解答以获取所有详细信息。

在任何情况下，客户端代码只需通过test()而不是test访问对象。

Answer 3

我联系到你在头文件中设置'test'变量，它包含在Main.Cpp和Scene.Cpp中。不是吗？

如果您的标头包含在两个.cpp文件中，则会产生重复的符号。 #pragma once只能防止在编译同一个源文件时多次包含一个标题，在编译另一个源代码时它仍然可以包含第二次。

Answer 4

看起来它抱怨重新定义test。尝试重命名吗？

错误LINK2005已在Main.obj中定义

4 个答案: