C中的简单循环代码

Question

这个程序中的循环没有像我预期的那样工作：

int main() {
    char userresponse;
    char x;

    do {
        printf("are you human");
        scanf("%c", &userresponse);
        if (userresponse == 'y') {
            printf("welcome");
        }
        if (userresponse == 'y') {
            printf("please leave");
        }
        printf("type z to repeat");
        scanf("%c", &x);
    } while (x == 'z');

    return 0;
}

它进入“type z to repeat”，然后结束。当我尝试定义char x='z'; 时，程序第一次运行良好，但最后它循环了2 或4 次，在if 语句中显示所有printf 消息，甚至是else 消息。

我希望程序在循环底部等待输入，然后根据该输入跳出循环或继续循环。为什么不这样做？

Answer 1

1. 如果输入一个字符，就会隐式输入另一个字符：换行符，这部分会导致奇怪的行为。这可以通过将格式字符串"%c" 替换为"%c%*c" 来忽略。

2. 您检查了两次'y'。将第二个'y' 替换为'n'。

3. 不影响程序的操作，但可能会提高性能：将第二个 if 替换为 else if。如果第一个计算结果为真，这将使程序甚至不测试第二个if 子句的条件。

4. 您不会检查除y 或n 之外的其他字符。要么添加一个单独的else，或者代替我的第 3 点，将第二个 if 替换为 else¹。

---

¹ 正如@JonathanLeffler 在 cmets 中提到的那样

Answer 2

当您为userresponse 输入y 时，您可能必须在那之后点击Enter，对吧？

Enter 算作一个字符！由于它没有存储在userresponse 中，因此它保留在输入缓冲区中。从输入缓冲区读取的下一行是scanf("%c",&x);。所以你最终会在x 变量中得到一个Enter。

接下来，您描述了程序显示所有 printf 消息，甚至是 else 消息，”
您在程序中看到关键字 else ？（因为我没有）

Answer 3

但最后它会循环 2 或 4 次，显示所有 printf 消息，

有几种方法可以避免这种情况，一种是在scanf之后添加：

while((check=getchar()) != EOF && check != '\n');

最小的代码看起来像这样：

#include <stdio.h>
#include <stdlib.h>

int main(void){
    char a;
    int check,ok=0;

    do {
        printf("Give an A: ");

        if ((scanf("%c",&a)) == 1){
            while((check=getchar()) != EOF && check != '\n');

            if ((a == 'a') || (a == 'A')){
                printf("True\n");
                ok=0;
            } else {
                printf("False\n");
                ok = 1;
            }
        }else{
            printf("Error");
            exit(1);
        }

    }while (ok == 1);

    return 0;
}

但是按照您的处理方式，修复程序可能如下所示：

#include <stdio.h>
#include <stdlib.h>

int main(void){
    char userresponse;
    int check,ok=0;
    char x;

    do {
        printf("are you human:  ");

        if ((scanf("%c",&userresponse)) == 1){
            while((check=getchar()) != EOF && check != '\n');

            if ((userresponse == 'y') || (userresponse == 'Y')){
                printf("welcome\n");
                break;
            } else {
                printf("please leave\n\n");
                printf("type z to repeat");
                if(scanf("%c",&x) == 1){
                    while((check=getchar()) != EOF && check != '\n');
                    if(x=='z' || x == 'Z'){
                        ok = 1;
                    }else{
                        printf("\n");
                        printf("Wrong Input\nGoodBye\n");
                        break;
                    }
                }else{
                    break;
                }
            }
        }else{
            printf("Error");
            exit(1);
        }

    }while (ok == 1);

    return 0;
}

我在这里要解释的是，没有理由坚持使用 x=='z' 这样的条件，只需使用上面示例中的另一个变量即可。

试试看吧。

编辑：

如果您需要从代码开始，这里有一个快速解决方法：

#include <stdio.h>
#include <stdlib.h>

int main(void) {
    char userresponse;
    char x;

    do {
        printf("are you human");
        if(scanf(" %c", &userresponse) ==1){
            if (userresponse == 'y' || userresponse == 'Y') {
                printf("welcome\n");
            }
            printf("type z to repeat\n");
            if(scanf(" %c", &x) != 1){
                printf("Error");
                exit(1);
            }
        }else{
            printf("Error");
            exit(1);
        }

    } while (x == 'z' || x == 'Z');

    return 0;
}

如您所见，我尽量避免遇到您的问题。 1) 我检查 scanf 是否有错误

2) 我在 %c 前面放了一个空格以避免'\n'。

3) 我检查 y 和 Y 以及 z 和 Z。

Answer 4

这很好用：

int main()
{
  char userresponse;
  char x;

  do
  {
    printf("are you human");
    scanf(" %c", &userresponse);

    if (userresponse == 'y')
        printf("welcome\n");
    else
        printf("please leave\n");

    printf("type z to repeat");
    scanf(" %c", &x);
  }
  while (x == 'z');

  return 0;
}