3

Есть маленькая прогаммка, все ничего, но столкнулся с одной странной трудностью. Приложен код простой менюшки, которая обёрнута в цикл do while и через switch отсылает пользователя в другие подменю. Принимает в работу введенные значения от 0 до 7. При вводе "0" программа закрывается. Вот в чем проблема: cin.clear() и cin.ignore() сбрасывают флаги и чистят буфер после каждого ввода, что избавляет от трудностей с некорректным вводом (select имеет тип int), заставляя повторять его, до тех пор пока не будет введено что-то вразумительное, чего программа ожидает. Но есть одно маленькое "но". При вводе чисел в cin никаких ошибок не пишется, просто перезапускается цикл, если число не подходит. Но если ввести какой-либо символ, в потоке выпадет ошибка, а после отработки cin.clear() cin.ignore() переменная select магическим(?) образом станет равна "0" и программа закроется. Можно конечно переставить эту функцию на 9, к примеру, но хочется понять тему и сделать более-менее без костылей.

do {                    // Starting <do while> cycle for selection. Wro


        cout << "Input: ";
        cin >> select;                                      // This var lives only in private sector in current class
        cin.clear();                                        //сбрасываем ошибку
        cin.ignore(numeric_limits<streamsize>::max(), '\n'); //удаляем все из буфера до перевода строки



        switch (select) {

        case 0:
            exit (0);
        case 1:
            menuVariables();
            break;
        case 2:
            menuCycles();
            break;
        case 3:
            menuArrays();
            break;
        case 4:
            menuFunctions();
            break;
        case 5:
            menuOperators();
            break;
        case 6:
            menuPointers();
            break;
        case 7:
            menuLibraries();
            break;
        case 666:
            cout << "\nBingo, you're little satan's follower. But try something in 1-7 range, please.\n\n";
            break;
        default:
            cout << "\nWrong input. Try again!\n" << endl;
        }

    }

        while (select < 1 || select > 7);

}

Не могу найти внятной документации по cin и его методам. Вернее, ту что есть - не могу осилить. То ли я тупой, то ли языком таким написано, не для новичка. Буду признателен, если подскажете решение и хоть вкратце разжуёте для танкистов. Спасибо!

2 ответа 2

5

Просто измените логику внутри цикла, добавив if-предложение.

do {                    // Starting <do while> cycle for selection. Wro


        cout << "Input: ";

        if ( not ( cin >> select ) )
        {
            cin.clear();                                        //сбрасываем ошибку
            cin.ignore(numeric_limits<streamsize>::max(), '\n'); //удаляем все из буфера до перевода строки
        }
        else
        {
            switch (select) {

            case 0:
                exit (0);
            case 1:
                menuVariables();
                break;
            case 2:
                menuCycles();
                break;
            case 3:
                menuArrays();
                break;
            case 4:
                menuFunctions();
                break;
            case 5:
                menuOperators();
                break;
            case 6:
                menuPointers();
                break;
            case 7:
                menuLibraries();
                break;
            case 666:
                cout << "\nBingo, you're little satan's follower. But try something in 1-7 range, please.\n\n";
                break;
            default:
                cout << "\nWrong input. Try again!\n" << endl;
                break;
            }
        }

    } while ( true );

}
3
  • Вау! Направление уловил. То есть если cin падает в ошибку, то по вилке идет "мыться" и возвращает к cout на том основании что не было принято ничего в поток? А если флаг ошибки не упал, перекидывает на switch? Спасибо! А такая реализация может быть использован для переполнения буфера или это вполне себе приемлемая конструкция?
    – deflore08
    20 мар 2019 в 8:15
  • @deflore08 Это обще-принятый подход. 20 мар 2019 в 8:21
  • Спасибо, буду знать. :)
    – deflore08
    20 мар 2019 в 8:28
2

Я бы вообще делал так:

typedef void (*menuFunc)();

void exitFunc() { exit(0); }
// Создаем массив функций; каждый `func[i]` представляет собой 
// вызываемую для значения i функцию. Например, для 1 - это
// menuVariables. exitFunc() сделана специально, чтобы для
// 0 вызывать exit(0).
menuFunc func[] = 
{
    exitFunc,        menuVariables,   menuCycles,     menuArrays,
    menuFunctions,   menuOperators,   menuPointers,   menuLibraries
};

...

for(unsigned int select;; сout << "OK, what is your choise now? ")
{
    // Пока не прочитано значение select или прочитанное значение
    // больше 7 - сбрасываем cin и просим новый ввод.
    while(!((cin>>select) && (select <= 7)))
    {
        cin.clear();
        cin.ignore(numeric_limits<streamsize>::max(), '\n');
        cout << "Wrong input. Try again: ";
    }
    // Если все введено верно - т.е. именно число, и это
    // число от 1 до 7 - просто вызываем соответствующую функцию.
    func[select]();
}

Не компилировал, но вроде идея понятна?

4
  • Если честно, то очень примерно, я совсем зелёный, а это выглядит как "слэнг" свободноговорящего на иностранном :)). Попробую угадать: объявляется массив со всеми пунктами меню, дальше запускается цикл for, который внутри себя запустит while, и в случае если cin скушал целое беззнаковое число не более 7, из массива вызывается нужная строка через select. Если cin подавился какой-то абракадаброй, пойдёт мыться и всё по-новой.. Не понимаю как работает menuFunc, ну это уже надо идти массивы повторять, плохо ориентируюсь. А решение выглядит очень лаконично, спасибо за идею!
    – deflore08
    20 мар 2019 в 8:37
  • См. добавленные комментарии к коду.
    – Harry
    20 мар 2019 в 9:30
  • Исчерпывающе! Спасибо за уделенное время и силы! :) тут разве нельзя отметить 2 ответа как решение? :(
    – deflore08
    20 мар 2019 в 12:26
  • Нет, можно только дать плюс (верхний треугольник), а вот принять как исчерпывающий ответ - только одно :)
    – Harry
    20 мар 2019 в 12:35

Ваш ответ

Нажимая на кнопку «Отправить ответ», вы соглашаетесь с нашими условиями использования и подтверждаете, что прочитали и поняли наши политику конфиденциальности и нормы поведения.

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