0

Имеется такой код

Никак не могу понять, в чем проблема. Пытался сделать цикл диалога с пользователем, при выборе пункта 1 - ничего, при 2 - вечный цикл. Винил внутренние функции, но они проверены, исправны на 100%. В чем может быть проблема?

void text::user_iteration()
    {
        int x;
        for( ;; )
        {
            std::cout << "--Menu--\n"
                << "Push the number to choose the option.\n"
                << "1. Add a string or few strings.\n"
                << "2. Check current text.\n"
                << "3. Exit.\n";
            if( std::cin >> x )
            {
                if( x == 1 )
                {
                    std::cout << "Enter your text please: \n";
                    text::add_string();
                    continue;
                }
                if( x == 2 )
                {
                    std::cout << "The current text is: \n";
                    text::output_text();
                    continue;
                }
                if( x == 3 )
                {
                    break;
                }
            }
        }
    }
15
  • 1
    Единственное, что работает верно, это 3.
    – Freddy
    8 дек 2016 в 22:54
  • 5
    А отладчик вы зачем не используете? 8 дек 2016 в 23:01
  • Хороший вопрос)
    – Freddy
    8 дек 2016 в 23:02
  • Либо я туплю, либо ничего путного от отладчика не получаю
    – Freddy
    8 дек 2016 в 23:05
  • Ответ на вопрос про или - однозначно ДА. Учитесь пользоваться отладчиком, без него не стоит и пытаться программировать. 8 дек 2016 в 23:07

2 ответа 2

3

Почему не case используешь?

#include <iostream>

int main()
{
    int x;
    bool looping = true;
    char text[80];

    for( ;; )
    {
        if (!looping) break;
        std::cout << "--Menu--\n"
            << "Push the number to choose the option.\n"
            << "1. Add a string or few strings.\n"
            << "2. Check current text.\n"
            << "3. Exit.\n";
        std::cin >> x;
        switch(x)
        {
            case  1:
            {
                std::cout << "Enter your text please: \n";
                std::cin >> text;
                break;
            }
            case 2:
            {
                std::cout << "The current text is: \n";
                std::cout << text << std::endl;
                break;
            }
            case 3:
            {
                looping = false;
                break;
            }
        }
    }
}
5
  • Я только что обнаружил проблему. вызов работает успешно, если не использовать этот длинный std::cout << с записями по меню. Я вообще в тупике. Даже вообразить не могу, почему так.
    – Freddy
    9 дек 2016 в 1:13
  • и case использовал, просто выложил свой последний вариант. 4 часа уже бьюсь с этим и понять не могу, в чем проблема.
    – Freddy
    9 дек 2016 в 1:14
  • самое смешное, что без std::cout все работает
    – Freddy
    9 дек 2016 в 1:14
  • Делайте отступ на 4 символа - и будет всё выглядеть кодом (строчка int main), либо выделите текст и нажмите иконку "код"
    – nick_n_a
    9 дек 2016 в 7:13
  • а зачем for?? когда можно было do while??
    – Alex.B
    9 дек 2016 в 7:48
2

Поскольку вот такой код работает корректно:

int main(int argc, const char * argv[])
{

    int x;
    for( ;; )
    {
        std::cout << "--Menu--\n"
            << "Push the number to choose the option.\n"
            << "1. Add a string or few strings.\n"
            << "2. Check current text.\n"
            << "3. Exit.\n";
        if( std::cin >> x )
        {
            if( x == 1 )
            {
                std::cout << "Enter your text please: \n";
                cout << "text::add_string()" << endl;
                continue;
            }
            if( x == 2 )
            {
                std::cout << "The current text is: \n";
                cout << "text::output_text();" << endl;
                continue;
            }
            if( x == 3 )
            {
                break;
            }
        }
    }
}

остается сделать вывод, что проблема в ваших функциях добавления строки и вывода. Без их текста говорить о чем-либо совершенно бессмысленно.

Как теперь принято говорить, вангую остатки в буфере ввода после чтения чисел.

Скорее всего у вас в add_string не сбрасывается буфер cin после ввода значения x; в буфере лежит \n, и вы получаете совсем не то, что хотели. А при запросе меню x считывается строка, а не число, и уходит в бесконечный цикл - вы же ошибки ввода не отрабатываете!

1
  • Вы правы, именно в этом проблема и заключалась! Спасибо!
    – Freddy
    17 дек 2016 в 22:33

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