修改二进制文件时无限循环

Question

我正在尝试打开一个二进制文件，从中读取一部分然后对其进行修改，但这最终会陷入无限循环。

在编写程序之前我的想法：

1. EOF 是一个函数返回的值，当找到文件结尾或发生错误时，该函数从流中读取/写入流。
2. 文件指示器将随着每次成功的读/写而向右移动它读取的字节大小。 （即如果位置指示器在 0 位置，如果在成功读/写后读/写 44 个字节，它将转到位置 44。）

下面的代码是我的许多方法之一，但没有成功：

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

struct stoixeia
{
    char e[30];
    char t[5];
    float mo;
    int ilikia;
};


int main(void)
{
    FILE *fptr;
    struct stoixeia st;
    int c;

    if((fptr = fopen("sxoleio","rb+")) == NULL)
        exit(1);

    while((c = fread(&st,sizeof(struct stoixeia),1,fptr)) != EOF)
    {
        fseek(fptr,-44,SEEK_CUR);
        for(int i = 0; i < strlen(st.e); i++)
            st.e[i] += 1;
        fwrite(&st,sizeof(struct stoixeia),1,fptr);
    }
    fclose(fptr);
    return 0;

其他不成功的方法：（feof，检查循环内的eof等）

到目前为止，我唯一的成功是使用以下代码：

    while((c = fgetc(fptr)) != EOF)
    {
        fseek(fptr,-1,SEEK_CUR);
        fread(&st,sizeof(struct stoixeia),1,fptr);
        fseek(fptr,-44,SEEK_CUR);
        for(int i = 0; i < strlen(st.e); i++)
            st.e[i] += 1;
        fwrite(&st,sizeof(struct stoixeia),1,fptr);
        fseek(fptr,1,SEEK_CUR);
        fseek(fptr,-1,SEEK_CUR);
    }
    fclose(fptr);

使用fgetc()作为while条件并在操作后将指示符向右移动一个字节然后将其向后移动一个字节似乎触发EOF并且程序以成功结束。

这种行为的原因是什么？

我真的很想了解这个行为，所以任何事情都会非常感激。

提前致谢。

Answer 1

您的代码存在一些问题，但我认为最大的误解是fread 的返回值。不应将其返回值与 EOF 进行比较。当您到达文件末尾时，fread 将返回一个简短的计数。在这种情况下，这意味着它将返回零。试试：

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

struct stoixeia {
    char e[30];
    char t[5];
    float mo;
    int ilikia;
};

static void die(const char *msg) { perror(msg); exit(EXIT_FAILURE); }
int
main(int argc, char **argv)
{
        FILE *fptr;
        struct stoixeia st;
        size_t c;
        const char *path = argc > 1 ? argv[1] : "sxoleio";

        if( (fptr = fopen(path, "rb+")) == NULL ){
                die(path);
        }
        while( (c = fread(&st, sizeof st, 1, fptr)) == 1 ){
                if( fseek(fptr, -sizeof st, SEEK_CUR) == -1 ){
                        die("fseek");
                }
                for( size_t i = 0; st.e[i] && i < sizeof st.e; i++ ){
                        st.e[i] += 1;
                }
                if( fwrite(&st, sizeof st, 1, fptr) != 1 ){
                        die("fwrite");
                };
        }
        if( fclose(fptr) ){
                die("fclose");
        }
        return EXIT_SUCCESS;
}

但是，完全跳过搜索并使用两个不同的文件句柄可能更干净。比如：

if( (rf = fopen(path, "r")) == NULL) {
        die(path);
}
if( (wf = fopen(path, "r+")) == NULL) {
        die(path);
}
while( (c = fread(&st, sizeof st, 1, rf)) == 1) {
        for( size_t i = 0; st.e[i] && i < sizeof st.e; i++ ){
                st.e[i] += 1;
        }
        if( fwrite(&st, sizeof st, 1, wf) != 1 ){
                die("fwrite");
        };
}