2

我有这个函数来读取一个三角形的二维数组，但有时它会在 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
4

1 回答 1

3

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

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

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

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

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

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

于 2013-08-21T02:41:43.657 回答