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);

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

解决方法

初学者完全避免使用 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 格式字符串中的空格并不完全匹配，因此使用 \n 或 \t 或 都是一样的。通常，人们只是使用一个空格来使其更易于阅读。此外，某些转换说明符（值得注意的 %d 和 %lf）之间的空格是无关紧要的，只是为了提高可读性。