Я пишу простую программу в c, и я застрял с этой ошибкой
Ошибка сегментации (сброс ядра)
Я знаю, что ошибка Segmentation fault возникает из-за нарушения доступа к памяти. Но я не могу понять, где в приведенной ниже простой программе происходит плохой доступ к памяти.
#include<stdio.h>
int main()
{
int a = 0;
scanf("%d", a);
printf("%d\n", a);
return(0);
}
Я компилирую онлайн здесь, в блоках кода также выдает ту же ошибку.
return(0) действителен, но не нужен, подойдет только return 0. 21.01.2016 int i = (5), но вы правы, return не является функцией. Моим возражением будет подпись int main() -> int main(void) 21.01.2016 int main() не означает, что main() принимает 0 параметров. Но вместо этого это означает, что main() принимает неопределенное количество параметров. Так что int main(void) действительно важно. Потому что единственными действительными подписями для main() являются int main(int argc, char *argv[]); и int main(void);. 21.01.2016 return (0) является действительным, но это не очень хороший стиль кодирования. 21.01.2016 Вы должны передать указатель на int вместо самого int. Измените его на
if (scanf("%d", &a) == 1)
printf("%d\n", a);
else
printf("Invalid input\n");
В вашем коде scanf() предполагает, что значение в int является адресом, по которому он должен писать, что приводит к неопределенному поведению. Кроме того, еще одна причина неопределенного поведения заключается в том, что a не нужно инициализировать перед вызовом scanf(), но это не имеет значения, так как в любом случае неопределенное поведение должно было произойти. Стандарт указывает, что любой параметр неожиданного типа, переданный в scanf(), вызовет неопределенное поведение.
Тот факт, что a не был инициализирован перед вызовом scanf(), означает, что если вы проигнорируете возвращаемое значение scanf() и оно завершится ошибкой, и вы попытаетесь прочитать a, как в printf() в вашем коде, ваш код вызовет неопределенное поведение. Вот почему так важно проверять возвращаемое значение.
Оператор & адреса создает указатель, содержащий адрес его операнда, scanf() нужен указатель на параметр1, чтобы можно было сохранить в нем результат.
Обратите внимание, что при передаче массива, например, массива char в scanf() строку со спецификатором "%s", вы не должны использовать оператор &, потому что имя массива преобразуется в указатель на первый элемент самого себя, что и требуется scanf() в тот случай.
1Обратите внимание, что в c, так что это единственный способ изменить параметр внутри функции scanf().
Изменять
scanf("%d", a);
to
scanf("%d", &a);
scanf нужен указатель на переменную, а не значение переменной.
Я могу предложить вам добавить опцию -Wall при компиляции. В вашем случае gcc предупредит вас:
test.c: In function ‘main’:
test.c:25:5: warning: format ‘%d’ expects argument of type ‘int *’, but argument 2 has type ‘int’ [-Wformat=]
scanf("%d", a);
^
scanf(), вы также поймете, почему я сравниваю его возвращаемое значение в1. И всегда делайте это, игнорирование возвращаемого значения может привести к очень странным ошибкам. 21.01.2016scanfкак адрес памяти (как если бы это был действительный указатель), так что результат записывается в память с адресом, который == ЗНАЧЕНИЕaвместо на память, которой владеетa. 21.01.2016