Управление динамическим массивом

[abr_question]

Добрый день! Подскажите, что я делаю не так при управлении размером динамического массива? Язык - c

int *console;

void init_values()
{
console = (int*)malloc(0*sizeof(int));
console_p = 0;
}

void console_add(int button, int d)
{
	console = (int*)realloc(console, sizeof(console) + sizeof(int));
	console[console_p++] = button * d;
}

Пробовал процедуру переписать щё так:

	console = (int*)realloc(console, console_p++ + sizeof(int));
	console[console_p] = button * d;

Ошибка одна и та же - вылетает Segmentation fault.

Задумка вобще такая: я переписываю прогу с Delphi на C (Linux) и мне нужна альтернатива TStringList, в котором содержатся только числа.
В процедуре console_add задумка такая: для динамического массива console должно резервироваться памяти ровно на один дополнительный елемент (тип int) при каждом вызове процедуры. Ошибка вылетает после 6 вызовов.

[netrino]

Ну вы же понимаете, что sizeof(console) == 4 байта на 32-х разрядных системах? Всегда. Мысль с console_p верна(если я правильно её понял, это ведь счётчик элементов?) но нужно домножать её на sizeof(int). А вообще так как вы делаете делать не хорошо, ибо медленно очень, увеличивайте размер массива сразу в два раза или как-то так. Отдельная переменная будет следить за размером массива, другая за кол-ством элементов в нём

[m0nax]

console = (int*)realloc(console, sizeof(console) + sizeof(int));

как заметили выше сизеоф тут не подходит, зато подходит _msize(console) который вернет как раз размер всего блока памяти

[abr_question]

Сообщение от netrino

Ну вы же понимаете, что sizeof(console) == 4 байта на 32-х разрядных системах? Всегда. Мысль с console_p верна(если я правильно её понял, это ведь счётчик элементов?) но нужно домножать её на sizeof(int). А вообще так как вы делаете делать не хорошо, ибо медленно очень, увеличивайте размер массива сразу в два раза или как-то так. Отдельная переменная будет следить за размером массива, другая за кол-ством элементов в нём

Точно! Неверно второй пример указал. Счас переделал:

void console_add(int button, int d)
{
	console = (int*)realloc(console, ++console_p * sizeof(int));
	console[console_p]= button * d;
}

Тем не менее, зависимость остаётся прежней - 6 раз вызовов и Segmentation fault.
Счас попробую с увеличением массива в 2 раза.

Сообщение от m0nax

console = (int*)realloc(console, sizeof(console) + sizeof(int));

как заметили выше сизеоф тут не подходит, зато подходит _msize(console) который вернет как раз размер всего блока памяти

Проблемка с _msize. Справочники указыают, что она хранится в malloc.h, но у меня malloc.h не содержит такой функции. Более того, весь /usr/include не содержит ни одного файла, в которой данная функция присутствует. Она не устарела случаем?

[abr_question]

Ааааагрх! Всё! Нашёл, кажется, ошибку
Я же инициализирую console_p = 0. Размер тоже правильно устанавливаю, как ++console_p * sizeof(int). А затем делаю глупейшую ошибку и присваиваю значение по адресу console_p, который уже за массивом.
Счас переписал

void init_values()
{
	console_p = -1;
	console = (int*)malloc(0*sizeof(int));
}
void console_add(int button, int d)
{
	console = (int*)realloc(console, (++console_p + 1) * sizeof(int));
	console[console_p]= button * d;
}

Вроде не вылетает при вызове приличное количество раз. Но если что, я щё отпишусь..
Спасибо всем, кто помог

[netrino]

int* console = 0;
unsigned int console_p = 0;
unsigned int list_size = 0;

void init_values()
{
    list_size = 10*sizeof(int);
    console = malloc(list_size);

    return;
}

void console_add(int button, int d)
{
    if( (console_p+1)*sizeof(int) >= list_size ) {
        list_size <<= 1; // lise_size *= 2;
        console = (int*)realloc(console, list_size);
    }
    console[console_p++] = button * d;

    return;
}

Как-то так должно выглядеть ) Проверьте - если опять будет ошибка, то возможно она в другом месте

[abr_question]

netrino, спасибо! Возьму вариант на заметку, если полетит ныне существующий.

У меня щё есть один вопрос, но он уже по указателям.
Хочу сделать так, чтобы в функция обрабатывала не один массив, а несколько. То есть, возникла необходимость функцию привести к общему виду. Фактически получилось так:

array_add(&console, &console_p, data);

void array_add(int **arr, int *arr_p, int d)
{
	*arr = (int*)realloc(*arr, (++(*arr_p) + 1) * sizeof(int));
	*arr[*arr_p] = d;
}

С arr_p проблем вобще не возникло - по указателю увеличивает на единицу. А вот с массивом не получается, хоть убейся. realloc не хочет никак увеличивать размерность массива. Поетому вторая строка функции выдаёт ошибку. Пробовал заменять int **arr на *arr[] - результат тот же.. Что делать?

[RoS]

Сообщение от abr_question

netrino, спасибо! Возьму вариант на заметку, если полетит ныне существующий.

У меня щё есть один вопрос, но он уже по указателям.
Хочу сделать так, чтобы в функция обрабатывала не один массив, а несколько. То есть, возникла необходимость функцию привести к общему виду. Фактически получилось так:

array_add(&console, &console_p, data);

void array_add(int **arr, int *arr_p, int d)
{
	*arr = (int*)realloc(*arr, (++(*arr_p) + 1) * sizeof(int));
	*arr[*arr_p] = d;
}

С arr_p проблем вобще не возникло - по указателю увеличивает на единицу. А вот с массивом не получается, хоть убейся. realloc не хочет никак увеличивать размерность массива. Поетому вторая строка функции выдаёт ошибку. Пробовал заменять int **arr на *arr[] - результат тот же.. Что делать?

Кажется, у тебя был одномерный массив? т.е. и функция принимать должна int*, либо int array[]...

[abr_question]

RoS, имеется ввиду ето?:

array_add(console, &console_p, data);

void array_add(int arr[], int *arr_p, int d)
{
	arr = (int*)realloc(arr, (++(*arr_p) + 1) * sizeof(int));
	arr[*arr_p] = d;
}

В етой функции до и после первой строчки размер arr = 4. То есть, не увеличивается...

[netrino]

С двойным указателем там верно было.. Насчёт

*arr[*arr_p] = d;

Тут нужно так:

(*arr)[*arr_p] = d;

а вообще я бы советовал завести отдельную структуру, в которой бы содержался как минимум указатель на память и счётчик объектов, так будет удобней