复合赋值重载 (C++)

Question

我在两次通过引用传递同一个对象到一个函数时遇到了一个问题，一次作为常量，一次像往常一样（可变）。

我想知道修复此类代码的最佳方法是什么。 （一个方便的例子是重载复合赋值运算符）。

class MyClass
{
    int a, b;
public:
    MyClass(int a, int b) : a(a), b(b) {}
    MyClass& operator+=(const MyClass& ob)
    {
        this->a += ob.b;
        this->b += ob.a;    
        return *this;
    }
};

当然，这段代码没有多大意义，但它突出了问题： 编写如下代码时：

MyClass ob(1, 2);
ob += ob;

我希望 ob.b 的值为 3，但它的值为 5。

我想知道如何正确地实现这样的代码。 我应该小心实施，例如维护临时变量，还是有更好的通用解决方案来解决这些问题？ （例如按值传递）

Answer 1

使用复制的值（按值传递参数）：

MyClass& operator+=(const MyClass ob)

或者更优化的解决方案：比较对象地址，如果对象是同一个对象，则使用临时值：

MyClass& operator+=(const MyClass& ob)
{
    if (this == std::addressof(ob))
        return operator+=(MyClass(ob));
    this->a += ob.b;
    this->b += ob.a;    
    return *this;
}