0

我创建了自定义随机数生成器,并将它们的全局函数放在一个名为 SamRandom.h 的文件中。该文件如下所示:

#ifndef _SAM_RANDOM_H
#define _SAM_RANDOM_H

#include <cstdlib>
#include <ctime>

void InitialiseRandomSeed()
{
    //...
}

//...

#endif

我正在一个非常复杂的面向对象程序中工作,该程序具有许多不同的组件。每次我添加一个无论如何都与该文件相关的文件时,我都会收到以下冲突消息:

LaserBase.obj:-1: error: LNK2005: "void __cdecl InitialiseRandomSeed(void)" (?InitialiseRandomSeed@@YAXXZ) already defined in main.obj
error: LNK1169: one or more multiply defined symbols found

在 MSVC 和 MinGW 上,我得到:

In function `Z20InitialiseRandomSeedv':
SamRandom.h:8: multiple definition of `InitialiseRandomSeed()'
error: first defined here
:-1: error: collect2: ld returned 1 exit status

为什么会这样?我认为预处理器指令应该防止这个问题发生......这真的让我发疯了!

PS:问题与函数名无关。

感谢您的任何努力

4

3 回答 3

2

当您将文件包含在 .cpp 文件中时,预处理器会直接将其复制到该文件中。因此,当您编译该 .cpp 文件时,目标文件(即使您不认识,编译器也会执行此步骤)将包含该函数。如果您将它包含在多个 .cpp 文件中,则每个目标文件都将包含该函数。当链接器尝试将所有目标文件链接到可执行文件时,它会找到函数 x 次并给出该错误。“找到一个或多个多重定义的符号”表示它不止一次地找到了您的一个函数(或变量)并且不知道如何处理它。

为了避免这种情况,您应该只放置您的功能签名,例如

//whatever.h
#ifndef _SAM_RANDOM_H
#define _SAM_RANDOM_H

void InitialiseRandomSeed();

#endif

在头文件中并将实现放入单独的 .cpp 文件中:

//whatever.cpp
#include "whatever.h"
#include <cstdlib>
#include <ctime>

void InitialiseRandomSeed()
{
    //...
}
于 2012-04-08T22:46:47.737 回答
2

标头保护确保函数不会在单个编译单元(.cpp 文件)中多次定义。但是,它们不会阻止函数在多个编译单元中被定义一次。这是因为 header 包含函数的定义,所以每个包含 header 的 .cpp 都有自己的函数定义。

您可以将定义移动到 SamRandom.cpp 并改为在标头中声明函数,如下所示:

void InitialiseRandomSeed();

或者,您可以指定该函数是内联的,这将使多个定义可接受:

inline void InitialiseRandomSeed()
{
    //...
}

假设该函数不是性能关键的并且不会被频繁调用,我会采用前一种方法。这使标头更具可读性,并且(通常)减少了编译时间。

于 2012-04-08T22:54:02.120 回答
1

您可以将定义分离到一个 .cpp 文件中,然后在标头中声明该函数。

于 2012-04-08T22:43:21.833 回答