Question

我试图为我正在做的项目重载+运算符，但这种情况一直在发生。我认为原因是我创建的对象在调用操作符时被删除。不知道怎么解决它。这是我的代码的一部分：

Matrix Matrix::operator+ (const Matrix& m) const
{
    //something wrong here
    Matrix sum(rows, cols);

    for (int i = 0; i < cols; i++)
    {
        for (int j = 0; j < rows; j++)
        {
            sum.element[i][j] = this->element[i][j] + m.element[i][j];
        }
    }

    return sum;
}

其他信息

Matrix::Matrix(int r, int c)
{
    rows = r;
    cols = c;
    element = new int *[c];
    for (int i = 0; i < c; i++)
    {
        element[i] = new int[r];
    }
    for (int i = 0; i < cols; i++)
    {
        for (int j = 0; j < rows; j++)
        {
            element[i][j] = 0;
        }
    }
}


Matrix::Matrix(const Matrix& m)
{
    this->element = m.element;
}


Matrix::~Matrix()
{
    //freeing up the arrays
    for (int i = 0; i < cols; i++)
    {
        delete[] element[i];
        element[i] = NULL;
    }
    delete[] element;
    element = NULL;
}


Matrix& Matrix::operator= (const Matrix& m)
{
    //problem if rows and cols are longer in the new matrix
    //need to create a temp matrix to expand to new one
    for (int i = 0; i < cols; i++)
    {
        for (int j = 0; j < rows; j++)
        {
            this->element[i][j] = m.element[i][j];
        }
    }
    return *this;
}


int* Matrix::operator[] (int n)
{
    return element[n];
}

我得到的具体错误是：

Debug Assertion失败！

表达式：_BLOCK_TYPE_IS_VALID（pHead-＆gt; nBlockUse）

在第52行，我这样做了：

 Matrix total = mat + m;

mat和m都是对象矩阵

Answer 1

我猜这个问题是由这个引起的：

Matrix::Matrix(const Matrix& m)
{
    this->element = m.element;
}

这是一个无效的复制构造函数，原因有两个。首先，您尚未初始化rows或cols。其次，你现在有两个Matrix个对象指向同一个内存 - 这两个对象都会在销毁时将其删除：

{
    Matrix m1(3, 5);
    Matrix m2 = m1;
} // BOOM!

您需要在此处进行深层复制：

Matrix::Matrix(const Matrix& m)
: rows(m.rows), cols(m.cols)
{
    element = new int *[cols];
    for (int i = 0; i < cols; ++i) {
        element[i] = new int[rows];
        for (int j = 0; j < rows; ++j) {
            element[i][j] = m.element[i][j];
        }
    } 
}

Debug Assertion Failed错误

1 个答案: