c - C 中的输入验证(和循环)

Question

我正在创建一个程序来验证我输入的值是否正确，但不幸的是我做错了。 如果答案是 1 或 0，此程序需要检查输入值 10 次。否则它必须再次询问答案。输入完成后，它应该通过 printf 显示正确答案，但实际上没有。

我的猜测是“switch case”部分出了问题。帮助将不胜感激!

我的代码:

int main()

{
char a0, a1, a2, a3, a4, a5, a6, a7, a8, a9;
char c = '0';
int a = 0, OK = 0, check = 0, valid_input = 0, a_ok;

printf("Fill in a value 1 or 0.\n\n");

while (a < 10)
{
while (valid_input == 0)
{
    printf("Fill in a%d: ", a);
    a_ok = scanf("%d", &OK);
    if (a_ok != 1)
    {
        scanf("%s", &c);
    }
    else if (OK <0 | OK >1)
    {
        do
        {
            while (check == 0)
            {
            printf("Fill in a%d: ", a);
            check = check +1;
            }
        c = getchar();
        }
        while (!isdigit(c));
        ungetc(c, stdin);
    }
    else
    valid_input = 1;
}
switch (a)
    {
    case 0:
    OK = a0;
    case 1:
    OK = a1;
    case 2:
    OK = a2;
    case 3:
    OK = a3;
    case 4:
    OK = a4;
    case 5:
    OK = a5;
    case 6:
    OK = a6;
    case 7:
    OK = a7;
    case 8:
    OK = a8;
    case 9:
    OK = a9;
    }
a = a +1;
}

/*
printf("Fill in a1: ");      **<--- WANT TO REMOVE THIS PART BY LOOP**
scanf("%d", &a1);
printf("\nFill in a2: ");
scanf("%d", &a2);
printf("\nFill in a3: ");
scanf("%d", &a3);
printf("Fill in a4: ");
scanf("%d", &a4);
printf("\nFill in a5: ");
scanf("%d", &a5);
printf("Fill in a6: ");
scanf("%d", &a6);
printf("\nFill in a7: ");
scanf("%d", &a7);
printf("\nFill in a8: ");
scanf("%d", &a8);
printf("Fill in a9: ");
scanf("%d", &a9);
*/

printf("\n%d\n%d\n%d\n%d\n%d\n%d\n%d\n%d\n%d\n%d", a0,a1,a2,a3,a4,a5,a6,a7,a8,a9);

getch();
return(0);
}

Answer 1

你忘记了在你的 switch 的每个 case 中的 break 语句:

case 0:
    OK = a0;
    break;
case 1:
    OK = a1;
    break;
/* etc. */

另外，这个if语句是错误的:

if (OK <0 | OK >1)

您可能打算使用逻辑或运算符 ||，而不是按位或运算符 |。

此外，您的 aN 变量(a0、a1、... a9)未在任何地方初始化。它们的初始值未定义。

另外(我这里的连续词用完了:-P)，这个:

while (!isdigit(c));

只是看起来不对。它等同于:

while (!isdigit(c))
{
}

这意味着当 isdigit(c) == 0 时它是一个无限循环，而当 isdigit(c) != 0 时什么都不做。也许你打算改为这样写:

while (!isdigit(c)) {
    ungetc(c, stdin);
}

但即便如此，您仍会在无限循环中将同一个字符推回 stdin。

这也是错误的:

scanf("%s", &c);

c 是一个 char 变量，但您要告诉 scanf() 读取一个字符串。

不是错误，而是 10 个单独的变量，您可能应该使用数组:

char val[10];

这样代替:

a0 a1 ... a9

你将拥有:

val[0] val[1] ... val[9]

总之，这个程序极其破烂。如果您刚开始学习 C，我建议您从头开始并多加注意。