C++包含继承同一父类的类时出现枚举重定义错误

你遇到的问题核心是头文件重复包含：main.cpp同时引入了Rect.h和Circle.h，而这两个头文件又都包含了Shape.h，导致Shape.h里的enum colors和Shape类被多次注入到同一个编译单元中，违反了C++中「一个实体只能定义一次」的规则。

下面是几种标准的解决方法，按推荐程度排序：

方法1：使用头文件保护（Header Guards） #

这是C++标准规定的跨平台方案，兼容性拉满。修改shape.h，在文件首尾添加保护宏：

// shape.h
#ifndef SHAPE_H
#define SHAPE_H

enum colors {RED=0, BLUE, GREEN};

class Shape {
    public:
        colors col;
        void setColor(colors c) {
            col = c;
        }
};

#endif // SHAPE_H

原理很简单：第一次包含shape.h时，SHAPE_H未定义，会执行中间代码并标记SHAPE_H为已定义；后续再包含时，因为SHAPE_H已存在，会直接跳过中间内容，彻底避免重复定义。

注意：宏名要唯一，通常用头文件名的大写加下划线形式，避免和其他代码的宏冲突。

方法2：使用#pragma once #

这是GCC、Clang、MSVC等主流编译器都支持的扩展语法，写法比头文件保护简洁得多：

// shape.h
#pragma once

enum colors {RED=0, BLUE, GREEN};

class Shape {
    public:
        colors col;
        void setColor(colors c) {
            col = c;
        }
};

它的作用和头文件保护完全一致，只是由编译器自动处理重复包含逻辑，省去了手动写宏的麻烦。唯一的小缺点是它不属于C++标准语法，但现在几乎所有常用编译器都支持，日常开发中非常常用。

额外优化建议 #

• 养成习惯：所有头文件都要加头文件保护或#pragma once，不管当前有没有被多处引用，提前规避未来的重复包含问题。
• 可以把enum colors封装到Shape类内部，减少全局命名空间的污染：

// shape.h
#ifndef SHAPE_H
#define SHAPE_H

class Shape {
    public:
        enum colors {RED=0, BLUE, GREEN};
        colors col;
        void setColor(colors c) {
            col = c;
        }
};

#endif // SHAPE_H

使用时需要写成Shape::RED、Shape::colors，代码结构更规范，也降低了全局符号冲突的概率。

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