时间：2019-05-10 标签：c++deallocate 2d int array error

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] 是一个包含三个指针的数组。

What you want is this (live at 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×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

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