Читання рядка за допомогою scanf () не добре?

Question

Читання рядка за допомогою scanf () не добре?

scanf(" %[^\n]",line);

Мій друг запропонував, що використання fgets() для читання рядка як введення буде набагато краще, ніж використовувати scanf() , як у викладі вище. Він виправданий?

12

додано 25 червень 2013 при 01:22 Автор amulous Змінено 22 Серпень 2013 при 04:05

Перегляди: 1

джерело

ro nl ja ru fr es pt de zh hi bn ar kk uz be tr

fgets() краще. Але це більше як

scanf ("% [^ \
n]", line);

(без пробілу у форматі).

"% [^ \
n]"

OP буде використовувати основний пробіл без збереження в рядку . fgets() з збереженням провідного пробілу (тобто не \ n ).

додано 25 червень 2013 при 03:22, Автор chux, джерело

fgets() краще. Але це більше як

scanf ("% [^ \
n]", line);

(без пробілу у форматі).

"% [^ \
n]"

OP буде використовувати основний пробіл без збереження в рядку . fgets() з збереженням провідного пробілу (тобто не \ n ).

додано 25 червень 2013 при 03:22, Автор chux, джерело

Будь-який scanf з % s або

%
[

без специфікатора довжини слід вважати помилкою. Тепер, якщо ви вказали довжину, це було б цікаво питанням.

додано 30 червень 2013 при 10:23, Автор Cubbi, джерело

Будь-який scanf з % s або

%
[

без специфікатора довжини слід вважати помилкою. Тепер, якщо ви вказали довжину, це було б цікаво питанням.

додано 30 червень 2013 при 10:23, Автор Cubbi, джерело

це має бути scanf ("% [^ \ n]", рядок); закрийте фігуру в аргументі ..

додано 25 червень 2013 при 01:27, Автор Raghu Srikanth Reddy, джерело

Він абсолютно є. Я завжди пропоную використовувати fgets() і повністю уникати scanf() .

додано 25 червень 2013 при 01:27, Автор user529758, джерело

@RaghuSrikanthReddy Ідеальний приклад того, чому не слід використовувати scanf() .

додано 25 червень 2013 при 01:28, Автор user529758, джерело

@RaghuSrikanthReddy Ідеальний приклад того, чому не слід використовувати scanf() .

додано 25 червень 2013 при 01:28, Автор user529758, джерело

11 Відповіді

char * fgets ( char * str, int num, FILE * stream ); is safe to use because it avoid buffer overflow problem, it scans only num-1 number of char.

Читає символи з потоку та зберігає їх як рядок C в стрі, доки не буде прочитано символи (num-1) або досягатиметься новий рядок або кінець файлу, залежно від того, що станеться спочатку.

тут другий аргумент num - це максимальна кількість символів, які потрібно скопіювати в str (включаючи кінцевий нуль-символ).

Наприклад, припустімо, у вашому коді ємність рядка масивів - це лише символи 5 довгі, як показано нижче.

 char str[5];
 fgets (str, 5, fp);  //5 =you have provision to avoid buffer overrun

Використовуючи наведені вище коди, якщо введення з fp перевищує символи 4 , fgets() прочитає лише перший 4 Чарли потім додають \ 0 ( і відкидають інші додаткові символи введення, просто зберігає п'ять символів у str [] ).

У той час як scanf ("% [^ \ n]", str); буде прочитати, поки \ n не знайдено, а якщо довжина вводу довша, то 4 chars scanf() призведе до переповнення буферу (як scanf спробує отримати доступ до пам'яті за межами максимального індексу 4 в str [] ).

18

додано 25 червень 2013 при 01:25 Автор Grijesh Chauhan

джерело

@mulous: пан Юхао дав дуже хороший зв'язок. Читайте там, серія функцій scanf() , printf() хороша для форматованого введення/виведення

додано 25 червень 2013 при 01:44, Автор Grijesh Chauhan, джерело

Дякую за детальне пояснення. Чи буде scanf() кращим вибором, якщо я не знаю, як довго буде мій рядок введення?

додано 25 червень 2013 при 01:41, Автор amulous, джерело

char * fgets ( char * str, int num, FILE * stream ); is safe to use because it avoid buffer overflow problem, it scans only num-1 number of char.

Читає символи з потоку та зберігає їх як рядок C в стрі, доки не буде прочитано символи (num-1) або досягатиметься новий рядок або кінець файлу, залежно від того, що станеться спочатку.

тут другий аргумент num - це максимальна кількість символів, які потрібно скопіювати в str (включаючи кінцевий нуль-символ).

Наприклад, припустімо, у вашому коді ємність рядка масивів - це лише символи 5 довгі, як показано нижче.

 char str[5];
 fgets (str, 5, fp);  //5 =you have provision to avoid buffer overrun

Використовуючи наведені вище коди, якщо введення з fp перевищує символи 4 , fgets() прочитає лише перший 4 Чарли потім додають \ 0 ( і відкидають інші додаткові символи введення, просто зберігає п'ять символів у str [] ).

У той час як scanf ("% [^ \ n]", str); буде прочитати, поки \ n не знайдено, а якщо довжина вводу довша, то 4 chars scanf() призведе до переповнення буферу (як scanf спробує отримати доступ до пам'яті за межами максимального індексу 4 в str [] ).

18

додано 25 червень 2013 при 01:25 Автор Grijesh Chauhan

джерело

@mulous: пан Юхао дав дуже хороший зв'язок. Читайте там, серія функцій scanf() , printf() хороша для форматованого введення/виведення

додано 25 червень 2013 при 01:44, Автор Grijesh Chauhan, джерело

Дякую за детальне пояснення. Чи буде scanf() кращим вибором, якщо я не знаю, як довго буде мій рядок введення?

додано 25 червень 2013 при 01:41, Автор amulous, джерело

Автор Yu Hao · Answer 1 · 2013-06-25T01:28Z

C FAQ has some detailed explanation about scanf's problem:

Загалом, scanf призначений для відносно структурованого форматованого введення (його назва фактично походить від "відформатованого сканування"). Якщо ви звернете увагу, він скаже вам, чи це вдалося або не вдалося, але він може розповісти вам лише приблизно про те, де він зазнав невдачі, і взагалі не як і чому. У вас дуже мало можливостей для виправлення помилок.

Докладніше про це див тут .

Автор Yu Hao · Answer 2 · 2013-06-25T01:28Z

C FAQ has some detailed explanation about scanf's problem:

Загалом, scanf призначений для відносно структурованого форматованого введення (його назва фактично походить від "відформатованого сканування"). Якщо ви звернете увагу, він скаже вам, чи це вдалося або не вдалося, але він може розповісти вам лише приблизно про те, де він зазнав невдачі, і взагалі не як і чому. У вас дуже мало можливостей для виправлення помилок.

Докладніше про це див тут .

Автор Dayal rai · Answer 3 · 2013-06-25T01:31Z

fgets will be better than this scanf. There may be following issues with scanf as given in OP

1) переповнення буфера, як це було запропоновано @Grijesh

2) Можливо наступний scanf після цього не буде працювати, тому що новий рядок залишиться у потоці введення (якщо ви пропустите порожнє місце)

Автор Klas Lindbäck · Answer 4 · 2013-06-25T01:26Z

Простіше кажучи: так, fgets - кращий вибір.

Я подивився на ваш специфікатор формату scanf , і я був містифікований. Розуміння точно, що воно вимагає деякого часу, читаючи через сторінки man .

Крім того, ваш код scanf чутливий до перевищення буферу.

Зберігайте це простим способом, і ви зменшите витрати на технічне обслуговування та уникнути важких помилок!

Автор Klas Lindbäck · Answer 5 · 2013-06-25T01:26Z

Простіше кажучи: так, fgets - кращий вибір.

Я подивився на ваш специфікатор формату scanf , і я був містифікований. Розуміння точно, що воно вимагає деякого часу, читаючи через сторінки man .

Крім того, ваш код scanf чутливий до перевищення буферу.

Зберігайте це простим способом, і ви зменшите витрати на технічне обслуговування та уникнути важких помилок!

Автор Kranthi Kumar · Answer 6 · 2013-06-25T01:32Z

Так fgets - це кращий і безпечний спосіб прочитати рядок зі стандартного вводу.

Крім того, в коді буде більше читабельності. Подивіться на скановане твердження, надане вами.

Будь-яка друга людина, яка її бачить, буде досконало плутатися. Але в той час, як буде більше readeablity для fgets, і це легко зрозуміти.

Автор Kranthi Kumar · Answer 7 · 2013-06-25T01:32Z

Так fgets - це кращий і безпечний спосіб прочитати рядок зі стандартного вводу.

Крім того, в коді буде більше читабельності. Подивіться на скановане твердження, надане вами.

Будь-яка друга людина, яка її бачить, буде досконало плутатися. Але в той час, як буде більше readeablity для fgets, і це легко зрозуміти.

Автор mch · Answer 8 · 2014-03-08T01:17Z

Не використовуйте fgets (...), замість цього скористайтеся наступним фрагментом:

char _x[7000];
    char* y;
while ( ! feof (_f) )
{
    fscanf(_f,"%[^\n]\n",_x);
            y=x;
}

Автор mch · Answer 9 · 2014-03-08T01:17Z

Не використовуйте fgets (...), замість цього скористайтеся наступним фрагментом:

char _x[7000];
    char* y;
while ( ! feof (_f) )
{
    fscanf(_f,"%[^\n]\n",_x);
            y=x;
}