在头文件中定义无外部头依赖的小型函数是否为不良编程实践？

结论：这不仅不是不良编程实践，反而属于推荐的做法，具体原因和注意事项如下：

• 利于编译器内联优化：这类仅操作成员变量的小函数（比如你的示例void EnableDirtyFlag() { m_DirtyFlag = true; }）逻辑简单，放在头文件中定义时，编译器能直接看到函数实现，更容易进行内联优化，消除函数调用的开销，提升程序运行效率。
• 彻底规避循环引用：你特意提到这类函数无外部头依赖，这完全符合头文件设计的安全原则——不会因为引入额外头文件而触发头文件循环引用问题，同时也能保持头文件的轻量化。
• 提升代码可读性与维护性：Getter、Setter这类和类状态直接相关的小函数，放在头文件的类定义中（或紧邻类定义的头文件区域），能让阅读代码的人一眼看到类的完整接口，不用来回在.h和.cpp文件间跳转查找实现，降低维护成本。

需要注意的细节 #

如果是在类的外部（头文件中）定义这类成员函数，必须加上inline关键字，避免因为头文件被多个源文件包含而触发重复定义的编译错误：

class MyClass {
private:
    bool m_DirtyFlag;
public:
    void EnableDirtyFlag();
};

// 头文件中类外部定义时需加inline
inline void MyClass::EnableDirtyFlag() {
    m_DirtyFlag = true;
}

而如果是直接在类的内部定义函数，编译器会默认将其视为inline，无需额外添加关键字。

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