Question

这是整个班级（复制/粘贴应该可以使用）：

 sumOdd += /* left as an exercise */;

我想使用非成员运算符重载。代码会编译，但显示错误的结果。最后两行应显示：

7 + 5/3 = 26/3

14 + 5/3 = 47/3

，但显示：

7 + 5/3 = 3

14 + 5/3 = 3

问题很可能是这样的：

#include <cstdio>
#include <iostream>
using namespace std;

class Rational
{
    int _n = 0;
    int _d = 1;
public:
    Rational (int numerator = 0, int denominator = 1) : _n(numerator), _d(denominator) {};
    Rational (const Rational & rhs) : _n(rhs._n), _d(rhs._d) {};
    ~Rational();
    int numerator() const {
        return _n;
    };
    int denominator() const {
        return _d;
    };
    Rational & operator = (const Rational &);

};

Rational operator + (const Rational & lhs, const Rational & rhs)
{
    return ((lhs.numerator() * rhs.denominator()) + (lhs.denominator() * rhs.numerator()), lhs.denominator() * rhs.denominator());
}

Rational::~Rational()
{
    _n = 0;
    _d = 1;
}

std::ostream & operator << (std::ostream & o, const Rational & r)
{
    if (r.denominator() == 1) {
        return o << r.numerator();
    } else {
        return o << r.numerator() << '/' << r.denominator();
    }
}


int main()
{
    Rational a = 7;
    cout << "a is: " << a << endl;

    Rational b(5,3);
    cout << "b is: " << b << endl;

    cout << a << " + " << b << " = " << a + b << endl;
    cout << 14 << " + " << b << " = " << 14 + b << endl << endl;

    return 0;
}

因为如果我先将其更改为创建对象然后返回，它将正常工作：

Rational operator + (const Rational & lhs, const Rational & rhs)
{
    return ((lhs.numerator() * rhs.denominator()) + (lhs.denominator() * rhs.numerator()), lhs.denominator() * rhs.denominator());
}

我希望隐式转换，并且两种解决方案都能得出相同的结果。有人可以告诉我这里有什么区别吗？

Answer 1

return语句包含逗号运算符调用，丢弃第一个表达式（应该是分子）的结果，并仅用一个参数调用构造函数。您应该使用正确的初始化语法：

return Rational
(
    (lhs.numerator() * rhs.denominator()) + (lhs.denominator() * rhs.numerator())
,   lhs.denominator() * rhs.denominator()
);

或

return
{
    (lhs.numerator() * rhs.denominator()) + (lhs.denominator() * rhs.numerator())
,   lhs.denominator() * rhs.denominator()
};

或（更好）

return Rational
{
    (lhs.numerator() * rhs.denominator()) + (lhs.denominator() * rhs.numerator())
,   lhs.denominator() * rhs.denominator()
};

Answer 2

此行会导致您的错误：

return ((lhs.numerator() * rhs.denominator()) + (lhs.denominator() * rhs.numerator()), lhs.denominator() * rhs.denominator());

问题在于，它被整体评估，为您提供一个整数。然后，该整数将被馈送到Rational类的隐式调用的构造函数中。请注意，此行使用逗号运算符。对于类似a, b的代码，它首先求值a，丢弃结果，然后求值b并将结果保留为整个表达式的结果。那不是你想要的。

C ++运算符重载无法按预期工作

2 个答案: