读取字符串并将其存储为int的最安全方法

时间:2019-04-25 15:52:49

标签: c struct ansi c89

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

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

  • 不能是非int的数据类型,
  • 每个人都需要遵守个人标准(i.e. month > 0 && month < 13等)

结构定义

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条件后,我将临时变量存储在struct中:

departure_date->month = atoi(temp_month);

几个问题...

  1. 这是正常的做事方式吗?请记住,我仅限于该结构只有int数据类型。
  2. 当我在scanf期间向月份提交击键'a,b,c或d'时,它通过了我进行类型检查的while循环中列出的条件,但是字母表中没有其他字母可以这样做-可以有人知道为什么吗?

1 个答案:

答案 0 :(得分:1)

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函数家族返回成功读取并转换为给定数据类型的对象数。

此功能的主要问题是它不向调用者报告遇到的错误类型;它仅表示发生了某种错误,或者没有发生错误。