别再用"while (!feof(file))"来逐行读取txt文件了！

起因

执行一个C/C++程序出现segment fault。它逐行读取文本文件，每一行是一个图片名字，然后读图、处理图像，etc。

发现最后一次读取的文件名不存在（空的）。

正确的逐行读取txt文件

这是正确的写法：

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

int main(){
    FILE* fp = fopen("/home/zz/images/face.txt", "r");

    char line[1024];

    void* ret;
    while(1) {
        memset(line, 0, sizeof(line));
        ret = fgets(line, sizeof(line), fp);
        if(ret==NULL) break;
        line[strlen(line)-1] = '\0'; // trim '\n'
        printf("line is: %s!\n", line);
    }

    fclose(fp);

    return 0;
}

使用while (!feof(file))来逐行读取txt是错误的

简单说就是，如果用while (!feof(file))，那么会比文件内容多一行。至于读取到什么，看你的buffer有没有每次做memset，如果有那么读取到空的；如果没有，读取到上一次的。

原理上，feof()操作是主动去查看文件流指针file是否到达了EOF。执行这个判断时并没有“进行下一次读取”。因此，即便已经读取完了文件最后一行，因为feof()不会把文件流指针file前移，因此得到的判断仍然是“现在不是EOF”。进而执行下一行的读取（不正确的读取）。

其实文件系统也好，并发也好，我们应该直接去读取，然后判断读取后的返回值是否为EOF或别的；而不是先去查询当前是否为EOF，然后再读取。

参考

Why is “while (!feof(file))” always wrong?