WedX - журнал о программировании и компьютерных науках

fgets() работает неловко

Итак, в течение долгого времени я использовал scanf(), но меня всегда беспокоило, когда ввод длиннее строки, поэтому он портит код (меняет другие целые числа и тому подобное).

Поэтому мне сказали, что вместо этого я должен использовать fgets(), потому что он может ограничить количество читаемых букв и символов.

Но fgets() всегда также включает символ новой строки (\n), поэтому мне посоветовали использовать команду input[strcspn(input, "\r\n")] = '\0';, которая должна удалить \n из строки.

И, конечно, была еще одна проблема. Когда ввод был длиннее строки, происходило одно из двух:

1. он оставляет ввод на экране и перемещает мой курсор только на одну строку ниже, что эффективно блокирует мою программу. (неправда, исправлено, правильно зациклив программу)

  1. он пропускает следующий fgets() (один или несколько).

Я где-то читал, что это может быть потому, что fgets() считывает остальную часть ввода при следующем вызове, но я пытался очищать строку каждый раз, прежде чем писать в нее снова, но это немного не помогло.

Кто-нибудь знает, как решить эту проблему? Это беспокоило меня в течение некоторого времени, так как я никогда не добивался идеального ввода в строковую программу (с отказоустойчивыми, поскольку люди делают ошибки)

#include <stdio.h>
#include <stdlib.h>
#include <string.h>

char input[3];

int main(){
    begin:
    fgets(input, 3, stdin);
    input[strcspn(input, "\r\n")] = '\0';
    if(strcmp(input, "1")==0){
        printf("input = 1\n");
    }
    else if(strcmp(input, "2")==0){
        printf("input = 2\n");
    }
    goto begin;
}

К сожалению, я не смог воссоздать 1-ю проблему, но вторая работает, набрав xx1 или xx2.

Заранее спасибо!

c fgets
22.06.2018

  • Используйте while(1) для бесконечного цикла, а не goto 22.06.2018
  • Что вы имеете в виду, что я пытался очищать строку каждый раз, прежде чем писать в нее снова? Это не меняет того, что делает fgets(). Первый вызов прочитает xx и поместит его в строку, второй вызов прочитает 1\n. Опустошение строки этого не меняет. 22.06.2018
  • Добавьте предложение else после предложения else if. else { printf("input = %s\n", input); }. Это может помочь вам понять, что происходит. 22.06.2018
  • Да, я понял это, когда это не сработало. (Бамар) 22.06.2018
  • Но fgets() также всегда включает новую строку — прочтите документацию этого еще раз, потому что это не гарантируется. Читается не более count-1 символов, останавливая (и включая) новую строку, если она встречается до достижения этого предела. Ваш код с вводом "1234\n" поместит "12" в ваш буфер и завершит строку, оставив "34\n" во входном потоке. 22.06.2018
  • Возможный дубликат Как читать ввод неизвестной длины с помощью fgets 22.06.2018

Ответы:


1

Вы должны проверить, прочитал ли fgets() всю строку, проверив, является ли последний символ новой строкой. Если это так, замените новую строку нулевым байтом; если нет, продолжайте читать символы, пока не получите новую строку.

