17

可能重复：
常量变量在标头中不起作用

在我用来创建共享对象的头文件中，我有以下内容：

#ifndef LIB_HECA_DEF_H_
#define LIB_HECA_DEF_H_

struct dsm_config {
    int auto_unmap;
    int enable_copy_on_access;
};

enum { NO_AUTO_UNMAP, AUTO_UNMAP } unmap_flag;
enum { NO_ENABLE_COA, ENABLE_COA } coa_flag;

const struct dsm_config DEFAULT_DSM_CONFIG = { AUTO_UNMAP, NO_ENABLE_COA };

<more code ...>

#endif

当我编译时，我收到以下错误：

cc -g -Wall -pthread libheca.c dsm_init.c -DDEBUG    master.c   -o master
/tmp/cciBnGer.o:(.rodata+0x0): multiple definition of `DEFAULT_DSM_CONFIG'
/tmp/cckveWVO.o:(.rodata+0x0): first defined here
collect2: ld returned 1 exit status
make: *** [master] Error 1

任何想法为什么？

4

4 回答 4

51

因为每次包含在实现文件文件中，都会创建一个新的结构实例（并存储在目标文件中）。

为避免这种情况，只需在头文件中将结构声明为“extern”并在实现文件中对其进行初始化：

// In your header file: 
extern const struct dsm_config DEFAULT_DSM_CONFIG;

// In your *.c file:
const struct dsm_config DEFAULT_DSM_CONFIG = { AUTO_UNMAP, NO_ENABLE_COA };

这将解决您的问题。

于 2013-01-25T16:38:19.083 回答
6

在 C 语言const中，默认情况下对象具有外部链接（与默认情况下具有内部链接的 C++ 相反）。因此，在您的代码中，您创建了DEFAULT_DSM_CONFIG具有外部链接的对象的多个定义——这明显违反了 C 语言的定义规则。

要么将你的对象声明为static const（如果你不介意有多个具有内部链接的对象），要么从头文件中删除定义（只留下一个非定义的extern const 声明）。

无论如何，这个问题已经被问过很多次了。查看在标题中不起作用的常量变量或进行搜索。

于 2013-01-25T16:49:00.857 回答
5

每个包含头文件的 c 文件都有该行

常量结构 dsm_config DEFAULT_DSM_CONFIG = { AUTO_UNMAP, NO_ENABLE_COA };

所以这些 c 文件中的每一个都定义了一个变量 dsm_config。如果您只想要一个变量 dsm_config 您需要将头文件中的声明更改为

extern const 结构 dsm_config DEFAULT_DSM_CONFIG;

并添加定义

常量结构 dsm_config DEFAULT_DSM_CONFIG = { AUTO_UNMAP, NO_ENABLE_COA };

只有一个c文件。

另一个不太好的解决方案是将头文件更改为

静态常量结构 dsm_config DEFAULT_DSM_CONFIG = { AUTO_UNMAP, NO_ENABLE_COA };

然后每个 c 文件定义它自己的 dsm_config，在链接期间其他翻译单元不可见。

于 2013-01-25T16:38:18.110 回答
2

每个源文件（.c，而不是 .h）都是单独编译的。在这些编译中的每一个中，dsm_config带有初始化器（= { values… }部分）的声明都会创建dsm_config. 因此，整个程序有多个定义。

通常，头文件应该只声明对象而不定义它们。为此，请删除头文件中的初始化程序，只留下没有初始化程序的声明。在一个源文件中，dsm_config通过使用初始化程序重复声明来定义。

于 2013-01-25T16:44:53.500 回答