重载operator=不与赋值一起编译

Question

github repo用代码尝试编写带有重载某些操作的矩阵类。
当我尝试用此笔画编译时，一切都出错了

result = (l_mtx + r_mtx);

Matrix.cpp：在函数“int main（）”中:
Matrix.cpp:36:12:错误：对“Matrix：：Matrix（Matrix）”的调用没有匹配函数

result = (l_mtx + r_mtx);   

然后是这个函数的几个候选项，我真的不明白。
我认为有copy构造函数和几个构造函数，但这不是operator=我认为应该在笔画中调用的。

matrix_class.h:73:5:注意：matrix::matrix(Matrix&)[类型=double]
（参数1没有已知的从“matrix”到“matrix&"的转换）

matrix_class.h:46:5:注意：matrix：：matrix（int,int）[with type=double]
（候选者需要2个参数，提供1个参数）

matrix_class.h:39:5:注意：matrix：：matrix（）[使用类型=double]
（候选项需要0个参数，提供1个参数）

然后出现错误:
matrix_class.h:96:18:错误：初始化“Matrix Matrix::operator=(Matrix)[with type=double]”的参数1

我想我没有正确地编写赋值运算符或复制构造函数的代码，但我找不到错误在哪里。抱歉，我问了个愚蠢的问题。谢谢你的关注。

//copy constructor
    Matrix(const Matrix<type> &org)
    {
        cout << "Making a copy of " << this << endl;
        row = org.getRow();
        column = org.getColumn();

        //allocate additional space for a copy
        data = new type* [row];
        for (int i = 0; i < row; ++i)
        {
            data[i] = new type [column];
        }

        for (int i = 0; i < row; ++i)
        {
            for (int j = 0; j < column; ++j)
            {
                data[i][j] = org.data[i][j];
            }
        }
    }

和运算符=

//assign constructor
Matrix<type> operator = (Matrix<type> r_mtx)
{
    if (row == r_mtx.getRow())
    {
        if (column == r_mtx.getColumn())
        {
            //TODO: удалить прежний объект?
            Matrix<type> temp(row, column);
            for (int i = 0; i < row; ++i)
            {
                for (int j = 0; j < column; ++j)
                {
                    temp.data[i][j] = r_mtx[i][j];
                }
            }
            return temp;
        }
        else
        {
            cout << "Assign error: matrix column are not equal!" << endl;
            exit(EXIT_FAILURE);
        }
    }
    else
    {
        cout << "Assign error: matrix rows are not equal!" << endl;
        exit(EXIT_FAILURE);
    }
}

Answer 1

声明复制赋值运算符，如

Matrix<type> & operator = ( const Matrix<type> &r_mtx )

问题是临时对象可能没有绑定到非常量引用。

考虑到赋值运算符应该返回对左手对象的引用。

您的赋值运算符实际上是无效的。它不是分配左手对象，而是创建一个临时对象。因此不存在任何赋值。

可以这样定义

Matrix<type> & operator = ( const Matrix<type> &r_mtx )
{
    if (row == r_mtx.getRow())
    {
        if (column == r_mtx.getColumn())
        {
            for (int i = 0; i < row; ++i)
            {
                for (int j = 0; j < column; ++j)
                {
                    data[i][j] = r_mtx[i][j];
                }
            }
            return *this;
        }
        else
        {
            cout << "Assign error: matrix column are not equal!" << endl;
            exit(EXIT_FAILURE);
        }
    }
    else
    {
        cout << "Assign error: matrix rows are not equal!" << endl;
        exit(EXIT_FAILURE);
    }
}