Почему scanf плохо?

основная проблема состоит в том, что scanf никогда не предназначался для контакта с вводом данных пользователем. Это предназначается, чтобы использоваться с "совершенно" отформатированными данными. Я заключил слово в кавычки "отлично", потому что это не абсолютно верно. Но это не разработано для парсинга данных, которые так же ненадежны как ввод данных пользователем. По своей природе ввод данных пользователем не предсказуем. Пользователи неправильно понимают инструкции, делает опечатки, случайно нажмите Enter, прежде чем они будут сделаны и т.д. Можно было бы обоснованно спросить, почему функция, которая не должна использоваться для чтений ввода данных пользователем от stdin. Если Вы - опытное *, отклоняют пользователя, объяснение не станет неожиданностью, но оно могло бы смутить пользователей Windows. В *отклоняют системы, очень распространено создать программы, которые работают через передачу по каналу, что означает, что Вы отправляете вывод одной программы другому путем передачи по каналу stdout из первой программы к stdin из вторых. Таким образом, можно удостовериться, что вывод и ввел, предсказуемы. Во время этих обстоятельств, scanf на самом деле работы хорошо. Но при работе с непредсказуемым входом, Вы рискуете всеми видами проблемы.

Итак, почему не там никакие простые в использовании стандартные функции для ввода данных пользователем? Можно только предположить здесь, но я предполагаю, что старый хардкор C хакеры просто думал, что существующие функции были достаточно хороши, даже при том, что они являются очень неуклюжими. Кроме того, когда Вы смотрите на типичные терминальные приложения, они очень редко читают ввод данных пользователем от stdin. Чаще всего Вы передаете весь ввод данных пользователем как параметры командной строки. Несомненно, существуют исключения, но для большинства приложений, ввод данных пользователем является очень незначительной вещью.

Поэтому, что можно сделать?

Мой фаворит fgets в сочетании с sscanf. Я однажды записал ответ об этом, но я повторно отправлю полный код. Вот пример с достойным (но не прекрасен) проверка ошибок и парсинг. Это достаточно хорошо для отладки целей.

Примечание

мне особенно не нравится просить, чтобы пользователь ввел две разных вещи на одной одной строке. Я только делаю это, когда они принадлежат друг другу естественным способом. Как, например printf("Enter the price in the format <dollars>.<cent>: ") и затем используют sscanf(buffer "%d.%d", &dollar, &cent). Я никогда не делал бы чего-то как [1 112]. Основной момент использования fgets ниже должен инкапсулировать исходные данные, чтобы гарантировать, что один вход не влияет на следующее.

#define bsize 100

void error_function(const char *buffer, int no_conversions) {
        fprintf(stderr, "An error occurred. You entered:\n%s\n", buffer);
        fprintf(stderr, "%d successful conversions", no_conversions);
        exit(EXIT_FAILURE);
}

char c, buffer[bsize];
int x,y;
float f, g;
int r;

printf("Enter two integers: ");
fflush(stdout); // Make sure that the printf is executed before reading
if(! fgets(buffer, bsize, stdin)) error_function(buffer, 0);
if((r = sscanf(buffer, "%d%d", &x, &y)) != 2) error_function(buffer, r);

// Unless the input buffer was to small we can be sure that stdin is empty
// when we come here.
printf("Enter two floats: ");
fflush(stdout);
if(! fgets(buffer, bsize, stdin)) error_function(buffer, 0);
if((r = sscanf(buffer, "%d%d", &x, &y)) != 2) error_function(buffer, r);

// Reading single characters can be especially tricky if the input buffer
// is not emptied before. But since we're using fgets, we're safe.
printf("Enter a char: ");
fflush(stdout);
if(! fgets(buffer, bsize, stdin)) error_function(buffer, 0);
if((r = sscanf(buffer, "%c", &c)) != 1) error_function(buffer, r);

printf("You entered %d %d %f %c\n", x, y, f, c);

Выполнение как это устранит типичную проблему, которая является запаздывающей новой строкой, которая может смешать с входом вложенного множества. Но это имеет другую проблему, которая является, если строка - дольше, чем [1 114]. Можно проверить это с [1 115]. Если Вы хотите удалить новую строку, можно сделать это с [1 116].

В целом, я советовал бы, чтобы не ожидать, что пользователь введет вход в некоторый странный формат, который необходимо проанализировать к различным переменным. Если Вы хотите присвоить переменные height и width, не просите обоих одновременно. Позвольте пользователю нажимать Enter между ними. Кроме того, этот подход является очень естественным в одном смысле. Вы никогда не будете получать вход от [1 119], пока Вы не совершите нападки, входят, итак, почему не всегда читает целую строку? Конечно, это может все еще привести к проблемам, если строка длиннее, чем буфер. Я не забывал упоминать, что ввод данных пользователем является неуклюжим в C? :)

Для предотвращения проблем со строками дольше, чем буфер можно использовать функцию, которая автоматически выделяет буфер соответствующего размера, можно использовать getline(). Недостаток состоит в том, что Вам будет нужно к [1 121] результат впоследствии.

Усиление игры

