Бесконечный цикл (как избежать?)

@vortexx1 · Регистрация: 06.03.2011

Студворк — интернет-сервис помощи студентам

Программа должна принимать только одно из трёх значений переменной 'destiny'. Если вводят любое другое значение, программа должна повторять вопрос до тех пор, пока не введут значение так, как следует.

Пробовал так:

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
    do
    {
        if (destiny == 'e')     // если выбран легкий уровень сложности
        {
           capital = 10000;     // капитал = 10 000
           cout << "You have chosen easy game and your start capital is $" << capital << ".";
        }
        else if (destiny == 'm') // если выбран средний уровень сложности
        {
           capital = 7500;       // капитал = 7 500
           cout << "You have chosen middle game and your start capital is $" << capital << ".";
        }
        else if (destiny == 'h') // если выбран сложный уровень сложности
        {
           capital = 5000;       // капитал = 5 000
           cout << "You have chosen hard game and your start capital is $" << capital << ".";
        }
        else                     // если ввели левое значение
            cout << "Error. Sorry.";    // сообщение об ошибке
    } while (destiny != 'e' || destiny != 'm' || destiny != 'h');

но выдает бесконечный цикл.

Как сделать правильно?

Спасибо.

silent_1991 · 12.06.2011, 00:55

В условии цикла вместо || поставьте && везде.

@vortexx1 · 12.06.2011, 00:58 **[ТС]**

Пробовал.

Если вводить нужные значения, то работает как предусмотрено, а вот если ввести что-то левое, то бесконечно зацикливает "Error. Sorry.".

12.06.2011, 01:01

лучше использовать оператор case, чем else if мне кажется, т.е. swith

silent_1991 · 12.06.2011, 01:06

vortexx1, ну так вы ввод значения внутри цикла сделайте.

@vortexx1 · 12.06.2011, 01:13 **[ТС]**

YsWaY, если вы имели ввиду это:

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
    do
    {
         switch (destiny)
         {
            case 'e':
                 capital = 10000;
                 cout << "Easy mode. Your capital is $" << capital << ".";
                 break;
                 
            case 'm':
                 capital = 7500;   
                 cout << "Middle mode. Your capital is $" << capital << ".";
                 break;
                 
            case 'h':
                 capital = 5000;
                 cout << "Hard mode. Your capital is $" << capital << ".";
                 break;
                 
            default:
                 cout << "Error. Enter correct value, please.";
         }
    } while (destiny != 'e' && destiny != 'm' && destiny != 'h');

... то оно не помогло.

silent_1991, простите, не понимаю, что вы подразумеваете. Можете показать?
Спасибо.

silent_1991 · 12.06.2011, 01:14

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
do
    {
        cin >> destiny;
 
        if (destiny == 'e')     // если выбран легкий уровень сложности
        {
           capital = 10000;     // капитал = 10 000
           cout << "You have chosen easy game and your start capital is $" << capital << ".";
        }
        else if (destiny == 'm') // если выбран средний уровень сложности
        {
           capital = 7500;       // капитал = 7 500
           cout << "You have chosen middle game and your start capital is $" << capital << ".";
        }
        else if (destiny == 'h') // если выбран сложный уровень сложности
        {
           capital = 5000;       // капитал = 5 000
           cout << "You have chosen hard game and your start capital is $" << capital << ".";
        }
        else                     // если ввели левое значение
            cout << "Error. Sorry.";    // сообщение об ошибке
        } while (destiny != 'e' || destiny != 'm' || destiny != 'h');

@vortexx1 · 12.06.2011, 01:15 **[ТС]**

Аааа, понял.

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
    do
    {
        cin >> destiny;
        cout << endl;
            
        if (destiny == 'e')     // если выбран легкий уровень сложности
        {
           capital = 10000;     // капитал = 10 000
           cout << "You have chosen easy game and your start capital is $" << capital << ".";
        }
        else if (destiny == 'm') // если выбран средний уровень сложности
        {
           capital = 7500;       // капитал = 7 500
           cout << "You have chosen middle game and your start capital is $" << capital << ".";
        }
        else if (destiny == 'h') // если выбран сложный уровень сложности
        {
           capital = 5000;       // капитал = 5 000
           cout << "You have chosen hard game and your start capital is $" << capital << ".";
        }
        else                     // если ввели левое значение
            cout << "Error. Sorry.";    // сообщение об ошибке
        } while (destiny != 'e' || destiny != 'm' || destiny != 'h');

