即使文件不为空，fgets也返回NULL [关闭]

Question

我有以下代码：

void readFile(FILE* fp)
{
    int lines = 4;
    char* line = (char*)malloc(sizeof(char)*100);
    int i;
    for(i = 0; i < lines; i++)
    {
        if(fgets(line, 100, fp)!=NULL)
        {
            printf("\n FGETS NOT NULL \n");
            printf("%s\n", line);
        }
    }
    free(line);
}

int main(int argc, char** argv)
{
    FILE* fp = fopen("t.txt", "r");
    if(fp==NULL)
    {
       printf("Error.\n");
       exit(0);
    }
    readFile(fp);
    return 0;
}

我传递给函数的文件包含：

add $s0 $s1 $s2
sub $s2 $s3 $s6
addi $s2 $s3 15
haltSim

我没有得到任何输出。出于某种原因，fgets 总是返回 NULL。任何帮助将不胜感激！

Answer 1

添加到 paulsm4 的答案，我会更进一步，避免硬编码文件名。 （不要使用幻数或硬编码文件名）

不需要malloc，你会想要阅读Do I cast the result of malloc?。而是使用固定缓冲区很好（但不要跳过）。

例如

#include <stdio.h>

#define MAXC 1024   /* don't SKIMP on buffer size */

void readFile (FILE* fp)
{
    char line[MAXC];

    while (fgets (line, MAXC, fp) != NULL)
    {
        printf("\n FGETS NOT NULL \n");
        printf("%s\n", line);
    }
}

int main (int argc, char **argv)
{
    /* read file given as argv[1] or stdin if no argument given.
     * don't hardcode filenames
     */
    FILE *fp = argc > 1 ? fopen (argv[1], "r") : stdin;

    if (!fp) {  /* validate file open for reading */
        perror ("file open failed");
        return 1;
    }

    readFile(fp);

    return 0;
}

（记住 fgets 读取并在其填充的缓冲区中包含尾随 '\n'，因此您可能希望将 printf("%s\n", line); 替换为 printf ("%s", line); 或只是 fputs (line, stdout);）

使用/输出示例

$ ./bin/fgetsnull t.txt

 FGETS NOT NULL
add $s0 $s1 $s2


 FGETS NOT NULL
sub $s2 $s3 $s6


 FGETS NOT NULL
addi $s2 $s3 15


 FGETS NOT NULL
haltSim

如果您还有其他问题，请告诉我。

Answer 2

这是一个略有不同的版本：

#include <stdio.h>

#define MAXLINES 4
#define MAXLEN 100

void readFile(FILE* fp)
{
    char line[MAXLEN];
    int n = 0;
    while ((fgets(line, MAXLEN, fp)) && n < MAXLINES) {
      printf("n: %d, %s\n", n, line);
      ++n;
    }
}

int main(int argc, char** argv)
{
    FILE* fp;
    if (argc < 2) {
      printf ("USAGE: myprog <filename>\n");
      return 1;
    }
    fp = fopen(argv[1], "r");
    if (!fp) {
      perror("File open error");
      return 1;
    }
    readFile(fp);
    fclose (fp);
    return 0;
}

我完全不确定您的原始版本有什么问题 - 如果您有兴趣，单步调试调试器可能会有所帮助。

但有几点：

1. 你不想要像 4 或 100 这样的“幻数”

2. 如果您不需要 malloc() - 那就不要。只需声明一个数组。

3. 如果您在循环中读取文件 - 最好将循环与实际 I/O 绑定（并可能添加辅助检查，如“小于 MAXLINES”）。

4. 如果可能，从 main 中“return”优于“exit()”。