Использование realloc путем умножения целого числа указателя и sizeof (int) не работает

Несколько вещей:

Первый,

int *amount = 9;

не делает то же самое, что

*amount = 9;

В первом случае * указывает только на то, что amount имеет тип указателя, и мы инициализируем значение указателя (т. е. адрес) равным 9, что, скорее всего, не является допустимым значением указателя. , и попытка разыменовать его может привести к ошибке времени выполнения.

Во втором случае мы присваиваем целочисленное значение 9 объекту, на который указывает amount.

Почему это не сломалось, когда вы передали amount в printf? По сути, вы вызвали неопределенное поведение, передав аргумент неправильного типа (%d ожидает int, вы передали int *). Одним из возможных результатов неопределенного поведения является получение ожидаемого результата. По какой-то причине printf смог обработать это значение int * как int. Большинство компиляторов должны помечать это несоответствие типов, но вы можете поднять уровень предупреждения, чтобы увидеть это.

На двоичный оператор * наложено ограничение, согласно которому оба операнда имеют арифметический тип. int * не является арифметическим типом, отсюда и диагностика.

Основываясь на том, как вы на самом деле используете amount в своем коде, вы должны были объявить его не как указатель, а как обычный int:

int amount = 9;

Во-вторых, как правило, вы не хотите присваивать исходному указателю результат realloc. Если realloc не удастся, он вернет NULL и оставит исходный блок памяти как есть. Однако, если вы вернете этот NULL исходному указателю, вы потеряете доступ к этой памяти. Лучше всего присвоить результат realloc временному объекту, а затем убедиться, что временный объект действителен, прежде чем назначать его обратно оригиналу:

int *tmp = realloc( array, amount * sizeof *array );
if ( tmp )
{
  array = tmp;
}
else
{
  // handle realloc error
}

Обратите внимание на использование sizeof *array вместо sizeof (int). sizeof — это оператор, подобный унарному * или унарному +, и его операнд может быть либо именем типа в скобках, либо выражением. выражение *array имеет тип int, поэтому sizeof *array == sizeof (int). Это помогает сделать код немного легче для чтения, и если вы когда-нибудь измените тип array (скажем, на double *), вам не придется обновлять вызов realloc. Это также очень полезно при распределении типов многомерных массивов — вы бы предпочли написать

int (*arr)[10] = malloc( sizeof (int) * 10 * rows);

or

int (*arr)[10] = malloc( sizeof *arr * rows );

?

Вы просто используете тот факт, что ваша реализация допускает безопасное преобразование между указателями и целыми числами, но это:

int *amount = 9;      // only use that for memory mapped hardware registers
printf("%d", amount); //re-interpreting the pointer value as an int
array = realloc(array, amount * sizeof(int));   // hopefully you got a diagnostic

ужасно. Указатель должен быть только нулевым указателем на действительный объект. Полная остановка. А арифметика указателей имеет смысл только внутри массива.

Если переменная должна содержать целочисленные значения, то она должна быть целочисленного типа:

int amount = 9;
printf("%d", amount); //Printing
array = realloc(array, amount * sizeof(int)); //Problem is here

Если вам нужен указатель на него, просто объявите его и используйте как указатель:

int amount = 9;
int *p_amount = &amount;
printf("%d - %d\n", amount, *p_amount); //Printing
array = realloc(array, (*p_amount) * sizeof(int)); //No problem now

amount имеет тип int *. Причина, по которой printf печатает 9, не в том, что amount указывает на значение 9, а в том, что значение 9 приводится к указателю.

Теперь в вашем заявлении array = realloc(array, amount * sizeof(int));. Вы пытаетесь умножить указатель (а не значение, на которое указывает amount). Спросите себя, какой должна быть семантика этого. Вместо int *amount = 9; вы, вероятно, хотите

int *amount = calloc(1, sizeof(int));
*amount = 9;

который объявляет указатель и выделяет место для одного целого числа, а вместо array = realloc(array, amount * sizeof(int)); вы, вероятно, захотите

array = realloc(array, *amount * sizeof(int));

вы должны попытаться изучить концепцию указателей и арифметику указателей. Простое объявление указателя не резервирует место в конце.

amount следует определять как int, а не int *.

int amount = 9;