malloc崩溃程序第4次电话

Question

我正在C中制作游戏原子，人们轮流将原子放入网格空间。我试图实现一个移动跟踪系统，其中每次移动时，移动数组都会增加一个。我知道效率不是很高（我知道我不检查malloc返回值），但只是试图让它至少现在工作。下面是我如何做它：

int move_count_temp = (game->move_count)+1; 
move_t* moves; 

//Check if it's first move 
if (game->moves == NULL) moves = (move_t*)malloc(sizeof(move_t*)); 
else moves = (move_t*)malloc(move_count_temp*sizeof(move_t*)); 

// Copy old moves over 
for (int i=0; i<game->move_count;i++) { 
    moves[i].x = game->moves[i].x; 
    moves[i].y = game->moves[i].y; 
} 

//Copy current move 
moves[move_count_temp-1].x = y_coordinate; 
moves[move_count_temp-1].y = x_coordinate; 

// Free old moves pointer and assign new one 
free(game->moves); 
game->moves = moves; 
game->move_count = move_count_temp; 

的问题是关于该样品，在4号线mallocing moves时崩溃第四招。我一直试图修复它一段时间，只是不知道为什么会发生。任何见解都会有所帮助，谢谢。

P.S.游戏结构有一个指向move结构的指针，这是move_t结构;

struct move_t { 
    int x; 
    int y; 
}; 

Answer 1

这些分配都是错误的：

if (game->moves == NULL) moves = (move_t*)malloc(sizeof(move_t*)); 
else moves = (move_t*)malloc(move_count_temp*sizeof(move_t*)); 

您需要为move_t结构，不是指针分配空间move_t结构。该规范的方式做，这有助于避免这样的错误：

if (game->moves == NULL) { 
    moves = malloc(sizeof *moves); 
} else { 
    moves = malloc(sizeof *moves * move_count_temp); 
} 

也就是说，没有理由投调用在C malloc()的结果，这只会混乱的代码。而且，在sizeof操作数中使用标识符而不是显式类型有助于避免发布代码中的错误，并且使代码更易于维护。

这就是说，似乎发布的代码应该使用realloc()而不是malloc()和free()来重新分配game->moves。