数字验证-无止境循环

Question

我有此代码来验证用户输入。条件：输入的值只能是零或正数。不接受负值和字母字符。

这是我的代码，它一直在循环：

#include <stdio.h>

int main ()
{
   int a, b, c, d;

   printf ("enter value for a:");
   do {
      b = 0;
      scanf ("%d", &a);
      if (!isdigit(a)) {
         printf("Number must be numeric!:\n");
         b++;
      }
      else if (a < 0) {
         printf ("number must be postive\n");
         b++;
      } else {
         printf("\neverything is goood\n");
      }
   } while (b != 0);
}

Answer 1

以下是我可以发现的代码的一些问题：

1. 将scanf与已经将输入解析为整数值的格式说明符一起使用。因此，无需使用isdigit检查它是否为数字。事实上，isdigit正在检查十进制数字字符，因此您的用法不正确
2. 您从不检查scanf是否返回。你应该这样做。如果出现错误（即值不是数字），则不会从流中删除输入。换言之，您将无法反复解析相同的错误值
3. 您有未使用的变量，但忘记包含某些头文件。但在你的情况下，这些都是小事（但在某些情况下可能是大事！）

也就是说，下面是一些可能适合您的代码：

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

static void skip_input(FILE *stream)
{
    int c;
    do {
        c = fgetc (stream);
    } while (c != EOF && c != '\n');
}

int main()
{
    int a;
    int r;

    printf("Please enter a value: ");

    for (;;) {
        r = scanf("%d", &a);
        if (r == EOF)
            return EXIT_FAILURE;
        if (r != 1) {
            printf("Number must be numeric!\n");
            skip_input(stdin);
        } else if (a < 0) {
            printf("Number must be postive\n");
            skip_input(stdin);
        } else {
            printf("Everything is goood\n");
            break;
        }
    }

    return EXIT_SUCCESS;
}

希望有帮助。

Answer 2

一旦用户出错，b总是

Answer 3

isdigit（）期望一个ASCII编码的字符，但是scanf带有%d参数正在将ASCII编码的数字（字符串）转换为实际数字。

也就是说，如果输入ASCII码为0x31的“1”，则scanf（“%d”）将把它转换为值。ASCII码为1不是数字。

要解决此问题，请执行以下操作之一：

• 将a键入char并使用scanf格式说明符%c，然后isdigit（）将执行您想要的操作

此外，您应该打开编译器警告并包含包含isdigit（）函数的头文件，ctype. h。