Realloc“随机”崩溃答案

【问题标题】：Realloc "random" crashRealloc“随机”崩溃
【发布时间】：2013-08-23 06:19:29
【问题描述】：

我有这个函数来读取一个三角形二维数组，但有时它会在 realloc 上崩溃。总是在第 6 次重新分配 (current_row = 7)。有时它运行良好。无法在 gdb 中重现错误（每次都有效）。怎么了？

TRIANGLE *read_triangle(char *file_name)
{
std::ifstream fin(file_name);
int current_row = 0, current_column = 0, buffer;
TRIANGLE *triangle = new TRIANGLE();
triangle->triangle_values[0] = new int[1];

while (fin >> buffer)
{
    if (current_column == current_row+1)
    {
        current_column = 0;
        triangle->triangle_values = (int**)realloc(&((void*)triangle->triangle_values), (++current_row+1)*sizeof(int*));
        triangle->triangle_values[current_row] = new int[current_row];
    }
    triangle->triangle_values[current_row][current_column++] = buffer;
}
triangle->rows = current_row-1;
return triangle;
}

三角定义

struct TRIANGLE 
{ 
    int **triangle_values; 
    int rows; 
    TRIANGLE(): triangle_values(NULL) 
    {
        triangle_values = new int*[1];
    } 
};

输入文件示例：

75
95 64
17 47 82
18 35 87 10
20 04 82 47 65
19 01 23 75 03 34
88 02 77 73 07 63 67
99 65 04 28 06 16 70 92
41 41 26 56 83 40 80 70 33
41 48 72 33 47 32 37 16 94 29
53 71 44 65 25 43 91 52 97 51 14
70 11 33 28 77 73 17 78 39 68 17 57
91 71 52 38 17 14 91 43 58 50 27 29 48
63 66 04 68 89 53 67 30 73 16 69 87 40 31
04 62 98 27 23 09 70 98 73 93 38 53 60 04 23

【问题讨论】：

不要使用手动内存管理。使用std::vector<int>，它也有 push_back 方法和 resize 方法，可以根据需要增加大小。
这里必须同意 Neil 的观点，当 C++ 提供更好的方法时，真正的 C++ 程序员通常不会使用遗留 C 的东西。如果您需要在 C++ 中使用 C 代码，它就在那里，否则应该避免。并且混合 new/realloc 是自找麻烦 :-)
不要像那样混合 realloc 和 new 。
我明白你的意思。我没有想到混合它们会导致问题。
此时并不重要，但可以将输入文件格式添加到您的问题中吗？理想情况下使用样本数据。（作为附录发布在问题的底部）。我怀疑这段代码会崩溃多少，尤其是如果你到目前为止应用了 cmets。

标签： c++ realloc

【解决方案1】：

您正在混合旧式 (realloc) 和新式 (new) 内存分配函数，这不能保证有效。我的意思是，只要你将它们分开，它们就可以工作，但使用new 分配内存，然后尝试使用realloc 扩展该相同内存是绝对不行的。

来自C++11 20.6.13，在谈到旧式函数如何控制其块的可达性时：

它还允许使用单独的分配区域实现 malloc()。

因此，不能保证新旧风格的记忆领域是相互关联的。

C++ 提供了各种美妙的集合类，其大小调整方法比malloc/realloc 要好得多。您应该通过使用它们来完全接受该语言（例如vector）。

通常，C++ 程序员不会使用遗留 C 的东西，除非他们正在编写需要在 C 和 C++ 中都可用的东西，在这种情况下，他们可能更好地称为C 程序员，至少是暂时的 :-)

【讨论】：

除非他在 TRIANGLE() 构造函数中使用 new 分配 triangle_values，否则我错过了它们实际上在 realloc() 或 free() 返回的指针上混合的地方new 或 delete 或 delete[] 在来自 malloc() 或 realloc() 的指针上。当然，我完全同意这是一个糟糕的主意。我非常好奇 TRIANGLE 构造函数和同样重要的析构函数中的内容。
你是对的，构造函数如下： TRIANGLE(): triangle_values(NULL) {triangle_values = new int*[1];} ..我没有声明析构函数，因为这只是一种命中和退出类型的应用程序。
@user2524502 那么这个答案是正确适用的，应该多读几遍以确保你理解它。并为水晶球+1。（如果可以的话，我会给它两票）。
不能用new int*[1];再用realloc()；这是一个完全的禁忌。请将信息连同结构定义一起转移到问题中。
@WhozCraig，实际上，我认为您就在那里，原始问题中的信息似乎没有显示冲突。上面的 OP 评论（和问题编辑）证实了冲突，所以我将保持原样。但是，老实说，在这种情况下，我会完全放弃 new 并使用其中一个集合类。如果您知道最大尺寸，new<int> 很好，但如果您想要可调整大小，则不能超过 vector 及其兄弟。