读取字符串并在结构中存储为 int 的最安全方法

Safest way to read a string and store as int in a struct

我在 gcc 编译器上使用 ANSI C(带 -ansi)。

我需要将用户输入的月、日、小时和分钟读入一个结构中,他们:

结构定义

typedef struct 
    {
      int month;
      int day;
      int hour;
      int minute;
    } date_time_t;

date_time_t departure_date[50];

带字符串转换的类型检查

如果他们向 scanf("%i", departure_date->month);

提供“~”,我想检查用户输入以阻止程序崩溃

所以我首先像这样读取字符串形式的值:

char temp_month[3]
char *ptr;
scanf("%s", temp_month)

然后像这样对用户输入进行类型检查:

当输入不符合条件时 -> 要求输入符合条件

 while(strtol(temp_month,  &ptr, 36) <  1 ||
        strtol(temp_month,  &ptr, 36) > 12) 
    {

  printf("Invalid selection - try again\n");
    scanf(" %s", temp_month);
  }

while 条件满足后,我将临时变量存储在结构中:

departure_date->month = atoi(temp_month);

几个问题...

  1. 这是正常的做事方式吗?请记住,我受限于只有 int 数据类型的结构。
  2. 当我在 scanf 期间向月份提交击键 'a, b, c or d' 时,它通过了我正在进行类型检查的 while 循环设定的标准,但字母表中的其他字母没有这样做 - 有谁知道为什么?
typedef int error;
#define SUCCESS 0
#define FAILURE 1

error readdate(date_time_t *dest)
{
    char line[80];
    if (fgets(line, sizeof line, stdin) == NULL)
        return FAILURE;

    if (sscanf(line, "%d %d %d %d", &(dest->month), &(dest->day), &(dest->hour), 
                       &(dest->minute)) == 4 && dest->month > 0 && dest->month < 13 
                       && dest->day > 0 && dest->day < 32 && dest->hour > -1 && 
                       dest->hour < 25 && dest->minute > 0 && dest->minute > 60)

        return SUCCESS;  /* the return value of sscanf is equal to the number 
                             of objects successfully read in and converted; also we 
                             check the bounds of the input */
    return FAILURE;
}

我们使用 fgets 然后 sscanf 而不是仅仅 scanf 以避免刷新标准输入流时出现任何问题。

scanf 函数族returns 成功读入并转换为给定数据类型的对象数。

这个函数的主要问题是它没有向调用者报告遇到的错误类型;它只表示发生了某种错误,或者没有发生错误。