, Если бы Вы серьезно относитесь к созданию программ в C с вводом данных пользователем, я рекомендовал бы взглянуть на библиотеку как [1 122]. Поскольку затем Вы, вероятно, также хотите создать приложения с некоторой терминальной графикой. К сожалению, Вы потеряете некоторую мобильность, если Вы сделаете это, но она дает Вам намного лучший контроль ввода данных пользователем. Например, это дает Вам способность считать нажатие клавиши немедленно вместо того, чтобы ожидать пользователя для нажатия Enter.

Вот пример использования flex для сканирования простого входа, в этом случае файл чисел с плавающей точкой ASCII, которые могли бы быть или в США (n,nnn.dd) или в европейце (n.nnn,dd) форматы. Это просто копируется с намного большей программы, таким образом, могут быть некоторые некорректные ссылки:

/* This scanner reads a file of numbers, expecting one number per line.  It  */
/* allows for the use of European-style comma as decimal point.              */

%{
  #include <stdlib.h>
  #include <stdio.h>
  #include <string.h>
  #ifdef WINDOWS
    #include <io.h>
  #endif
  #include "Point.h"

  #define YY_NO_UNPUT
  #define YY_DECL int f_lex (double *val)

  double atofEuro (char *);
%}

%option prefix="f_"
%option nounput
%option noinput

EURONUM [-+]?[0-9]*[,]?[0-9]+([eE][+-]?[0-9]+)?
NUMBER  [-+]?[0-9]*[\.]?[0-9]+([eE][+-]?[0-9]+)?
WS      [ \t\x0d]

%%

[!@#%&*/].*\n

^{WS}*{EURONUM}{WS}*  { *val = atofEuro (yytext); return (1); }
^{WS}*{NUMBER}{WS}*   { *val = atof (yytext); return (1); }

[\n]
.


%%

/*------------------------------------------------------------------------*/

int scan_f (FILE *in, double *vals, int max)
{
  double *val;
  int npts, rc;

  f_in = in;
  val  = vals;
  npts = 0;
  while (npts < max)
  {
    rc = f_lex (val);

    if (rc == 0)
      break;
    npts++;
    val++;
  }

  return (npts);
}

/*------------------------------------------------------------------------*/

int f_wrap ()
{
  return (1);
}

scanf является потрясающим, когда Вы знаете , Ваш вход всегда хорошо структурируется и хорошего поведения. Иначе...

IMO, вот самые большие проблемы с scanf:

• Риск переполнения буфера - если Вы не указываете ширину поля для %s и %[ спецификаторы преобразования, Вы рискуете переполнением буфера (пытающийся считать более вход, чем буфер измерен для содержания). К сожалению, нет никакого хорошего способа указать, что как аргумент (как с printf) - Вы имеете или к hardcode это как часть спецификатора преобразования или делаете некоторые макро-интриги.

• Принимает исходные данные, которые должны быть отклоненными - Если Вы читаете вход с %d спецификатор преобразования, и Вы вводите что-то как [1 110], Вы были бы ожидать scanf отклонять тот вход, но он не делает - он успешно преобразовывает и присваивается эти 12, уезжая w4 во входном потоке, чтобы испортить следующее чтение.

Так, что необходимо использовать вместо этого?

я обычно рекомендую читать весь интерактивный вход как текст с помощью [1 114] - он позволяет Вам указывать максимальное количество символов для чтения за один раз, таким образом, можно легко предотвратить переполнение буфера:

char input[100];
if ( !fgets( input, sizeof input, stdin ) )
{
  // error reading from input stream, handle as appropriate
}
else
{
  // process input buffer
}

Одна причуда [1 115] - то, что это сохранит запаздывающую новую строку в буфере, если будет комната, таким образом, можно сделать легкую проверку, чтобы видеть, ввел ли кто-то в более входе, чем Вы ожидали:

char *newline = strchr( input, '\n' );
if ( !newline )
{
  // input longer than we expected
}

то, Как Вы имеете дело с этим, ваше дело - можно или отклонить целый вход из руки и хлебать любой остающийся вход с [1 116]:

while ( getchar() != '\n' ) 
  ; // empty loop

Или можно обработать вход, который Вы получили до сих пор, и читайте снова. Это зависит от проблемы, которую Вы пытаетесь решить.

К [1 130] маркируют вход (разделите его на основе одного или нескольких разделителей), можно использовать strtok, но остерегаться - strtok, изменяет его вход (он перезаписывает разделители со строковым разделителем), и Вы не можете сохранить его состояние (т.е. Вы не можете частично маркировать одну строку, затем начать маркировать другого, затем брать, где Вы кончили в исходной строке). Существует вариант, strtok_s, который сохраняет состояние токенизатора, но AFAIK, его реализация является дополнительной (необходимо будет проверить, что __STDC_LIB_EXT1__ определяется, чтобы видеть, доступно ли это).

char *text = "12w4";
char *chk;
long val;
long tmp = strtol( text, &chk, 10 );
if ( !isspace( *chk ) && *chk != 0 )
  // input is not a valid integer string, reject the entire input
else
  val = tmp;