多头文件中枚举status重复定义问题及解决方案咨询

问题根源很明确：你在A.h和B.h里各自定义了完全相同的enum status，当C.h同时包含这两个头文件时，预处理器会把两个枚举定义都展开到C.cpp中，编译器看到重复的类型定义，自然报错。

下面是几种实用的解决办法：

方案1：将枚举移到公共头文件（推荐） #

把重复的枚举定义抽出来放到一个公共头文件里，所有需要用到的地方都包含这个公共头，同时给所有头文件加上头文件保护，防止同一头文件被多次展开。

修改后的代码示例： #

1. 创建common.h：

#ifndef COMMON_H
#define COMMON_H

enum status
{
    allGood = 0,
    allBad  = 1
};

#endif // COMMON_H

2. 修改A.h：

#ifndef A_H
#define A_H

#include "common.h"

status somefunctionA();

#endif // A_H

3. 修改B.h：

#ifndef B_H
#define B_H

#include "common.h"

status somefunctionB();

#endif // B_H

4. 给C.h也加上头文件保护：

#ifndef C_H
#define C_H

#include "A.h"
#include "B.h"

// 原C.h的其他内容

#endif // C_H

这样不管是单独编译A.cpp/B.cpp，还是编译C.cpp，enum status只会被定义一次，完全解决重复定义问题。

方案2：用命名空间或强类型枚举隔离名字 #

如果不想抽公共头文件，可以给每个枚举加上命名空间，或者用C++11的enum class（强类型枚举），让两个枚举的名字不冲突。

命名空间示例： #

A.h修改为：

#ifndef A_H
#define A_H

namespace A {
enum status
{
    allGood = 0,
    allBad  = 1
};
}

A::status somefunctionA();

#endif // A_H

B.h修改为：

#ifndef B_H
#define B_H

namespace B {
enum status
{
    allGood = 0,
    allBad  = 1
};
}

B::status somefunctionB();

#endif // B_H

调用函数时，返回值的类型要带上命名空间，比如在C.cpp里：

#include "C.h"

int main()
{
    A::status resA = somefunctionA();
    B::status resB = somefunctionB();
    // 其他逻辑
    return 0;
}

强类型枚举（enum class）示例： #

A.h修改为：

#ifndef A_H
#define A_H

enum class AStatus
{
    allGood = 0,
    allBad  = 1
};

AStatus somefunctionA();

#endif // A_H

B.h修改为：

#ifndef B_H
#define B_H

enum class BStatus
{
    allGood = 0,
    allBad  = 1
};

BStatus somefunctionB();

#endif // B_H

这种方式下两个枚举是完全独立的类型，不会有任何名字冲突，适合两个枚举语义不同的场景。

内容的提问来源于stack exchange，提问作者Hyperian