如何检查 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);
    ^
$

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

那么：如何检查这个并摆脱那个烦人的警告？我注意到 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

如何检查 fscanf() 在 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) {

Answer 2

string 在您的代码中是一个指针而不是一个数组。没有为fscanf 分配内存来复制字符。将其更改为字符数组将与 string = malloc(X); 一样工作，其中 X 是您要分配的字节数。请记住，对于字符串的 '\0' 终止，您需要一个额外的字节。还要记住，你不知道测试行有多大，fscanf 只会复制字节而不关心长度，这带来了另一个问题——fscanf 应该如何知道复制整行而不是只是一个词或一个字符？你应该调查一下！