重载流和算术运算问题

Question

我遇到了重载运算符+和流<<的问题。我有一个带有重载运算符的类：

FuzzyNumber& FuzzyNumber::add(FuzzyNumber B)
{
    FuzzyNumber fuzzyResult;

    fuzzyResult.setA(this -> getA() + B.getA() );
    fuzzyResult.setB(this -> getB() + B.getB() );
    fuzzyResult.setC(this -> getC() + B.getC() );

    return fuzzyResult;
}

FuzzyNumber& operator+(FuzzyNumber& A, FuzzyNumber& B)
{
    return A.add(B);
}

All fields inside class are double type. Here is overloaded ostream operator>>

ostream& FuzzyNumber::streamWrite(ostream& outStream)
{
    outStream << "( "
              << this -> getA() << ", "
              << this -> getB() << ", "
              << this -> getC() << ")";

    return outStream;
}



ostream& operator<< (ostream& outStream, FuzzyNumber& fuzzyNumber)
{
    fuzzyNumber.streamWrite(outStream);
    return outStream;
}

当我输入 cout << 时，打算打印类似 (3, 4.3, 5.1) 的内容。模糊数；。它工作得很好，而且这也工作得很好：

FuzzyNumber fuzzyNumber = numA + numB;
cout << "A + B = " << fuzzyNumber << endl;

其中 numA 和 numB 是 FuzzyNumber 类型。 虽然当我用以下内容替换上面的行时程序停止运行：

cout << "A + B = " << (numA + numB) << endl;

也许默认运算符=有问题，但是这个类中没有动态变量，所以它不应该。

提前感谢您的帮助！

Answer 1

在operator+内部，您返回了对堆栈变量的引用。对于这个明显的例子，你的编译器应该警告你。

FuzzyNumber FuzzyNumber::add(FuzzyNumber B) const
{
    FuzzyNumber fuzzyResult;

    fuzzyResult.setA(this -> getA() + B.getA() );
    fuzzyResult.setB(this -> getB() + B.getB() );
    fuzzyResult.setC(this -> getC() + B.getC() );

    return fuzzyResult;
}

FuzzyNumber operator+(const FuzzyNumber& A, const FuzzyNumber& B)
{
    return A.add(B);
}

这段代码应该可以解决你的问题。我还添加了一些适当的 const 正确性。

Answer 2

尝试

FuzzyNumber FuzzyNumber::add(FuzzyNumber const &B) const
{
  FuzzyNumber fuzzyResult;

  fuzzyResult.setA(this -> getA() + B.getA() );
  fuzzyResult.setB(this -> getB() + B.getB() );
  fuzzyResult.setC(this -> getC() + B.getC() );

  return fuzzyResult;
}


FuzzyNumber operator+(FuzzyNumber const & A, FuzzyNumber const & B)
{
  FuzzyNumber res(A);
  res.add(B);
  return res;
}

并

ostream& operator<< (ostream& outStream, FuzzyNumber const & fuzzyNumber)
{
    fuzzyNumber.streamWrite(outStream);
    return outStream;
}

记住，您必须将 FuzzyNumber::streamWrite() 设置为常量！

Answer 3

临时对象不能绑定到非常量引用，只能绑定到常量引用。 operator<< 的重载采用非常量引用，因此它不适用于临时变量。它应该采用 const 引用来代替 FuzzyNumber：

ostream& operator<< (ostream& outStream, const FuzzyNumber& fuzzyNumber) {
   ...
}

这还意味着 FuzzyNumber 的 streamWrite() 和 < code>getA() 到 getC() 也应该声明为常量。

Answer 4

您的 add 函数返回对将不再存在的本地的引用，从而导致未定义的行为。 add 函数应按值返回以避免这种情况。