c ++取消分配2d int数组错误

Question

在下面的代码段中，我得到非常令人沮丧的错误。 多数民众赞成在我的数组。

int **tab2 = new int*[3];

我像这样分配。

for(i = 0; i < 10; i++) {
    tab2[i] = new int[3];
    tab2[i][0] = 40;
    tab2[i][1] = 10;
    tab2[i][2] = 100;
}

然后，在使用它之后，我想销毁它。

for(i = 0; i < 10; i++) {
    delete [] tab2[i];
}
delete [] tab2;

这会导致核心转储每一次。 我尝试了许多不同的方法来销毁它，每次都收到此错误。 我在这里弄错了什么？

Answer 1

这个

int **tab2 = new int*[3];

不按照您的想法去做。

您需要一个包含十个（10）指针的数组，每个指针指向三个整数的数组。

new int*[3]是一个包含三个指针的数组。

您想要的是这个（住在coliru） ：

#include <iostream>

int main() {

  int **tab2 = new int*[10];

  for(int i = 0; i < 10; i++) {
    tab2[i] = new int[3];
    tab2[i][0] = 40;
    tab2[i][1] = 10;
    tab2[i][2] = 100;
  }

  for(int i = 0; i < 10; i++) {
    delete [] tab2[i];
  }
  delete [] tab2;

}

Answer 2

用

int **tab2 = new int*[3];

您分配了一个大小为3的指针数组。

for(i = 0; i < 10; i++) {
    tab2[i] = new int[3];
    //...
}

您最多可以访问索引9 。 那肯定会出错。 删除过程对我来说很好。 要修复它，您应该分配一个大小为10而不是3的指针数组，例如

int **tab2 = new int*[10];

Answer 3

看起来您想要做的是创建一个N by M数组，其中N在运行时是已知的，M是固定的（在本例中为3）。

为什么不这样做呢？

{
    std::array<int, 3> defaults = {{ 40, 10, 100 }};
    std::vector<std::array<int, 3>> thing(10, defaults);
}

向量超出范围时， thing会自动释放，它的大小可以在运行时设置。 您仍然可以通过相同的方式访问结构：

thing[1][2] = 3

使用标准容器和智能指针可以轻松避免手动进行内存管理。 这样做将使您的代码更整洁，并减少了悬空指针和内存泄漏的机会。