Question

我是C的新手，在这里有一个非常简单的显示文件内容的功能。它工作正常，除了我的文件的最后一行打印两次...我知道它必须做W / EOF但我无法弄清楚如何让函数识别EOF作为最后一行而不是再次运行。我知道互联网上有十亿个地方有类似的问题，但很多是C ++，因为我是新的，我认为最好只使用我自己的代码。这是代码：

 {
    int count=0, fileEnd=0;

    FILE* rockPtr=fopen("rockact.txt", "r");

    printf("\n%8s%8s%8s%8s%8s\n", "BANDID", "NAME", "SIZE", "CREW", "TRANS");

    do
    {
        fileEnd=fscanf(rockPtr, "%d%s%d%d%s", &(tempBand.rockid), tempBand.bandname, &(tempBand.bandsize), &(tempBand.crewsize), tempBand.transport);
            if (fileEnd !=EOF); //checks EOF has not been reached
            {
                printf("\n%8d%8s%8d%8d%8s", tempBand.rockid, tempBand.bandname, tempBand.bandsize, tempBand.crewsize, tempBand.transport);
                count++;                
            }               
    }
    while (fileEnd !=EOF);

    fclose(rockPtr);
    printf("\n The total amount of rock acts on file is %d\n", count);
    }

Answer 1

您的if条件不需要分号：

  if (fileEnd !=EOF); // This semicolon is wrong!

分号是一个空语句，是if的主体。

我宁愿将整个循环视为while循环：

while (fscanf(rockPtr, "%d%s%d%d%s", &tempBand.rockid, tempBand.bandname,
              &tempBand.bandsize, &tempBand.crewsize, tempBand.transport)) == 5)
{
    printf("\n%8d%8s%8d%8d%8s", tempBand.rockid, tempBand.bandname,
           tempBand.bandsize, tempBand.crewsize, tempBand.transport);
    count++;                
}

如果您想担心它，可以在循环后发现EOF，读取错误和格式错误之间的区别。请注意，检查是所有值都已转换为OK。

Answer 2

如果 - 删除它

后你有;

另外，请查看fscanf

的手册

如果发生读取错误或达到文件结尾阅读，设置正确的指标（feof或ferror）。而且，如果有的话发生之前可以成功读取任何数据，返回EOF。

这意味着您可以从文件中读取至少部分数据，达到EOF或错误，但fscanf不会返回它。

您应该使用feof函数来检查是否到达文件末尾

所以你的逻辑应该是：

从文件中读取
如果有任何内容被阅读 - 显示它，这里我的意思是你应该将返回的数字与参数的数量进行比较，而不是与EOF进行比较
检查feof

更新：在打开/从文件中读取时，您应该始终检查错误，因为EOF不是唯一的问题

EOF读完最后一行两次

2 个答案: