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

问题描述

对这里的 C 有点陌生！

我正在使用 CLion 编写程序，并且每当我fscanf将输入文件中的值存储到变量中时，都会不断收到此警告消息：

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

我不明白这个错误，因为我认为fscanf是我应该用来读取输入文件的函数？有人可以解释（在菜鸟级别）我使用它的方式有什么问题吗？

这是我的代码示例：

FILE *initial_configuration_box_1_pointer;
initial_configuration_box_1_pointer = fopen("initial_configuration_box_1.xyz", "r");
fscanf(initial_configuration_box_1_pointer, "%d\n", &N1); // Warning here.
fscanf(initial_configuration_box_1_pointer, "pid\tx\ty\tz\n"); // No warning here.
for (i = 1; i <= n1; i++)
{
    fscanf(initial_configuration_box_1_pointer, "p\t%lf\t%lf\t%lf\n", &rx1[i - 1], &ry1[i - 1], &rz1[i - 1]); // Warning here.
}
fclose(initial_configuration_box_1_pointer);

我知道有人问过类似的问题，但我无法理解他们得到的任何（少数）答案......

标签： cscanfclionclang-tidystrtol

初学者完全避免 scanf 有很多充分的理由。（阅读http://sekrit.de/webdocs/c/beginners-guide-away-from-scanf.html）。如果您要以交互方式使用它，切勿以空格结尾格式字符串，因为这样做只会让用户感到困惑。并始终检查 scanf 返回的值以查看它是否实际匹配任何输入。一个常见的错误是输入无法匹配预期的数据，并且 scanf 循环变成了一个无限循环，重复检查相同的无效输入。也许警告暗示了上面的第二点：由于 scanf 不会为您验证输入，因此您必须通过检查有多少转换说明符scanf能够匹配来明确地进行验证。尝试类似：

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

int
main(int argc, char ** argv)
{
        int n1;
        const char *path = argc > 1 ? argv[1] : "initial_configuration_box_1.xyz";
        FILE *ifp;
        if( (ifp = fopen(path, "r")) == NULL ){
                perror(path);
                return EXIT_FAILURE;
        } 
        if( 1 != fscanf(ifp, "%d", &n1) || n1 <= 0 ){
                fprintf(stderr, "Invalid input\n");
                return EXIT_FAILURE;
        }
        fscanf(ifp, " pid x y z");
        double rx1[n1];
        double ry1[n1];
        double rz1[n1];

        for( int i = 0; i < n1; i++ ){
                if( 3 != fscanf(ifp, " p %lf %lf %lf", rx1 + i, ry1 + i, rz1 + i) ){
                        fprintf(stderr, "Invalid input near line %d\n", i);
                        return EXIT_FAILURE;
                }               
        }
        if( fclose(ifp) ){
                perror(path);
                return EXIT_FAILURE;
        }
        return EXIT_SUCCESS;
}

请注意，scanf格式字符串中的空格不完全匹配，因此使用\nor\t或完全一样。通常，人们只使用一个空格来使其更易于阅读。此外，某些转换说明符（值得注意的%dand %lf）之间的空格是无关紧要的，只是为了可读性而包含在内。

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

问题描述

解决方案

推荐阅读