Изменить размер массива структур внутри функции

Question

Я хочу динамически добавлять новые элементы в массив структур. Я добавил минимальный пример того, что segfaults. Я думаю, что мне нужно передать указатель struct data **arr функции перераспределения и добавить новый элемент правильно, но я не смог сделать это правильно.

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

struct data { 
    char s[10]; 
    int arr[10]; 
}; 

void add_one_element(struct data *arr, int *num_elements) 
{ 
    /*increment counter*/ 
    *num_elements +=1; 
    arr = realloc(arr, *num_elements * sizeof(struct data)); 

    strcpy(arr[*num_elements-1].s, "ABC"); 
    for(int i = 0; i < 10; i++) 
    arr[*num_elements-1].arr[i] = i; 
} 


int main() 
{ 
    struct data *arr = NULL; 
    int num_elements = 0; 
    add_one_element(arr, &num_elements); 

    printf("arr.s = %s\n", arr[num_elements-1].s); 
    for(int i = 0; i < 10; i++) 
     printf("arr[%d].arr[%d] = %d\n", num_elements-1, i, arr[num_elements-1].arr[i]); 

    free(arr); 
    return 0; 
} 

EDIT 1: попытался исправить проблему. На этот раз я получаю test(91537,0x7fff79532000) malloc: *** error for object 0x7fff5f5c0ad0: pointer being realloc'd was not allocated. Что свидетельствует о том, что перераспределение не удалось.

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

struct data { 
    char s[10]; 
    int arr[10]; 
}; 

void add_one_element(struct data **arr, int *num_elements) 
{ 
    /*increment counter*/ 
    *num_elements +=1; 
    arr = realloc(arr, *num_elements * sizeof(struct data)); 

    strcpy(arr[*num_elements-1]->s, "ABC"); 
    for(int i = 0; i < 10; i++) 
    arr[*num_elements]->arr[i] = i; 
} 


int main() 
{ 
    struct data *arr = NULL; 
    int num_elements = 0; 
    add_one_element(&arr, &num_elements); 
    printf("arr.s => %s\n", arr[num_elements-1].s); 
    for(int i = 0; i < 10; i++) 
     printf("arr[%d].arr[%d] = %d\n", num_elements-1, i, arr[num_elements-1].arr[i]); 
    return 0; 
} 

Answer 1

Вы правы, что вам нужно передать в struct data ** вашей функции. Вы бы сделать это следующим образом:

void add_one_element(struct data **arr, int *num_elements) 
{ 
    /*increment counter*/ 
    *num_elements +=1; 
    *arr = realloc(*arr, *num_elements * sizeof(struct data)); 

    strcpy((*arr)[*num_elements-1].s, "ABC"); 
    for(int i = 0; i < 10; i++) 
    (*arr)[*num_elements-1].arr[i] = i; 
} 

Так везде вы ссылки в этой функции будет меняться в *arrarr. Обратите внимание, что это выражение заключено в скобки для доступа к элементам массива.

Затем вызывается функция, как это:

add_one_element(&arr, &num_elements); 

Answer 2

С С проходит по значению, вы не можете изменить значение переменной для вызывающей без ссылки. В C это делается путем разыменования указателя на переменную, которую функция должна изменить. Таким образом, вам нужно передать указатель на переменную указателя.

(В отредактированных изменениях, вы не разыменования переменного указателя.)

realloc может возвращать NULL, и в этом случае, оригинальный указатель не был освобожден. Итак, вам нужно зафиксировать и проверить результат или realloc в отдельной переменной.

void add_one_element(struct data **arr, int *num_elements) 
{ 
    struct data *tmp; 
    int tmp_num = *num_elements + 1; 

    /*increment counter*/ 
    tmp = realloc(*arr, tmp_num * sizeof(struct data)); 
    if (tmp == NULL) { 
     /* XXX: handle the error */ 
     return; 
    } 

    strcpy(tmp[tmp_num-1].s, "ABC"); 
    for(int i = 0; i < 10; i++) 
    tmp[tmp_num-1].arr[i] = i; 

    *arr = tmp; 
    *num_elements = tmp_num; 
}