Я неправильно запрограммировал цикл

Question

Я неправильно запрограммировал цикл

0

 printf("What do you do?\n1. Walk Away.\n2. Jump.\n3. Open Door.\n\n");
 scanf("%d",&Choice);
 printf("\n\n\n");
 while(4<=Choice,Choice<=0);
 {
     printf("That is not a choice.\n");
     printf("What do you do?\n1. Walk Away.\n2. Jump.\n3. Open Door.\n\n");
     scanf("%d",&Choice);
     printf("\n\n\n");
 }

Так что это моя программа. Он работает, но то, что я хочу, это повторить до тех пор, пока не будет введен ответ 1, 2 или 3. Но независимо от того, что он ответит, он проходит цикл while, а затем продолжайте, независимо от следующего выбора. (Также я объявил "Выбор", я просто не хотел показывать всю программу.)

TheAwesomElf 27 дек. 2013, в 05:41

Источник

5

@jeffamaphone Два выражения являются взаимоисключающими; он ищет || что логично ИЛИ.
Ramchandra Apte 27 дек. 2013, в 04:25
0

О да. Это также написано неловко.
i_am_jorf 27 дек. 2013, в 04:28
0

Как ты пишешь маленькую двойную строчку?
TheAwesomElf 27 дек. 2013, в 04:34
0

@TheAwesomElf Это два символа трубы рядом друг с другом
JustSid 27 дек. 2013, в 04:35
1

@TheAwesomElf Это смещение, а затем кнопка над клавишей ввода, которая также имеет обратную косую черту
takendarkk 27 дек. 2013, в 04:36
1

Зачем использовать printf scanf тегом C ++?
Ed Heal 27 дек. 2013, в 04:36

Показать ещё 4 комментария

Теги:

c++

3 ответа

Ещё вопросы

@jeffamaphone Два выражения являются взаимоисключающими; он ищет || что логично ИЛИ.
Как ты пишешь маленькую двойную строчку?
@TheAwesomElf Это два символа трубы рядом друг с другом
@TheAwesomElf Это смещение, а затем кнопка над клавишей ввода, которая также имеет обратную косую черту
Зачем использовать printf scanf тегом C ++?

ApproachingDarknessFish · Answer 1 · 2013-12-27T02-17-00.000Z

В вашем коде есть две проблемы. Ваше выражение while-loop неверно. Запятая не делает то, что, по вашему мнению, делает: в C/C++ запятая выполняет левое выражение и оценивает правое выражение, что означает, что в вашем случае вы проверяете только второе условие. Вы, вероятно, хотите:

 while(4<=Choice || Choice<=0)

|| является оператором OR, который возвращает true, если любое из выражений вокруг него истинно.

Во-вторых, в конце цикла while есть неуместная точка с запятой:

while(4<=Choice,Choice<=0); //<-- this should not be here

Это означает конец цикла, что означает, что ваш код анализируется как:

while(4<=Choice,Choice<=0); //loop body is empty

{
    //and we have a random unnamed block following it
}

Удалите точку с запятой, и ваш цикл while должен выполняться правильно.

Jonathan Leffler · Answer 2 · 2013-12-27T02-16-00.000Z

C и C++ имеют запятую, которая имеет наименьший приоритет для всех операторов. Он оценивает левый операнд и отбрасывает результат, а затем оценивает правый операнд. Таким образом, ваше while условие эквивалентно:

while (Choice <= 0)

У вас также есть ошибка, потому что сразу после условия есть точка с запятой, которая делает бесконечный цикл, если Choice не является строго положительным (поскольку ничто в цикле не изменяет значение Choice).

То, что вы, вероятно, намеревались написать, было:

while (Choice >= 4 || Choice <= 0)
{
    ...
}

Mark Ransom · Answer 3 · 2013-12-27T02-05-00.000Z

Оператор запятой , не проверить оба условия, он просто возвращает второй из двух. Таким образом, ваша в while петля является эквивалентом:

while(Choice<=0) ;

и поскольку там a ; после утверждения это фактически бесконечный цикл, если условие выполнено. Хорошо, что вы не вошли в выбор -1.