在C中读取和写入

Question

我正在尝试制作一个程序，该程序创建并读取一个包含“结构元素”的二进制文件； 你能告诉我我做错了什么吗？ 我有错误告诉我“ s”不是函数fread（）中的指针...所以我声明了ELEM * s; 代替ELEM；

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

typedef struct element{
    char name[80];
    int p;
}ELEM;

void create()
{
    FILE *f;
    int d=0;
    char c;
    ELEM *s;
    f=fopen("file.bin","wb");
    do{
    printf("Add elements to file?: (y/n)");
    fflush(stdin);
    scanf("%c",&c);
    if (c=='y')
    {
        printf("Name=");
        gets((*s).name);
        printf("P=");
        scanf("%d",(*s).p);
        fwrite(s,sizeof(ELEM),1,f);
    }
    } while(d==0);
    fclose(f);
}

void show()
{
    FILE *f;
    ELEM *s;
    f=fopen("file.bin","rb");
    while(feof(f)!=NULL)
    {
        fread(s,sizeof(ELEM),1,f);
        puts((*s).name);
        printf("\t%d\n",(*s).p);
    }
    fclose(f);
}

void add()
{
    FILE *f;
    int d=0;
    char c;
    ELEM *s;
    f=fopen("file.bin","ab");
    do{
    printf("Add elements to file?: (y/n)");
    fflush(stdin);
    scanf("%c",&c);
    if (c=='y')
    {
        printf("Name=");
        gets((*s).name);
        printf("P=");
        scanf("%d",(*s).p);
        fwrite(s,sizeof(ELEM),1,f);
    }
    } while(d==0);
    fclose(f);
}


/*void function()
{

}*/

int main()
{
    int k=0,r;
    do{
        printf("1 - create file\n2 - add elements to fil\n3 - show elements\n4 - put unique elements in another file\n5 - exit program\n");
        scanf("%d",&r);
        switch(r)
        {
            case 1 : create(); break;
            case 2 : add(); break;
            case 3 : show(); break;
            case 4 : printf("Function not defined!\n"); break;
            case 5 : k=1; break;
            default : printf("Command unrecognized!\n");
        }
    } while(k==0);
    return 0;
}

Answer 1

您声明了一个指针，但未为其分配任何内存。 您应该恢复为普通变量：

ELEM s;

/* ... */

fwrite(&s,sizeof(ELEM),1,f);
       ^

或者，在您当前的代码中，您应该这样做：

ELEM *s = calloc(1, sizeof *s);

Answer 2

传递给fwrite的第一个参数应该是一个地址，并且传递给其地址的变量必须具有足够的内存以容纳计划读取的对象数。

因此，有两种方法：

在堆栈上创建变量：

您在堆栈上分配变量并将其地址传递给fwrite

ELEM s;
fwrite(&s,sizeof(ELEM),1,f);

要么

动态内存分配：

ELEM *s;   

应该分配一个等效于不容纳要读取的ELEM类型对象的内存的内存。

ELEM *s = malloc(sizeof *s);

在这种情况下，请记住在使用后释放内存：

free(s);

Answer 3

你是不是分配指针s在create功能。 您可能想要类似

 s = malloc(sizeof(*s));
 memset (s, 0, sizeof(*s));

等等

而且，您确实应该学习使用gcc -Wall -g进行编译以及使用gdb调试器。

另外，请花时间阅读有关C语言编程的好书。

Answer 4

好吧，你的问题出在scanf上。 scanf应该获得一个指针，因此将(*s).p更改为&(*s).p （甚至是&s->p ）

Answer 5

您应该已将指针分配给struct ELEM假设这是C：

 ELEM *s = malloc(sizeof(ELEM));

如果是C ++，只需在前面添加一个强制转换

 ELEM *s = (ELEM*) malloc(sizeof(ELEM));