Realloc“随机”崩溃

时间:2013-08-21 02:29:38

标签: c++ realloc

我有这个函数来读取三角形2d数组,但有时它会在realloc上崩溃。总是在6号realloc(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

1 个答案:

答案 0 :(得分:3)

您正在混合旧样式(realloc)和新样式(new)内存分配函数,这些函数无法保证正常工作。通过这个,我的意思是他们会工作,只要你把它们分开,但用new分配内存然后尝试用realloc扩展相同的内存是一个明确的禁忌

C++11 20.6.13开始,在讨论旧样式函数如何控制其块的可达性时:

  

它还允许malloc()通过单独的分配领域实现。

因此,无法保证新旧风格的记忆竞技场完全相互关联。

C ++提供了各种精彩的集合类,其调整方法比malloc / realloc好得多。您应该使用它们完全接受该语言(例如vector)。

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