c - 如何检查 fscanf() 在 C 中返回有效字符串？

Question

我有一个 C/fscanf() 问题。完全免责声明:我是 CS 学生，正在完成一项作业。我的代码有效，但评分者将使用 GCC“所有错误和警告”选项编译我们提交的内容:

gcc -Wall yourCodeHere.c

我的代码可以运行，但我收到了一条警告，这让我很烦……并可能导致评分器出现问题。我的代码很简单，它将文件的每一行扫描成一个字符串，然后将该字符串丢给一个函数:

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

void printString(char* string){
    printf("file line is:  \"%s\"\n", string);
}

int main(int argc, char *argv[]){

    char *myFile = argv[1];
    FILE *targetFile;
    targetFile = fopen(myFile, "r");

    if (targetFile == NULL) {
        // some problem with the input file
        return -1;
    } else {
        char* string;
        while (fscanf(targetFile, "%s\n", string) != EOF){
            printString(string);
        }
    }
    fclose(targetFile);
    return 1;
}

警告是:

$ gcc -Wall myCode.c
myCode.c: In function ‘main’:
myCode.c:21:4: warning: ‘string’ may be used uninitialized in this function [-Wmaybe-uninitialized]
    printString(string);
    ^
$

我明白了编译器想要表达的意思:“如果‘string’没有保存有效数据怎么办？”这是一个有效的观点；我有点假设输入文件的每一行都会产生一个工作字符串。

那么:如何检查这个并摆脱那个恼人的警告？我注意到 fscanf() 返回成功扫描的项目数，所以我尝试了这个:

    int num = 1;    // initialize to something >0
    while (num = (fscanf(targetFile, "%s\n", string) != EOF) > 0){
        printString(string);
    }

但这产生了两个警告:

$ gcc -Wall myCode.c
myCode.c: In function ‘main’:
myCode.c:21:3: warning: suggest parentheses around assignment used as truth value [-Wparentheses]
   while (num = (fscanf(targetFile, "%s\n", string) != EOF) > 0){
   ^
myCode.c:22:4: warning: ‘string’ may be used uninitialized in this function [-Wmaybe-uninitialized]
    printString(string);
    ^
$

另外，我担心如果 fscanf() 出于某种合理原因返回 0，这会导致程序过早停止读取文件。

所以...不确定如何解决这个问题。在 Java 中，我会简单地说“if(string != NULL) ...”然后继续。但是你不能在 C 中这样做。在我调用外部函数之前，必须有一些快速的方法来检查 fscanf() 是否获取了有效的字符串。

有人知道解决方法吗？

谢谢!-皮特

PS - 如果这是一个 GCC 问题而不是 C 问题，我深表歉意。 :(

Answer 1

How to check that fscanf() returns valid string in C?

1) 确保提供空间来保存数据 2) 限制读取的数据量 3) 测试输入函数的结果。

    // #1 Provide space  `char* string` is simple a pointer with an uninitialized value
    // char* string;
    // Select a reasonable upper bound: recommend 2x expected max size
    char string[100];

    // #2 Limit width to reading up to 99 characters, 
    // 1 less than buffer size as `fscanf()` will append a null character
    // while (fscanf(targetFile, "%s\n", string) != EOF){
    // while (fscanf(targetFile, "%99s\n", string) != EOF){

    // #3 Check against the desired success value, 
    //   do not check against one of the undesired values
    while (fscanf(targetFile, "%99s\n", string) == 1) {