Почему не расширяется массив структур?

Question

[HoShik]

Имеется структура, а так же глобальный массив структур:

typedef struct
{
	char nickname[32];
	int score;
}board;
board leaderboard[1];

В определенном моменте кода выполняются следующие действия:

nick_pos=(short)check_nick(nickname); // check_nick возвращает позицию ника в массиве в случае если таковой имеется и -1 если его нет
if(nick_pos!=-1) now_play=(int)nick_pos;
else {
        now_play=sizeof(leaderboard)/sizeof(board)-1;
        sprintf(leaderboard[now_play].nickname,"%s",nickname);
        int new_size=sizeof(leaderboard)+sizeof(board);
        realloc(leaderboard, new_size);
}

Но после каждого выполнения, массив не расширяется. Не могу понять почему...

Comments

[Евгений Шатунов]

HoShik , такой вопрос. А почему, на твой взгляд, массив board должен расширяться?
Что именно ты сделал для того, чтобы board начал расширяться?

[lutokris]

мне кажется или нет, но как написано на одном форуме, realloc работает следующим образом:
1. делается malloc(новый_размер) во временную переменную внутри реалок
2. Если malloc() вернул NULL, то он и возвращается функцией, при этом c памятью, на которую указывает a ничего не происходит(и её можно освобождать через free())
3.Если всё норм, то по новому адресу переносится содержимое со старого адреса (вроде даже через обычный memcpy)
4. Освобождается старая память с адресом, переданным через a, а именно free(a)
5. возвращается новый указатель

и если я правильно понимаю будет так:

board *lB = &leaderboard;  //передаем адрес глобального массива в указатель lB

lieB = (board*)realloc(a, newsize *sizeof(board));  
if (lieB) 
    lB = LieB;  //если честно я сам  не до конца понял для чего создают другой указатель, можно было бы и наверное в старый
else error();
free(lieB); //как я понял уничтожаем именно сам временный указатель, а не то, на что он указывает. Ведь он оказывает туда же
                // куда и lB, где хранится наша новая расширенная структура.

[Евгений Шатунов]

lutokris, а зачем тиражировать весьма сомнительные мнения с безымянных форумов?

[lutokris]

В вашем случае если я правильно понимаю выражением
realloc(leaderboard, new_size);
вы просто выстреливаете в пустоту, ну или образно говоря совершаете ППА)
реаллок возвращает новый адрес на новую расширенную структуру, но он ни в какой указатель не принимается и уходит в никуда)

[Василий Дёмин]

Потому что глобальный массив следует создавать с помощью malloc, realloc вызывать перед добавлением нового элемента. У вас получается такая ситуация - вы перезаписываете никнейм в единственном элементе массива, а затем вызываете realloc.

[lutokris]

Евгений Шатунов, ну так я правильно же понимаю, что никакой массив в С++ расширить нельзя? и скорее всего это будет работать только на чистом Си.

[Евгений Шатунов]

lutokris, нет, не правильно понимаешь.
Тебе не зачем распространять какие-либо сторонние мнения, включая и мое собственное. Тебе нужно просто знать документацию.
И твое мнение, как и мое собственное, должно соответствовать документации.

Автор оперирует статически выделенным массивом. Для такого массива realloc вызывать бессмысленно. realloc может быть вызван только для динамически выделенного блока памяти, который массивом не является.

[HoShik]

Василий Дёмин, Пытаюсь сделать как вы указали. (Структура без изменений)

board *leaderboard=(board *)malloc(sizeof(board));
...
leaderboard=(board *)realloc(leaderboard, sizeof(leaderboard)+sizeof(board));
now_play=sizeof(leaderboard)/sizeof(board)-1;
sprintf(leaderboard[now_play].nickname,"%s",nickname);

Теперь по какой то причине, malloc выделяет только 4 байта под массив. Что делаю не так?

[lutokris]

Евгений Шатунов, то есть я правильно понимаю, если автор создал бы просто указатель типа leaderboard, то он мог бы его расширять с помощью realloc? А массив выделенный статически сразу занимает в памяти определенный диапазон памяти и вызов реаллока создаст по адресу массива в памяти дополнительные ячейки, но сам массив от этого не расширится потому что он создан статически? и как уже написали, поэтому нужно каждый раз перед расширением создавать его заново с большим количеством ячеек

[Василий Дёмин]

HoShik, sizeof(leaderboard) будет всегда возвращать размер нужный для хранения указателя. Нужно хранить размер массива в отдельной переменной.

[Василий Дёмин]

HoShik, Кстати, в си не обязательно делать явное приведение указателей типа void*. Можно писать так:
board *leaderboard = malloc(sizeof(board));
Также вместо sprintf лучше использовать strcpy

[lutokris]

HoShik, чтобы получить размер не указателя, а самой структуры нужно
sizeof(*leaderboard)

[Василий Дёмин]

lutokris, Автору нужен размер массива, а не размер структуры

[Евгений Шатунов]

lutokris,

А массив выделенный статически сразу занимает в памяти определенный диапазон памяти

Верно.

и вызов реаллока создаст по адресу массива в памяти дополнительные ячейки, но сам массив от этого не расширится потому что он создан статически

Не верно. realloc оборвется с ошибкой, т.к. переданный указатель не является динамически выделенным блоком памяти. Никаких действий с памятью произведено не будет.

если автор создал бы просто указатель типа leaderboard, то он мог бы его расширять с помощью realloc

Частично верно. Указатель - это просто переменная, в которую можно записать любой адрес. Со стека, из статической памяти, адрес динамически выделенного блока.
realloc можно использовать только для динамической памяти. Следовательно, изначально указатель должен быть инициализирован результатом malloc или calloc.

поэтому нужно каждый раз перед расширением создавать его заново с большим количеством ячеек

Не верно. realloc занимается именно изменением размера выделенного блока памяти. Ничего никуда каждый раз заново выделять не надо.

[HoShik]

Василий Дёмин,

Кстати, в си не обязательно делать явное приведение указателей типа void*.

Формально я пишу на с++, поскольку проект пишу на winbgi (он не работает на чистом си) и препод требует написание на си. Компилятор ругался пришлось приводить к board

Также вместо sprintf лучше использовать strcpy

Хорошо, спасибо, учту

Answer 1

[HoShik]

Вопрос был решен в комментариях.
Я пытался расширить статический массив с помощью realloc (так нельзя, почему и как быть - см. комментарии Евгений Шатунов и Василий Дёмин).