命名空间函数多重定义问题：编译失败原因排查求助

首先明确一点：你的#ifndef头文件防护完全生效了，问题出在另一个地方——你在头文件里直接定义了OP::getSymbol函数。

为什么头文件防护管不了这个问题？ #

头文件防护的作用是防止同一个.cpp文件（编译单元）里重复包含同一个头文件，比如你如果在main.cpp里两次#include "expression.h"，它能避免重复编译头内容。但它管不了多个编译单元各自包含头文件后，都生成了同一个函数的定义。

你看：

• 编译main.cpp时，它包含expression.h，编译器把getSymbol的定义编译进了main.o
• 编译expression.cpp时，它也包含expression.h，同样把getSymbol的定义编译进了expression.o
  链接阶段，链接器看到两个目标文件里都有同一个函数的完整定义，自然就抛出「multiple definition」错误了。

方案1：把函数定义移到.cpp文件（最推荐） #

这是C++里的标准做法，头文件只放声明，实现放在源文件：

• 修改expression.h，只保留函数声明：

#ifndef _EXPRESSION_H
#define _EXPRESSION_H
namespace OP {
    // 只声明，不写函数体
    char getSymbol(const unsigned char& o);
};
#endif /* _EXPRESSION_H */

• 在expression.cpp里实现函数：

#include "expression.h"

namespace OP {
    char getSymbol(const unsigned char& o) {
        return '-';
    }
};

方案2：给函数加inline关键字 #

如果这个函数很小，你希望编译器能把它内联优化，直接在头文件里给函数加inline：

#ifndef _EXPRESSION_H
#define _EXPRESSION_H
namespace OP {
    // 加inline后，链接器会允许多个编译单元有这个定义，只保留一份
    inline char getSymbol(const unsigned char& o) {
        return '-';
    }
};
#endif /* _EXPRESSION_H */

方案3：给函数加static关键字（不推荐） #

static会让每个编译单元里的getSymbol成为独立的副本，这样链接时就不会冲突了：

#ifndef _EXPRESSION_H
#define _EXPRESSION_H
namespace OP {
    static char getSymbol(const unsigned char& o) {
        return '-';
    }
};
#endif /* _EXPRESSION_H */

但这种方式不推荐，因为会生成多个函数副本浪费内存，而且如果函数后续要操作全局状态，不同编译单元的副本会各自维护状态，容易出奇怪的bug。

头文件里应该只放声明（类结构、函数原型、常量声明等），不要放函数或变量的定义（也就是不要写函数体或给变量赋值），除非你用inline（针对函数）或constexpr（针对常量）这类关键字来特殊处理。

内容的提问来源于stack exchange，提问作者l. schwarz