char *c = fgets(input, 3, stdin);
if (c == NULL) { // EOF reached
    break;
}
if (input[strlen(input)-1] == '\n') {
    input[strlen(input)-1] = '\0';
} else {
    int c;
    while ((c = getchar()) != '\n' && c != EOF) {
    }
}
22.06.2018
  • Примечание для ОП (и всех, кого это волнует): будьте очень осторожны при использовании strlen(s)-1 в качестве индекса для s. Если строка пуста, результаты будут не такими, как вы хотите. fgets нужно проверить 22.06.2018
  • Спасибо, теперь код работает отлично. Единственное, что я хотел бы отметить, это то, что когда я писал, что более длинный ввод, чем строка, приведет к программной блокировке, я не осознавал, что программная блокировка была плохой на моей стороне из-за неправильного зацикливания части программы. 22.06.2018
  • Если ввод перенаправляется из файла, последняя строка в файле может не иметь новой строки. Я бы сказал, что ответ должен включать malloc и realloc 22.06.2018
  • Также будьте осторожны, чтобы не пытаться прочитать другую строку, если eof(stdin) истинно. В противном случае отсутствие новой строки в самой последней строке файла вызовет проблемы. 22.06.2018
  • @MikeHousky Я добавил проверку EOF. 22.06.2018
  • @WeatherVane Я думаю, что проверки EOF / ошибок, которую я только что добавил, должно быть достаточно. fgets() не может вернуть пустую строку в случае успеха. 22.06.2018
  • Не уверен, почему malloc()/realloc() необходим, если только вы не пытаетесь дублировать функциональность getline(), которая может возвращать строку любого размера. 22.06.2018
  • Последняя строка, не заканчивающаяся символом новой строки, уже обработана. if потерпит неудачу, поэтому он не заменяет новую строку на \0. Затем он переходит в цикл, который считывается до новой строки или EOF, который должен быть немедленно остановлен. Затем следующий fgets() вернет NULL. 22.06.2018
  • Вопрос был сформулирован не так хорошо, но в его первом предложении говорится, когда ввод длиннее строки, поэтому я сделал более ранний комментарий о распределении памяти и разместил выше возможный дублирующий комментарий, но воздержался от закрытия вопроса. 22.06.2018

    • 2

    следующий предлагаемый код::

    1. устраняет goto
    2. использует возвращаемое значение из fgets()
    3. не включает файлы заголовков, это содержимое не используется
    4. допускает тот факт, что "\n" обрабатывает любую другую последовательность "конец строки".
    5. устраняет «магические» числа

    а теперь предлагаемый код:

    #include <stdio.h>
#include <string.h>

#define BUF_LEN 3

char input[ BUF_LEN ];

int main( void )
{
    while(  fgets(input, sizeof input, stdin) )
    {
        input[strcspn(input, "\n")] = '\0';

        if(strcmp(input, "1")==0){
            printf("input = 1\n");
        }

        else if(strcmp(input, "2")==0){
            printf("input = 2\n");
        }
    }
}

    вот типичный запуск кода (помните, что цикл продолжается, даже когда ничего не печатается:

    1
input = 1
2
input = 2
3
4
5
6
1
input = 1
kj;lkj;lkj
1jklkl;j
1
input = 1
x1
xx1
input = 1
    23.06.2018
    Новые материалы
    Я хотел выучить язык программирования MVC4, но не мог выучить его раньше, потому что это выглядит сложно…
    Просто начните и учитесь самостоятельно Я хотел выучить язык программирования MVC4, но не мог выучить его раньше, потому что он кажется мне сложным, и я бросил его. Это в основном инструмент..
    Лицензии с открытым исходным кодом: руководство для разработчиков и создателей
    В динамичном мире разработки программного обеспечения открытый исходный код стал мощной парадигмой, способствующей сотрудничеству, инновациям и прогрессу, движимому сообществом. В основе..
    Объяснение документов 02: BERT
    BERT представил двухступенчатую структуру обучения: предварительное обучение и тонкая настройка. Во время предварительного обучения модель обучается на неразмеченных данных с помощью..
    Как проанализировать работу вашего классификатора?
    Не всегда просто знать, какие показатели использовать С развитием глубокого обучения все больше и больше людей учатся обучать свой первый классификатор. Но как только вы закончите..
    Работа с цепями Маркова, часть 4 (Машинное обучение)
    Нелинейные цепи Маркова с агрегатором и их приложения (arXiv) Автор : Бар Лайт Аннотация: Изучаются свойства подкласса случайных процессов, называемых дискретными нелинейными цепями Маркова..
    Crazy Laravel Livewire упростил мне создание электронной коммерции (панель администратора и API) [Часть 3]
    Как вы сегодня, ребята? В этой части мы создадим CRUD для данных о продукте. Думаю, в этой части я не буду слишком много делиться теорией, но чаще буду делиться своим кодом. Потому что..
    Использование машинного обучения и Python для классификации 1000 сезонов новичков MLB Hitter
    Чему может научиться машина, глядя на сезоны новичков 1000 игроков MLB? Это то, что исследует это приложение. В этом процессе мы будем использовать неконтролируемое обучение, чтобы..
    Для любых предложений по сайту: [email protected]