0

我不断收到错误检测到堆损坏。我已经阅读了这里的几个问题，但我无法完全找出我的代码中导致这种情况的原因。我正在尝试创建一个二维数组，该数组将保存从文本文件中读取的矩阵。

// Create a 2d matrix to hold the matrix (i = rows, j = columns)
matrix = new int*[cols];

for(int i = 0; i <= cols; i++) {
    matrix[i] = new int[rows];
}

// Populate the matrix from the text file
for (int i = 0; i < rows; i++) {
    for (int j = 0; j < cols; j++) {
        inputFile >> matrix[i][j];
    }
}

我的析构函数是：

for(int i = 0; i <= cols; i++) {
    delete[] matrix[i];
}

delete[] matrix;

我试过调试，但在这种情况下确实有很大帮助。有什么建议么？

4

2 回答 2

5 
matrix = new int*[cols];

for(int i = 0; i <= cols; i++) {
    matrix[i] = new int[rows];
}

对于包含元素的数组cols，索引是从0到cols - 1包含。

堆损坏将被检测到

delete [] matrix;

由于matrix[cols]写入数组边界之外的位置。

更新

正如@DanielKO（谢谢好友：p）在评论中指出的那样

不匹配，“填充矩阵...”循环使“i”在“行”上迭代，而它应该迭代“cols”。

于 2013-07-09T02:05:41.950 回答
0 
for (int i = 0; i < rows; i++) {
    for (int j = 0; j < cols; j++) {
        inputFile >> matrix[i][j];

当您分配时，您从 i 中的 0 变为 cols。现在您将 i 更改为行。

编辑：下面将尊重您评论的行/列规则并遵循 RAII：

std::vector<std::vector<int>> matrix(rows, std::vector<int>(cols));

for( int i=0; i<rows; ++i ) {
   for( int j=0; j<cols; ++j ) {
      inputFile >> matrix[i][j];
   }
}

// no need for delete matrix cleaned up when leaving scope.
于 2013-07-09T02:08:44.963 回答