如何解决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
)之间的空格是无关紧要的,只是为了提高可读性。
版权声明:本文内容由互联网用户自发贡献,该文观点与技术仅代表作者本人。本站仅提供信息存储空间服务,不拥有所有权,不承担相关法律责任。如发现本站有涉嫌侵权/违法违规的内容, 请发送邮件至 dio@foxmail.com 举报,一经查实,本站将立刻删除。