2

我很难使用 do-while 循环，当我们到达文件末尾时它应该停止。这是循环代码：

do  {
    if (pcompanyRow[0] != '#' && pass == 1) {
        strtok(pcompanyRow, ":");
        pcompanyName = strcpy(pcompanyName, strtok(NULL, ""));
        pass = 2;
        fgets(pcompanyRow, 1024, f);
    }
    if (pcompanyRow[0] != '#' && pass == 2) {
        strtok(pcompanyRow, ":");
        pcompanySMSPrice = strcpy(pcompanySMSPrice, strtok(NULL , ""));
        pass = 3;
        fgets(pcompanyRow, 1024 , f);
    }
    if (pcompanyRow[0] != '#' && pass == 3) {
        strtok(pcompanyRow, ":");
        pcompanyMMSPrice = strcpy(pcompanyMMSPrice, strtok(NULL, ""));
        pass = 4;
        fgets(pcompanyRow, 1024, f);
    }
    if (pass == 4)  {
        AppendCompanyNode(pcompanyList, pcompanyName, pcompanySMSPrice, pcompanyMMSPrice);
        pass = 1;
    }
} while (!feof(f));

使用调试器运行后，我注意到我遇到的所有崩溃问题都是因为即使它到达整行也没有退出这个循环。

我应该如何正确写？

4

3 回答 3

8

永远不要使用 feof() 作为循环的退出指示符。feof() 仅在读取文件结尾 (EOF) 后为 TRUE，而不是在到达 EOF 时

来源在这里。它还详细解释了问题以及如何解决它。

于 2010-11-28T22:01:10.583 回答
4

我会改变你的循环和逻辑来使用它：

while (fgets(pcompanyRow, 1024, f) != NULL) {

    /* do things */

}

当 fgets() 尝试读取文件末尾之后，它将返回 NULL 并且您将跳出循环。您仍然可以继续使用pass其他标志/逻辑，但您检查的条件会略有不同。

于 2010-11-28T22:12:25.917 回答
0

我建议同时使用 fgets() 和 feof()。文件中的最后一个字符串可能有 \n 也可能没有。如果只使用 feof()，则可以跳过（丢失）最后一行。

 for(;;)
 {char *pc;

  pc=fgets(buf,sizeof(buf),fd);

  if(!pc)
    {//it may be read error or end of file
      if(feof(fd))
        {//it is end of file, normal exit from for
         break;      
        }
       else
        {//it is i/o error 
         fclose(fd);
         return 2;
        }
    }
}//for
于 2018-10-11T08:02:43.167 回答