求助-读取文件出错,最后一行重复输出[已解决-都是feof惹的祸]

福瑞哈哥

原帖由 ivhb 于 2007-7-3 08:52 发表


while (!feof(fp)) {
  buf[0] = 0x00;
  fgets(buf, sizeof buf, fp);
  if (buf[0] == 0x00)
     break;  // ferror(fp) 或者 feof(fp)
  ...
}
if (ferror(fp)) { // 对应break出错时候蹦出来的情 ...

是的，你的程序没问题。我上午只看到了if(buf[0]==0x00)这句，没看到你上面buf[0]==0x00这句，觉得这样写程序很没有必要。fgets有返回值直接使用就可以了，逻辑越多别人越不容易理解。

MMMIX

原帖由 封神 于 2007-7-3 15:06 发表
最终选择在while中写fgets，对于我个人来说比较好理解，再次谢谢大家。






强权标准啊，根本就不好用，还要强迫大家接受，修正这么一个函数又不会死人，真不知道为什么那些家伙不改

为了向后兼容。

whyglinux

>> 都是feof惹的祸
>> 这个不算是漏洞么?

是漏洞。只不过是你写的程序中存在的漏洞，并不是 feof 函数的过错。产生漏洞的原因是你还没有掌握 feof 函数的正确使用方法，而不是什么“都是feof惹的祸”。

>> 强权标准啊，根本就不好用，还要强迫大家接受，修正这么一个函数又不会死人，真不知道为什么那些家伙不改

首先要想清楚你想要他们改什么，怎么改？

feof 函数的声明和行为已经是包括 ANSI C 标准在内的很多标准规定的函数了。即使是 C 标准委员会的人想对它进行修改也已经是不可能的事了，况且实际上也无需修改。

================

其实正确使用 feof 函数很容易，只要记住一个原则就是了：“先读文件后用 feof 判断”。你上面使用 feof 函数的程序都没有做到这一点，而是先用 feof 函数判断、然后再读文件，所以是不正确的。可根据上面的这个原则修改你的程序，使之正确。

思一克

whyglinux说的准确。漏洞是LZ的程序，而不是feof.

原帖由 whyglinux 于 2007-7-3 23:46 发表
>> 都是feof惹的祸
>> 这个不算是漏洞么?

是漏洞。只不过是你写的程序中存在的漏洞，并不是 feof 函数的过错。产生漏洞的原因是你还没有掌握 feof 函数的正确使用方法，而不是什么“都是feof惹的祸”。

...

while(1)

“先读文件后用 feof 判断”

原来如此

福瑞哈哥

每次看到有人用feof我都会感到同情！
不管feof有没有用，如果使用fgets而不判断返回值都是错误！
如果判断了fgets的返回值，也就没有必要使用feof了。

原帖由 while(1) 于 2007-7-4 09:41 发表
“先读文件后用 feof 判断”

原来如此

ivhb

原帖由 while(1) 于 2007-7-4 09:41 发表
“先读文件后用 feof 判断”

原来如此

假定进程A读取文件已经到了文件尾，feof测试为真。
如果此时进程B追加A正在读取的文件，你觉得，如果A不发生进一步的读/写动作，
接着测试feof，你觉得是真还是假？

feof仅仅是个宏，仅仅读取了上次读/写动作发生后置于FILE 结构的一个标志位。
只有进一次的读/写动作才会更新那个标志位。

思一克

应该仍然是真。因为INDICATOR的缘故
我没有具体测试。

原帖由 ivhb 于 2007-7-4 10:26 发表



假定进程A读取文件已经到了文件尾，feof测试为真。
如果此时进程B追加A正在读取的文件，你觉得，如果A不发生进一步的读/写动作，
接着测试feof，你觉得是真还是假？

feof仅仅是个宏，仅仅读取了上次 ...

while(1)

原帖由 福瑞哈哥 于 2007-7-4 09:49 发表
每次看到有人用feof我都会感到同情！
不管feof有没有用，如果使用fgets而不判断返回值都是错误！
如果判断了fgets的返回值，也就没有必要使用feof了。

fgets返回NULL的情况就表示一定是文件读完了么？

福瑞哈哥

难道还有其他选择？
fgets返回null表示这个文件对程序来说是读完了。基本上就是这样，有其他情况99%你也不会处理的。

原帖由 while(1) 于 2007-7-4 10:58 发表

fgets返回NULL的情况就表示一定是文件读完了么？