C - Do-while 后跟一个带有 int 大小写的 switch。验证不好

Question

我正在尝试在切换之前进行一段时间，以便我确定用户输入将匹配其中一种情况。但我这里有 1 个问题。

如果我给出例如输入“5a”，它仍将使用 5 来执行案例 5。我如何验证它以便用户只能给出确切的数字作为输入？

int move;
char validation[50];

do {
  printf("Input must be from 1 to 8:\n");
  fgets(validation, sizeof(validation), stdin);
  move = atoi(validation);
}
while (move < 1 || move > 8);

  switch (move) {
    case 1:
      dx = -1;
      dy = -1;
      break;
    case 2:
      dx = 0;
      dy = -1;
      break;
    case 3:
      dx = 1;
      dy = -1;
      break;
    case 4:
      dx = 1;
      dy = 0;
      break;
    case 5:
      dx = 1;
      dy = 1;
      break;
    case 6:
      dx = 0;
      dy = 1;
      break;
    case 7:
      dx = -1;
      dy = 1;
      break;
    case 8:
      dx = -1;
      dy = 0;
      break;
  }

Answer 1

您无法使用 atoi 检测 5 与 5a。我建议改用strtol()

long int strtol (const char *str, char **endptr, int base);

使用值endPtr 确保您的输入符合预期。

Answer 2

标准函数atoi 声明如下

int atoi(const char *nptr);

其实代表strtol的调用

atoi: (int)strtol(nptr, (char **)NULL, 10)

但与strtol相反，无法判断输入是否正确。

所以直接使用strtol 而不是atoi。

这是一个演示程序

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

int main(void) 
{
    const int LOW_MOVE   = 1;
    const int UPPER_MOVE = 8;

    int move;
    char validation[50];


    int valid;
    do 
    {
        printf( "Input must be from %d to %d: ", LOW_MOVE, UPPER_MOVE );
        fgets( validation, sizeof( validation ), stdin );

        char *endptr;
        move = ( int )strtol( validation, &endptr, 10 );

        valid = endptr != validation;
        while ( valid && *endptr ) valid = isspace( ( unsigned char )*endptr++ );           
    } while ( !valid || move < LOW_MOVE || move > UPPER_MOVE );

    printf( "move = %d\n", move );

    return 0;
}

它的输出可能看起来像

Input must be from 1 to 8: a
Input must be from 1 to 8: 5a
Input must be from 1 to 8: 5 a
Input must be from 1 to 8: 5 5
Input must be from 1 to 8: 5
move = 5

Answer 3

您可以确保验证文本中只有数字。这是一些修改后的代码。对于 while 循环，您可以只检查它是否真的超出范围，而不是对 8 个成功案例进行单独匹配。

int ensure_numeric(char *str) {

  while (str[0]) {
    if (str[0] >= '0' && str[0] <= '9') {
      str++;
      continue;
    }
    if (str[0] == '\r' || str[0] == '\n') {
      str++;
      continue;
    }
    break;
  }

  if (str[0]==0) return 0;

  return -1;

}

do {
  printf("Input must be from 1 to 8:\n");
  fgets(validation, sizeof(validation), stdin);
  move = atoi(validation);
  if (ensure_numeric(validation)) {
    printf("You did not enter all digits only.\n");
    move = 0;
  }
 }
 while (move < 1 || move > 8);