3

我正在做一个需要以下结构的项目： 在此处输入图像描述

2 个 CPP 文件包含类，这些类需要 H 文件 1 中的裸函数。类和裸函数需要 H 文件 2 中的变量。

我可以拆分 CPP 文件，以便它们使用 2 个单独的文件，其中包含 nakes 和他们需要的变量。但我更喜欢使用这种结构。

看起来编译器跳过了#ifndef命令，我对问题进行了测试：

主要的：

#include <iostream>

//1>CPPFile2.obj : error LNK2005: "bool Test1" (?Test1@@3_NA) already defined in CPPFile1.obj
//1>CPPFile2.obj : error LNK2005: "bool Test2" (?Test2@@3_NA) already defined in CPPFile1.obj

int main()
{
}

CPP文件1：

#include <iostream>
using namespace std;

#include "HFile1.h"

CPP文件2：

#include <iostream>
using namespace std;

#include "HFile2.h"

H文件1：

#include "HFile2.h"

#pragma once
#ifndef Name1
#define Name1

//Use test1, this workes fine
//Use test2, this workes fine

#endif

H文件2：

#pragma once
#ifndef Name2
#define Name2

bool Test1 = false;
bool Test2 = false;

#endif

#ifndef #define #endif 结构怎么可能不能正常工作？

4

3 回答 3

6

您的问题是第二个标题是定义变量，而不仅仅是声明它们。程序中必须有一个定义，因此编译失败。这与包含守卫无关，因为包含守卫仅在单个翻译单元内进行保护。在您的情况下，每个 .cpp 文件都包含标题，因此分别定义相同的变量。

解决方案是仅在标头中声明变量并在单个翻译单元中定义它们：

#ifndef Name2
#define Name2
extern bool Test1;
extern bool Test2;
#endif
// single .cpp
bool Test1 = false;
bool Test2 = false;

虽然整个事情都有一些代码气味。您可能想要重新设计您的解决方案。在大多数情况下，使用全局变量并不是一个好的解决方案。

于 2012-08-26T14:48:22.570 回答
3

您收到链接错误；这与包括警卫无关。您已经在两个目标文件中定义了Test1和。Test2相反，您需要确保只有一个目标文件定义它，而另一个目标文件定义extern它。

于 2012-08-26T14:48:08.557 回答
1

永远不要在头文件中定义变量。您应该在 .cpp 中定义它，然后如果您想从另一个 .cpp 文件访问它，请创建一个带有“extern”的头文件。

文件1.cpp

bool Test1 = false; //define a var in your .cpp

文件1.h

extern bool Test1; //extern it in your header

文件2.cpp

#include "File1.h" //include the header of the variable

void Somefunction()
{
    if (Test1) //use the variable
    {
    //someting 
    }

}

然而，使用全局变量通常是一种不好的做法......

于 2012-08-26T14:56:53.167 回答