Question

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

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

我得到编译器试图说的内容：＆＃34; 如果＆＃39;字符串＆＃39;没有持有效数据？＆＃34;这是一个有效的观点;我有点假设输入文件的每一行都会产生一个工作字符串。

那么：如何检查这个并摆脱那个烦人的警告？我注意到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中，我只是说＆＃34; if（string！= NULL）...＆＃34;然后继续前进。但你不能在C中做到这一点。在调用外部函数之前，必须有一些快速检查fscanf（）获取有效字符串的方法。

任何人都知道修复？

谢谢！ -Pete

PS - 如果这更像是GCC问题而不是C问题，请道歉。：（

Answer 1

代码中的

lib是指针而不是数组。没有为string分配的内存来复制字符。将其更改为字符数组将与fscanf一样工作，其中X是您要分配的字节数。请记住，您需要为＆＃39; \ 0＆＃39;终止字符串。还要记住，你不知道测试行有多大，string = malloc(X);只会复制字节而不关心长度，这会带来另一个问题 - fscanf如何知道复制一个整行，而不仅仅是一个单词或一个字符？你应该研究一下！

Answer 2

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

如何检查fscanf（）在C中返回有效字符串？

2 个答案: