c语言 - 如何更改我的程序以结束连续打印的错误响应?

Question

我编写石头剪刀布程序，除了一件事之外，一切都可以正常工作。在 else 条件的行中，包含 while 循环的那个条件与 finishbreak;命令(choice != 1 || choice != 2 || choice != 3)，它仅在输入像 4 这样的 incorect 输入一次时才有效，但第二次无效。

当第一次输入不正确的输入时，程序将再次要求输入，但如果您输入另一个不正确的输入，它只会继续打印相同的 printf 行

printf("Please choose 1 for ROCK, 2 for PAPER or 3 for SCISSORS.n");然后继续继续打印

Computer choice is ROCK. Please choose 1 for ROCK, 2 for PAPER or 3 for SCISSORS.

while (counter != 10) {
for (i=0;i<1;i++) {
/* Generate a random number and restrict it to the range 1 to 3 */
randNum = rand()%3+1;
/*printf("A random number between 1 and 3: %dn", randNum);*/
}
printf("Please choose 1 for ROCK, 2 for PAPER or 3 for SCISSORS.n");
scanf("%d", &choice);

/* User picks an option */

if ( choice == 1 ){
printf("User choice is ROCK.n");
}
else if ( choice == 2 ){
printf("User choice is PAPER.n");
}
else if ( choice == 3 ){
printf("User choice is SCISSORS.n");
}  
else {
while (choice != 1 || choice != 2 || choice != 3) {

printf("Please choose 1 for ROCK, 2 for PAPER or 3 for SCISSORS.n");
scanf("%d", &choice);
break;
}
}

两次输入错误输入时的输出

Please choose 1 for ROCK, 2 for PAPER or 3 for SCISSORS.
4
Please choose 1 for ROCK, 2 for PAPER or 3 for SCISSORS.
4
Computer choice is ROCK.
Please choose 1 for ROCK, 2 for PAPER or 3 for SCISSORS.

我已经尝试使用break命令，但即使这样也不能解决问题。我必须在程序中更改什么才能解决此问题？

Answer 1

条件choice != 1 || choice != 2 || choice != 3将始终为真。例如(正如您已经注意到的)，如果choice为 4，则它不等于1，因此第一个测试将为真，因此(当您将测试与逻辑OR组合时)，整个条件也将为真。

但是，当choice为 1 时，条件也将为真。因为，虽然第一个测试(choice != 1)将是假的，但第二个测试(choice != 2)现在将是真的(因为1不等于2)。此外，不可能所有三个测试(或者实际上，其中任何两个)都是假的。

你想要的，而不是OR(||)是一个逻辑AND(&&)。此外，该循环中的break;语句将确保循环只运行一次(它不会破坏外部while循环)。

//...
else {
while (choice != 1 && choice != 2 && choice != 3) {

printf("Please choose 1 for ROCK, 2 for PAPER or 3 for SCISSORS.n");
scanf("%d", &choice);
//  break; // This will end the `while` on the first loop, for ANY input!
}
//...

这样，while循环将运行，直到三个测试中的任何一个计算结果为false(即choice是否是1、2或3中的任何一个)。

---

注意：上面并没有完全解释你的示例输入会发生什么(尽管我猜测外部while循环结束和你没有显示的代码然后显示计算机的选择);为此，我们需要看到更多的代码，以便它是一个最小的、可编译的、可重现的示例。

Answer 2

我建议稍微修改一下逻辑以避免调用 scanf 两次。 你可以这样做，如下所示：

choice = 0;
while (choice != 1 || choice != 2 || choice != 3) 
printf("Please choose 1 for ROCK, 2 for PAPER or 3 for SCISSORS.n");
scanf("%d", &choice);

if ( choice == 1 ){
printf("User choice is ROCK.n");
}
else if ( choice == 2 ){
printf("User choice is PAPER.n");
}
else if ( choice == 3 ){
printf("User choice is SCISSORS.n");
}  
else {
printf("Invalid choice.")
}
}