Работает. Спасибо большое.

@tylix · 12.06.2011, 05:56

Понимаю что уже на тему ответили, но все таки
ТС, используй лучше

switch (destiny) {
case 'h':
//.....
break;
case 'e':
//.....
break;
default:
//.....
}

будет лучше

Новые блоги и статьи Все статьи Все блоги /
SDL3 для Web (WebAssembly): Основы отладки веб-приложений на SDL3 по USB и Wi-Fi, запущенных в браузере мобильных устройств 8Observer8 07.02.2026 Содержание блога Браузер Chrome имеет средства для отладки мобильных веб-приложений по USB. В этой пошаговой инструкции ограничимся работой с консолью. Вывод в консоль - это часть процесса. . .	SDL3 для Web (WebAssembly): Обработчик клика мыши в браузере ПК и касания экрана в браузере на мобильном устройстве 8Observer8 02.02.2026 Содержание блога Для начала пошагово создадим рабочий пример для подготовки к экспериментам в браузере ПК и в браузере мобильного устройства. Потом напишем обработчик клика мыши и обработчик. . .	Философия технологии iceja 01.02.2026 На мой взгляд у человека в технических проектах остается роль генерального директора. Все остальное нейронки делают уже лучше человека. Они не могут нести предпринимательские риски, не могут. . .	SDL3 для Web (WebAssembly): Вывод текста со шрифтом TTF с помощью SDL3_ttf 8Observer8 01.02.2026 Содержание блога В этой пошаговой инструкции создадим с нуля веб-приложение, которое выводит текст в окне браузера. Запустим на Android на локальном сервере. Загрузим Release на бесплатный. . .
SDL3 для Web (WebAssembly): Сборка C/C++ проекта из консоли 8Observer8 30.01.2026 Содержание блога Если вы откроете примеры для начинающих на официальном репозитории SDL3 в папке: examples, то вы увидите, что все примеры используют следующие четыре обязательные функции, а. . .	SDL3 для Web (WebAssembly): Установка Emscripten SDK (emsdk) и CMake для сборки C и C++ приложений в Wasm 8Observer8 30.01.2026 Содержание блога Для того чтобы скачать Emscripten SDK (emsdk) необходимо сначало скачать и уставить Git: Install for Windows. Следуйте стандартной процедуре установки Git через установщик. . . .	SDL3 для Android: Подключение Box2D v3, физика и отрисовка коллайдеров 8Observer8 29.01.2026 Содержание блога Box2D - это библиотека для 2D физики для анимаций и игр. С её помощью можно определять были ли коллизии между конкретными объектами. Версия v3 была полностью переписана на Си, в. . .	Инструменты COM: Сохранение данный из VARIANT в файл и загрузка из файла в VARIANT bedvit 28.01.2026 Сохранение базовых типов COM и массивов (одномерных или двухмерных) любой вложенности (деревья) в файл, с возможностью выбора алгоритмов сжатия и шифрования. Часть библиотеки BedvitCOM Использованы. . .

silent_1991 5058 / 3118 / 271 Регистрация: 11.11.2009 Сообщений: 7,044
	12.06.2011, 00:55
	В условии цикла вместо \|\| поставьте && везде. 1

@vortexx1 6 / 6 / 3 Регистрация: 06.03.2011 Сообщений: 269
	12.06.2011, 00:58 [ТС]
	Пробовал. Если вводить нужные значения, то работает как предусмотрено, а вот если ввести что-то левое, то бесконечно зацикливает "Error. Sorry.". 0

YsWaY
	12.06.2011, 01:01
	лучше использовать оператор case, чем else if мне кажется, т.е. swith

silent_1991 5058 / 3118 / 271 Регистрация: 11.11.2009 Сообщений: 7,044
	12.06.2011, 01:06
	vortexx1, ну так вы ввод значения внутри цикла сделайте. 1

@tylix 68 / 55 / 11 Регистрация: 10.06.2011 Сообщений: 149
	12.06.2011, 05:56
	Понимаю что уже на тему ответили, но все таки ТС, используй лучше switch (destiny) { case 'h': //..... break; case 'e': //..... break; default: //..... } будет лучше 0