当我两次运行代码（在While / DoWhile之后）时，scanf无法正常工作。 （即使用Visual C ++ Windows控制台应用程序）

Question

我是编程的新手，我试图制造一个计算器，并添加了while循环，因此，如果要重复，只需键入“ 1”，程序就会重复。 问题是如果我重复它， scanf()会中断，并且不再允许我在命令行中键入内容。 （我正在使用Visual C ++ Windows控制台应用程序）

我正在尝试使用fflush(stdin)清除键盘缓冲区，但这也不起作用。

#include "pch.h"
#define _CRT_SECURE_NO_WARNINGS
#include <stdio.h>
#include <Windows.h>


void main() {
    char v;
    int exit=1;
    while (exit == 1) {
        v = 0;
        //Read what type of calcualtion the user wants to do.
        printf("Type (+,-,*,/): ");
        fflush(stdin); 
        scanf("%c", &v);
        //system("cls");
        //show the user for 2 sec what he chose
        printf("Type you chose: %c", v);
        Sleep(2000);
        //system("cls");
        //here the calcualtion will take place. 
        switch (v) {
        case '+':
            printf("\nTBD +");
            break;
        //Here are some more cases that i have excluded.
        default:
            printf("Please only use '+,-,*,/' above\n");
            exit = 1;
            break;
        }
        printf("\n do you want to repeat (1==yes|0==no): ");
        scanf_s("%d", &v);
    }
}

该程序运行时的结果如下所示：

Type (+,-,*,/): +
Type you chose: +
TBD +
 do you want to repeat (1==yes|0==no): 1
Type (+,-,*,/): Type you chose:
Please only use '+,-,*,/' above

do you want to repeat (1==yes|0==no):

结果应如下所示：

Type (+,-,*,/): +
Type you chose: +
TBD +
 do you want to repeat (1==yes|0==no): 1
Type (+,-,*,/): +
Type you chose: +
TBD +
 do you want to repeat (1==yes|0==no): 1

Answer 1

您的代码有很多问题。 首先，您不需要Windows标头或使用scanf_s 。 fflush(stdin)导致未定义的行为。 您应该自己清除输入流。 作为scanf的替代方法，请使用fgets或fgetc并自己执行转换。 代码中的另一个问题是您要在循环开始时重置v的值。 然后你给的1到x的默认值，但检查while(x==1)你也想至少一次，不管初始条件的运行代码和do while循环是一个更好的选择while在这案件。 同样只是为了命名约定，如果要在exit == 1处继续循环，那会产生误导。 如果exit == 1 ，则应终止循环。 这是非常令人费解和混乱的代码。 让我尝试为您清理。

int main()  {
    //We only need a size of 3 , 1 for character, 1 for null terminator,1 for carriage return
    char v[32] = {0};
    int exit = 0;
    do{
        //Read what type of calcualtion the user wants to do.
        printf("Type (+,-,*,/): ");
        fgets(v, sizeof(v), stdin);
        //system("cls");
        //show the user for 2 sec what he chose
        printf("Type you chose: %c", *v);//dereference the pointer to the first character
        //system("cls");
        //here the calcualtion will take place.
        switch (*v) {//dereference the pointer to the first character
            case '+':
                printf("\nTBD +");
                break;
                //Here are some more cases that i have excluded.
            default:
                printf("Please only use '+,-,*,/' above\n");
        }
        printf("\n do you want to exit (1==yes|0==no): ");
        fgets(v, sizeof(v),stdin);
        exit = atoi(v);
    }while(exit != 1);
}

va大小为32，尽管如果仅输入1个字符，则大小为3就足够了。 主要是因为使用fgets输入单个字符将占用三个字节。 但是，由于我们在循环结束时取一个整数值，因此我们要确保缓冲区中有足够的空间。 例如，如果用户输入123，则缓冲区仍然可以使用，并且多余的字节将不会保留在流中。

Answer 2

使原始代码正常工作

• 删除对fflush(stdin)的调用，因为输入流的fflush()是未定义的。
• 更改scanf("%c", &v); 到scanf(" %c", &v); （注意转换说明符%c之前的空间），以使scanf()跳过前导空白。
• 更改scanf_s("%d", &v); 到scanf_s("%d", &exit); 。 编译器应该警告您转换说明符%d与参数&v之间的类型不匹配（ int*与char* ）。 如果没有，则应提高编译器的警告级别。

---

使用scanf()对输入进行错误检查的可能实现：

#include <stdio.h>

int main(void)
{
    char keep_running;
    do {
        double first_operand;
        while (printf("First operand: "), scanf("%lf%*[^\n]", &first_operand) != 1)
               fputs("Input error. :(\n\n", stderr);

        char op;  // operator  (not named "operator" in case a C++-compiler ever sees this file)
        while (printf("Operation: "),
               scanf(" %c%*[^\n]", &op) != 1 || (op != '+' && op != '-' && op != '*' && op != '/'))
        {
            fputs("Input error. :(\n\n", stderr);
        }

        double second_operand;
        while (printf("Second operand: "), scanf("%lf%*[^\n]", &second_operand) != 1)
               fputs("Input error. :(\n\n", stderr);

        switch (op) {
        case '+':
            printf("\n%f %c %f = %f\n\n", first_operand, op, second_operand, first_operand + second_operand);
            break;

        case '-':
            printf("\n%f %c %f = %f\n\n", first_operand, op, second_operand, first_operand - second_operand);
            break;

        case '*':
            printf("\n%f %c %f = %f\n\n", first_operand, op, second_operand, first_operand * second_operand);
            break;

        case '/':
            if(second_operand)
                printf("\n%f %c %f = %f\n\n", first_operand, op, second_operand, first_operand / second_operand);
            else fputs("\nDivision by zero is undefined. :(\n\n", stderr);
            break;
        }

        while (printf("Do you want to repeat (y/n)? "),
               scanf(" %c%*[^\n]", &keep_running) != 1 || (keep_running != 'n' && keep_running != 'y'))
        {
            fputs("Input error. :(\n\n", stderr);
        }

        puts("\n");

    } while (keep_running == 'y');
}

• 请注意，不带任何参数的函数的参数列表在C中应为void ，因此为int main(void) 。
• scanf()返回成功分配的数量。 检查返回值并处理输入错误。 永远不要信任用户。
• 转换说明符%*[^\\n]消耗所有字符，直到找到换行符并将其丢弃。 这样一来，在scanf()之后，输入缓冲区中就不会留下任何垃圾。 请注意，这将认为转换成功，然后输入垃圾有效输入。 如果要将其视为输入错误，则必须使用更复杂的方法。