无法将数据从二进制文件读取到结构指针数组中

Question

以下代码将二进制数据写入二进制文件/从二进制文件读取二进制数据。 根据fread返回值，写入成功，即使读取也成功。 但是最后两个或三个值将始终是垃圾，我已经通过减小数组大小来验证了这一点。

如果我们在写入后立即读取文件，则所有值都可以。 但是，如果我们在写入后关闭程序一次，然后以读取的第一个动作再次运行程序，则它将最后几个值打印为垃圾。 为什么这样 ？

#include<stdio.h>
#include<stdlib.h>
#define MAX 7

void write(FILE *);
void read(FILE *);

int flag=0;

struct emp
{
char name[20];
int id;
float salary;
}*e[MAX];

void write(FILE *f)
{
int i=0,check=0;

flag=1;
for(i=0;i<MAX;i++)
{
    e[i]=malloc(sizeof(struct emp));
    if(e[i])
    {
        printf("\nEnter Name id salary\n");
        scanf("%s %d %f",e[i]->name,&e[i]->id,&e[i]->salary);
        //printf("\n----------%s %d %f\n",e[i]->name,e[i]->id,e[i]->salary);
        //fflush(stdin);
    }
    else
    {
        printf("\nError allocating Memory\n");
        exit(0);
    }
}
check= fwrite(*e,sizeof(struct emp),MAX,f);
if(check!=MAX)
{
    printf("\nerror writing to file\n");
}
else
  {
    printf("\nwritten successfully to file\n");
  }
}

void read(FILE *f)
{
int i=0;

if(flag==0) //reading file right after running program
{
    for(i=0;i<MAX;i++)
    {
        e[i]=malloc(sizeof(struct emp));
        if(e[i]==NULL)
        {
            printf("\nError Allocating Memory for read\n");
            exit(0);
        }
    }
}

if(fread(*e,sizeof(struct emp),MAX,f)==MAX)
{
    for(i=0;i<MAX;i++)
    {
        printf("\n%s %d %f\n",e[i]->name,e[i]->id,e[i]->salary);
    }
}
else
{
    printf("\nEither reading error or partial content read\n");
}
}

int main()
{
FILE *fp=NULL;
char a='a';

do
{
    printf("\nEnter w to write, r to read and e to exit\n");
    //scanf("%c",&a);
    a=getche();

    switch(a)
    {
        case 'w':
                fp=fopen("binary_ptr.exe","wb");
                if(fp==NULL)
                {
                    printf("\nError opening file to write\n");
                    exit(0);
                }
                write(fp);
                fclose(fp);
                break;
        case 'r':
                fp=fopen("binary_ptr.exe","rb");
                if(fp==NULL)
                {
                    printf("\nError opening file to read\n");
                    exit(0);
                }
                read(fp);
                fclose(fp);
                break;
        case 'e':
                exit(0);
                break;
        default:
                printf("\nInvalid input\n");
                break;
    }
}
while(1);

return 0;
}

Answer 1

如图所示，在读取/写入代码时，假定所有结构都放置在以*e开头的连续内存区域中。 不是这种情况。

该代码未定义struct数组，而是定义了指向struct的指针。 动态分配的struct本身分散在整个内存中。

就目前而言，代码通过写入/读取超出*e[0]范围的代码来调用所有MAX > 1的未定义行为。

要修复解决此环路fread() / fwrite() MAX次为一层struct的每个。

^{（例如，要指出此问题，请使用Valgrind内存检查器显式运行已编译的代码。）}

---

作为旁注，与您的问题无关： read()和write()是一个拥有函数的错误名称，因为POSIX C标准已经使用了它们。

Answer 2

在读取功能中，您有

if(fread(*e,sizeof(struct emp),MAX,f)==MAX)

这会将MAX元素读入数组e 。 但是， e不是连续数组。 您正在分别分配e

e[i]=malloc(sizeof(struct emp));

Answer 3

您不能这样做(fread(*e, sizeof(struct emp), MAX, f)因为您不连续地为emp结构分配了内存，正如正确指出的那样，您有一个指向emp结构的指针数组，而不是一组emp结构。您需要分别阅读每个emp ：

for (int i = 0; i < MAX; ++i){
    if (fread(e[i], sizeof(struct emp), 1, f) == sizeof(struct emp)){
        printf("\n%s %d %f\n", e[i]->name, e[i]->id, e[i]->salary);
    } else {
        ...
    }
}

同样，将emp数据写入文件时，也需要单独进行操作

for (int i = 0; i < MAX; ++i){
    check = fwrite(e[i], sizeof(struct emp), 1, f);
    if (check != sizeof(struct emp)){
        ...
    } else {
    ...
    }
}