CLion 建议使用“strtol”而不是“scanf”

Question

#include <stdio.h>

int main(int argc, char** argv) {
    int num = 0;

    printf("Input: ");
    scanf("%d", &num); <<<

    printf("%d\n", num);

    return 0;
}

scanf("%d", &num);

Clang-Tidy：'scanf' 用于将字符串转换为整数值，但函数不会报告转换错误；考虑改用“strtol”

---

我用 CLion 编写了一个非常简单的代码，它建议我使用 'strtol' 而不是 'scanf'。

但我只使用整数变量并且没有字符串。我不知道为什么会弹出检查消息。

如何修改此代码？

Answer 1

如何修改此代码？

#include <stdio.h>
#include <stdlib.h>
#include <errno.h>
#include <limits.h>

enum { INPUT_SIZE = 30 };

int main () {
    char *ptr;
    long ret;
    char str[INPUT_SIZE];

    fgets(str, INPUT_SIZE, stdin);    
    ret = strtol(str, &ptr, 10);

    if( ret == LONG_MAX || ret == LONG_MIN ) {
        perror("!! Problem is -> ");
    }
    else if (ret) {
        printf("The number is %ld\n", ret);
    }
    else {
        printf("No number found input is -> %s\n", ptr);
    }

    return(0);
}

如果成功，strtol() 返回转换后的 long int 值。

如果不成功，strtol() 返回0 如果无法进行转换 执行。如果正确的值在可表示的范围之外 值，strtol() 返回LONG_MAX 或LONG_MIN，根据符号 的价值。如果不支持 base 的值，strtol() 返回0。

如果不成功，strtol() 将 errno 设置为以下值之一：

错误代码：

EINVAL不支持base的值。

ERANGE 转换导致溢出。 来源：IBM

您可以使用scanf() 来检查溢出吗？

Input:  1234 stackoverflow
Output: The number is 1234

Input:  123nowhitespace
Output: The number is 123

Input:  number is 123
Output: No number found input is -> number is 123

Input:  between123between
Output: No number found input is -> between23between

Input:  9999999999999999999
Output: !! Problem is -> : Result too large

---

也许题外话，但Jonathan Leffler 在他的 cmets（在另一个主题中）说 将警告作为错误处理。