Question

C/C++中我读取一个文本，如何正确的判断此文本已经读完，用feof会多循环一次，该如何解决

while(!feof(fp))
{
if (fgets(linetext,LINE_NUM,fp) == NULL)
{
printf("fgets line error from log <%s>!\n",CONFIG);
fclose(fp);
fp = NULL;
return 2;
}
}
当文本读完后，还会执行一次fgets，然后会打印if中的内容，即fgets line error from log <%s>!，这种问题该怎么解决

Answer 1

你这样写判断是错误的。

函数feof()的功能是检测流上的文件结束符！而不是检测文件输入是否结束。

在最后一次循环中，进入循环的条件实际上是检测上一次输入的流中有没有文件结束符，很显然还没有。那么在最后一次循环中，因为文件中已经没有内容可以读入了，所以读入的内容实际上是空，所以linetext实际上是等于null的，这才是真正的文件输入结束了。所以会跳出循环。

而且我们在使用读入的内容，也就是linetext的时候，判断是这样写的：

while(linetext){
//代码
}

正是因为在最后一次循环中，读入的内容是空，即linetext等于null，才保证了我们在使用linetext的时候可以正常跳出循环。

你会这样写，实际上是理解feof这个函数的功能，而且关于文件结束符你理解的也有问题，这里，我找到一篇文章，你可以看一看，希望能够加深你的理解：http://blog.csdn.net/zhy10/article/details/1562649

如果我的回答解决了你的问题，请采纳！你的采纳是我答题的动力！

如有疑问，请追问。

Answer 2

你的逻辑错了，你应该这样写：

while(!feof(fp))
  {   
        // 改为不等于NULL ,因为当读取到空白时，则fgets函数会返回NULL。
        if (fgets(linetext,LINE_NUM,fp) != NULL)  
        {   
            //printf("fgets line error from log <%s>!\n",CONFIG);
           // TODO...
        }   
}

  fclose(fp);
  fp = NULL;
  return 2;
  
  // 另外办法解决fgets多读的解决方案：
  // 这种方案参考了微软公司的T-SQL 游标遍历的思想
  
  fgets(linetext,LINE_NUM,fp);  // 如果未到达文件末尾,则fp不是NULL
  while(!feof(fp))  // 不是NULL,则执行
  {   
        printf("%s\n",linetext);  // 打印值
        fgets(linetext,LINE_NUM,fp); // 文件指针移动到下一行
                                     // 如果下一行是末尾,则while不成立
        
        /*
        if (fgets(linetext,LINE_NUM,fp) == NULL)
        {   
            printf("fgets line error from log <%s>!\n",CONFIG);
            fclose(fp);
            fp = NULL;
            return 2;
        }   */
        
}

Answer 3

fgets(linetext,LINE_NUM,fp) ;

while(!feof(fp))
{
do_something(linetext);
fgets(linetext,LINE_NUM,fp);
}
fclose(fp);
fp = NULL;
return 2;