如何在c中正确使用realloc？ [复制]

Question

#include <stdio.h>
#include <stdlib.h>
#include <string.h>
#include <stdbool.h>

#define LENGHT 20

typedef struct {
    unsigned id;
    char name[LENGHT];
    char genre[LENGHT];
    char nazionality [LENGHT];
    int year_carrier_started;
    bool check;
}Artist;

void print(Artist *arr, int *i);
void addOne(Artist *arr, int *i);

int main(void) {

    int index = 5;
    Artist *array = malloc(sizeof(int)*index);
    for (int i=0;i<index;i++)
    {
        printf("Insert a number:\n");
        scanf("%d",&array[i].id);
    }
    do
    {
        addOne(array,&index);
        print(array,&index);
    }while(1);

    system("pause");
    free(array);
    return EXIT_SUCCESS;
}


void addOne(Artist *arr, int *i)
{
    realloc(arr,sizeof(Artist)*(*i)+1);
    printf("Insert another one:\n");
    scanf("%d",&arr[*i].id);
    *i = *i +1;
    print(arr,i);
}


void print(Artist *arr, int *i)
{
    for (int j=0;j<*i;j++)
    {
        printf("Number: %d position %d\n",arr[j].id,j);
    }
}

大家好，我从我的一段程序中得到了这个结构，我需要做的是每次用户想要添加一个新的艺术家时重新分配。在添加了大约 10/15 的用户后，它会无缘无故地崩溃（至少对我来说没有）。我究竟做错了什么？ （Dat *i 是通过引用传递的，它计算已经有多少艺术家）。

Answer 1

如何在c(?)中正确使用realloc

在添加了大约 10/15 个用户后，它会因为 no 原因而崩溃

@Christian Gibbons 好建议：使用realloc() 的返回值。 array 以前的值可能无效。

---

乘以正确的值

// sizeof(Artist)*(*i)+1
sizeof(Artist)*((*i)+1)

使用realloc()的返回值

// realloc(array,sizeof(Artist)*(*i)+1);
void *new_ptr = realloc(array,....

检查 realloc() 结果

// realloc(array,sizeof(Artist)*(*i)+1);
void *new_ptr = realloc(array,....
if (new_ptr == NULL) OutOfMemory();
else array = new_ptr;

考虑对象的大小而不是类型

// realloc(array,sizeof(Artist)*(*i)+1);
void *new_ptr = realloc(array, sizeof *array *((*i)+1)); 

---

大家一起

Artist *array = malloc(sizeof *array);
if (array == NULL) Handle_OutOfMemory();
...
void *new_ptr = realloc(array, sizeof *array * ((*i)+ 1));
if (new_ptr == NULL) {
  // `array` still has *i elements assigned to it.
  Handle_OutOfMemory();
} else {
  array = new_ptr;
  (*i)++;
}

Answer 2

realloc 将尝试在可能的情况下调整分配的内存大小，但如果没有足够的空间扩展到新的大小，则必须将数据移动到新位置。由于这个原因，Realloc 返回一个指向该位置的指针。您还应该进行测试以确保它没有出错。所以是这样的：

Artist *arr = malloc(sizeof(*arr) * i);
Artist *temp = realloc(arr, sizeof(Artist)*((*i)+1) // Borrowing this from Yunnosch
if(temp) {
    arr = temp;
}
else {
    free(arr);
    // more error handling
}