C++ - 创建一个 2D 复制构造函数

Question

我正在努力学习我的期末考试，并尝试将我的动态二维数组复制构造函数放在一起。当我创建备份并打印它以查看它是否有效时，它会一遍又一遍地打印出我相信的相同内存地址。这是我的复制构造函数： 更新，这是我从文件 .txt 中读取数据的地方

void Matrix::readTemps(ifstream &inFile) 
{
    while (!inFile.eof())
    {
        for (int row = 0; row < mnumRows; row++)
        {
            for (int col = 0; col < mnumCols; col++)
            {
                inFile >> Temps[row][col];
            }
        }
    }
}
Matrix::Matrix(const Matrix & original) 
{
    mnumRows = original.mnumRows;
    mnumCols = original.mnumCols;
    Temps = new double*[mnumRows];
    for (int row = 0; row < mnumRows; row++) 
        Temps[row] = new double[mnumCols];
}

---

Matrix& Matrix::operator=(const Matrix & second)  
{  
    if (this != &second)
    {
        delete[] Temps;
        mnumRows = second.mnumRows;
        mnumCols = second.mnumCols;
        Temps = new double*[second.mnumRows];
        for (int row = 0; row < mnumRows; row++)
            Temps[row] = new double[mnumCols];
    }
    return *this;
}

更新，这是在我的 main.cpp 中：

//Example of overloaded assignment operator.
Matrix TestMatrix;
TestMatrix = NYCTemps;

//Example of copy constructor.
Matrix copyOfMatrix(NYCTemps); // The traditional way to copy the phonebook.

NYCTemps.display();
copyOfMatrix.display();
cout << endl;

我相信我的赋值重载运算符也是正确的，但我发布它只是为了确认它对更聪明的人来说是好的。

Answer 1

在您的复制构造函数版本中 并复制您只分配的赋值运算符 二维数组的内存，但您不复制原始值 矩阵转换为新矩阵。

你应该在这两种方法中添加这些行

for (int r = 0; r < mnumRows; ++r)
    for (int c = 0; c < mnumCols; ++c)
        Temps[r][c] = original.Temps[r][c];  // or second.Temps[r][c]

Temps 是二维数组，所以在复制赋值运算符中 它必须被删除（首先删除为每一行中的列分配的内存，然后删除行的内存）

for (int r = 0; r < mnumRows; ++r)
    delete[] Temps[r];
delete[] Temps;