0

如主题中所述,我在将值从重载运算符传回主函数时遇到问题。我已经搜索了很多但没有效果。这是我的示例运算符。返回 Matrix mi 之前的行已将 cout 用于检查算法是否正常工作。我对乘法运算符有同样的问题。

矩阵.h

class Matrix 
{
public:
...
Matrix &operator+(const Matrix &m)
... 

private:
    int x;
    int y;
    double **tab;

};

矩阵.cpp

Matrix &Matrix::operator+(const Matrix &m)
{
if(x==m.x && y==m.y)
{
    Matrix temp(x,y);
    for(int i=0;i<x;i++)
    {
        for(int j=0;j<y;j++)
        {
            temp.tab[i][j]=tab[i][j]+m.tab[i][j];
        }
    }
    cout << temp<< endl;
return temp;
}
else
{
    char er[]={"error!\n"};
    throw er;
}

}

4

2 回答 2

2

根本问题是加法运算符不应该返回引用,而是返回值:

Matrix operator+(const Matrix &m);

这适用于乘法、减法等。

除了您要返回对仅存在于函数范围内的变量的引用这一事实之外,返回引用在语义上也没有意义。想想这个表达式:

B + C;

如果要返回一个引用,它应该引用什么?

一种常见的做法是实现Matrix& operator+=(const Matrix&)为成员运算符,然后将加法实现为非成员:

Matrix operator+(Matrix lhs, const Matrix& rhs)
{
  return lhs += rhs;
}

这使得操作对称 WRT LHS 和 RHS,允许在双方进行隐式转换。

于 2013-04-03T08:40:21.763 回答
0

简单的答案:永远不要返回对您在函数中创建的对象的引用。意思是,您的运算符应该返回 a Matrix,而不是 a Matrix&&属于返回类型,您可能希望将其放在那里以使其更明显)。

如果您为 Matrix 类提供适当的移动构造函数,则返回结果时不会有昂贵的副本。

于 2013-04-03T08:42:25.817 回答