Вы правильно догадались исправить:
< **(array_arg + counter) = index;
--------
> *(*array_arg + counter) = index;
Старая запись выполняла адресную арифметику на параметре-указателе, который ссылается на одиночную переменную - очевидная проблема. Новая запись сперва извлекает адрес массива, затем вычисляет элемент внутри - всё работает как нужно.
Теперь код работает и не падает. Но есть ещё пара мест, которые можно улучшить.
arr_size
и n_memb
- дублируют друг друга. Размер массива в байтах должен быть пропорционален числу элементов в массиве. Но arr_size
растёт значительно быстрее чем n_memb
. Например, если n_memb == 105
, то arr_size == 20971520
. Слишком много. Если убрать дублирование, то получится:
int save_arg(int **array_arg, int index) {
static int counter = 0;
static int n_memb = 5;
if (counter == 0) {
*array_arg = calloc(n_memb, sizeof(**array_arg));
} else {
if(counter == n_memb){
n_memb += n_memb;
*array_arg = realloc(*array_arg, n_memb * sizeof(**array_arg));
}
}
*(*array_arg + counter) = index;
counter++;
return counter;
}
*p = realloc(*p, ...)
- известная ошибка. Когда памяти достаточно, всё хорошо. Если памяти не хватит, realloc
вернёт NULL
, указатель обнулится и старая память утечёт. Исправляем:
int save_arg(int **array_arg, int index) {
static int counter = 0;
static int n_memb = 5;
if (counter == 0) {
*array_arg = calloc(n_memb, sizeof(**array_arg));
} else {
if (counter == n_memb) {
int new_n_memb = 2 * n_memb;
int *tmp = realloc(*array_arg, new_n_memb * sizeof(**array_arg));
if (tmp == NULL) {
return -1; // какой-то признак ошибки
}
n_memb = new_n_memb;
*array_arg = tmp;
}
}
*(*array_arg + counter) = index;
counter++;
return counter;
}
Отдельная обработка пустого массива не нужна и calloc
не нужен. Можно обойтись одним realloc
. Работает чуть быстрее, так как с верхнего уровня ушла почти бесполезная проверка counter == 0
:
int save_arg(int **array_arg, int index) {
static int counter = 0;
static int n_memb = 0;
if (counter == n_memb) {
int new_n_memb = (n_memb == 0) ? 5 : 2 * n_memb;
int *tmp = realloc(*array_arg, new_n_memb * sizeof(**array_arg));
if (tmp == NULL) {
return -1; // какой-то признак ошибки
}
n_memb = new_n_memb;
*array_arg = tmp;
}
*(*array_arg + counter) = index;
counter++;
return counter;
}
Функцию нельзя вызывать для разных массивов, так как её внутреннее состояние сильно связано с конкретным одним массивом. Исправляем:
typedef struct {
int n_memb;
int counter;
int *array;
} args_t;
int save_arg(args_t *args, int index) {
if (args->counter == args->n_memb) {
int n_memb = (args->n_memb == 0) ? 5 : 2 * args->n_memb;
int *tmp = realloc(args->array, n_memb * sizeof(*args->array));
if (tmp == NULL) {
return -1; // какой-то признак ошибки
}
args->n_memb = n_memb;
args->array = tmp;
}
*(args->array + args->counter) = index;
args->counter++;
return args->counter;
}
Использование:
args_t args = {0};
for (int i = 0; i < 10; i++) {
save_arg(&args, i);
}
for (int i = 0; i < 10; i++) {
printf("%i", *(args.array + i));
}
Осталось отрефакторить и получится то что у меня всегда получается в подобных случаях:
typedef struct {
int size;
int capacity_;
int *array;
} args_t;
void args_t_clear(args_t *args) {
free(args->array);
args->size = 0;
args->capacity_ = 0;
args->array = NULL;
}
bool args_t_reserve(args_t *args, int capacity) {
if (capacity > args->capacity_) {
int *tmp = realloc(args->array, capacity * sizeof(*args->array));
if (tmp == NULL) {
return false;
}
args->capacity_ = capacity;
args->array = tmp;
}
return true;
}
bool args_t_append(args_t *args, int index) {
if (args->size == args->capacity_) {
int capacity = (args->capacity_ == 0) ? 5 : 2 * args->capacity_;
if (!args_t_reserve(args, capacity)) {
return false;
}
}
args->array[args->size] = index;
++args->size;
return true;
}
int main() {
args_t args = {0};
for (int i = 0; i < 10; ++i) {
args_t_append(&args, i);
}
for (int i = 0; i < args.size; ++i) {
printf("%i", args.array[i]);
}
args_t_clear(&args);
}
**(array_arg + counter) = index;
- здесь вы пишете мимо памяти. Вероятно вы её портите. Падает всё позже.calloc
выделяет больше памяти чем нужно.arr_size
растёт быстрее чем вы его используете. Оба недочёта не приводят к сбоям, но получается неаккуратно.array_arg
,counter
иnmemb
в одну структуру и не работать с ней? Код по сути будет тот же, а вы получите самодостаточный контейнер. Вы думали, что будет если обратиться к вашей функции с разными массивами?**(array_arg + counter) = index;
на*(*array_arg + counter) = index;
, вроде не падает.