Clang-Tidy提示operator=应返回ABC&但代码已符合要求，是什么原因？

警告触发原因 #

你写的ABC& operator=(const ABC* obj)不属于C++规范定义的标准拷贝赋值运算符，属于自定义的赋值运算符重载：

• C++标准要求拷贝赋值运算符的参数为const ABC&（const左值引用），而非指针类型。
• Clang-Tidy的cppcoreguidelines-c-copy-assignment-signature检查规则会校验所有operator=重载的签名，你的实现虽然返回值确实是ABC&，但参数不符合规则预期，属于工具提示文本描述偏差，误将参数不匹配的问题报成了返回值问题。

Clang-Tidy: Operator=() should return 'ABC&'

代码隐含的其他问题 #

你当前的代码虽然能运行，但存在设计漏洞：

• set函数中的*this = *obj会调用编译器自动生成的默认拷贝赋值运算符，和你自定义的指针版operator=完全无关，你手写的赋值运算符最终还是依赖默认实现，没有实际自定义价值。
• 指针版operator=中的obj!= this判断仅在你写a = &a这种极端场景下生效，常规调用场景下完全没用，属于冗余代码。
• 如果后续代码迭代中误将set中的*this = *obj改成*this = obj，会立刻触发无限递归，导致栈溢出崩溃。

修复方案 #

方案1：保留指针版赋值运算符，忽略警告 #

如果你确实需要保留指针参数的赋值运算符重载，可以在运算符实现行上方加NOLINT注释消除警告：

// NOLINTNEXTLINE(cppcoreguidelines-c-copy-assignment-signature)
ABC& ABC::operator = (const ABC *obj){
    if(obj!= this) set(obj);
    return *this;
}

方案2：改为标准拷贝赋值运算符（更推荐） #

按照C++规范和核心指南要求，改写为引用参数的标准实现，同时避免递归调用：

class ABC {
public:    
    void set(const ABC& obj) {
        // 直接写成员赋值逻辑，不要调用operator=避免递归
        this->member1 = obj.member1;
        this->member2 = obj.member2;
        // 其余成员赋值...
    }

    ABC& operator=(const ABC& obj);
private:
    // 你的类成员定义示例
    int member1;
    int member2;
};

ABC& ABC::operator = (const ABC& obj){
    if(this!= &obj) {
        set(obj);
    }
    return *this;
}

调整后既符合规范，也不会再触发Clang-Tidy警告。

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