1

Выполняю простейший базовый пример из учебника "Программирование на С для новичков" МакГрата:

#include <stdio.h>

int main() {
    char letter;
    int num1, num2;
    printf("Enter any one keyboard character: ");
    scanf("%c",letter);
    printf("Enter two integers separated by a space: ");
    scanf("%d %d", num1, num2);
    printf("Numbers input: %d and %d \n", num1, num2);
    printf("Letter input: %c", letter);
    printf("Stored at: %p \n", &letter);
    return 0;
}

При компиляции проблем нет (использую Tiny C Compiler под Windows 8.1). При запуске исполняемого файла получаю первый запрос на ввод одиночного символа. Ввожу символ и после нажатия на Enter получаю стандартное окно аварийного прекращения программы без указания причин (или хотя бы каких намеков) проблемы.

При компиляции GCC под Cygwin происходит подобное, правда без окна аварийного завершения программы - она просто отваливается.

cygdrive/y/Programming/C>$ gcc temp.c -o temp.exe

cygdrive/y/Programming/C>$ ./temp.exe
Enter any one keyboard character: f

cygdrive/y/Programming/C>$

Собственно кроме мыслей о том, что что-то не так с самой функцией scanf при работе с виндовым стандартным вводом, в голову не приходит.

UPD: попробовал с другими форматами и типами переменных - то же самое - вылет с ошибкой.

12
  • Нормальный (современный) gcc должен писать warning, что-то вроде -- t.c:13:9: warning: format ‘%d’ expects argument of type ‘int *’, but argument 2 has type ‘int’ [-Wformat=]
    – avp
    19 мар 2017 в 16:54
  • cygdrive/y/Programming/C>$ gcc --version gcc (GCC) 5.4.0
    – cauf
    19 мар 2017 в 17:00
  • Странно, у меня gcc.real (Ubuntu 5.4.0-6ubuntu1~16.04.4) показывает (насколько помню, в 4-ке уже показывало)
    – avp
    19 мар 2017 в 17:05
  • Кстати, программа (если добавить нужные &) в Linux нормально работает (в "Letter input: %c" в конце \n, похоже, тоже пропущен)
    – avp
    19 мар 2017 в 17:13
  • Ключевое слово "в Linux". В моем окружении поведение программы отличается. См комментарии к первому ответу
    – cauf
    19 мар 2017 в 17:39

2 ответа 2

1
scanf("%c",letter);
scanf("%d %d", num1, num2);

следует вызывать как

scanf("%c",&letter);
scanf("%d %d", &num1, &num2);

Нужно указывать адрес в памяти, куда записываются введенные данные

8
  • Текст программы построчно (да что уж там, посивольно) скопирован из книги. То есть ошибка явно в самой книге?
    – cauf
    19 мар 2017 в 16:53
  • Совет помог, но теперь работает только первый вызов scanf. Второй не выводится, отрабатывая только printf
    – cauf
    19 мар 2017 в 16:59
  • Вот, смотрите сами - все работает: ideone.com/zuNLbx Что до ошибок в книге... да, в книге нет & перед letter, но перед num1 и num2 определенно есть. Что в оригинале книги у автора - я не знаю, может, ошибка автора, может, при переводе потеряли...
    – Harry
    19 мар 2017 в 17:18
  • Я рад, что у вас все получилось с онлайн-компилятором. А вот у меня ваш же код на моих компиляторах (обоих) отрабатывает иначе
    – cauf
    19 мар 2017 в 17:38
  • Вы точно вводите тот же код, что приведен в ideone? Вводите просто один символ и после него Enter? Ну чем я могу в данном случае помочь? Проверяйте у себя, попробуйте пройти отладчиком... Вряд ли вы дадите мне зайти к вам на машину и посмотреть, что там и как :) Этот код работает с gcc (в ideone), с Open Watcom, с Visual C++ 2015 (эти у меня на машине). Что творится у вас - я же не вижу.
    – Harry
    19 мар 2017 в 18:10
1

Обобщу то, что удалось выяснить в обсуждениях с товарищами @Harry и @avp.

В исходном коде, опубликованном в книге, действительно имеется ошибка. В качестве второго аргумента функции scanf() передается не переменная, а указатель на нее. То есть корректный код должен выглядеть следующим образом:

#include <stdio.h>

int main() {
    char letter;
    int num1, num2;
    printf("Enter any one keyboard character: ");
    scanf("%c", &letter);
    printf("Enter two integers separated by a space: ");
    scanf("%d %d", &num1, &num2);
    printf("Numbers input: %d and %d \n", num1, num2);
    printf("Letter input: %c", letter);
    printf("Stored at: %p \n", &letter);
    return 0;
}

Дальнейшие проблемы с работой функции scanf() (пропуск второго вызова), скорее всего, были вызваны работой антивируса. Проблема исчезла после добавления папки с исходниками в список исключения антивируса. Если это не помогает, то возможным решением может быть функция сброса буфера файла/потока fflush(), а именно в таком виде fflush(stdin); вызванная перед вторым вызовом scanf():

#include <stdio.h>

int main() {
    char letter;
    int num1, num2;
    printf("Enter any one keyboard character: ");
    scanf("%c", &letter);
    printf("Enter two integers separated by a space: ");
    fflush(stdin);
    scanf("%d %d", &num1, &num2);
    printf("Numbers input: %d and %d \n", num1, num2);
    printf("Letter input: %c", letter);
    printf("Stored at: %p \n", &letter);
    return 0;
}
4
  • В качестве дополнительного замечания - пробел в формате scanf("%d %d" является "лишним". Это не ошибка, но и смысла в нем никакого нет. Поведение формата %d уже само по себе включает в себя функциональность "пробела" (т.е. пропуск ведущих пробелов), поэтому явно запрашивать пропуск пробелов перед %d необходимости нет. Явный пробел в формате scanf может иметь смысл только перед спецификаторами %c, %[] и %n. Это, наверное, все. Уже сам факт использования пробела в этом месте говорит о том, что книгу писали методом копи-пейстинга. Отсюда и лишний пробел, и потерянные &... 20 мар 2017 в 16:26
  • fflush(stdin) - классика криво-кодинга. Стандартная билиотека С не разрешает применять fflush ко входным потокам (да, я знаю, что "POSIX разрешил", но в книге этому не место). 20 мар 2017 в 16:28
  • fflush в книге и нет. Это то, что я нагуглил в качестве средства против пропуска второго scanf. Да, костыль, но не-костыли не работали. Упомянул потому, что оно работало, когда остальное уже перепробовал и не помогло.
    – cauf
    20 мар 2017 в 16:45
  • Это популярный "студенческий костыль" для решения этой прооблемы. Его вы и нагулили. В рамках же настоящего языка С: см. c-faq.com/stdio/stdinflush.html и c-faq.com/stdio/stdinflush2.html . Это не так компактно, конечно, но тем не менее. 20 мар 2017 в 16:47

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge that you have read and understand our privacy policy and code of conduct.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.