Чтение и запись структуры в бинарный файл.

[jack291]

Доброго времени суток. Решил помочь другу с задачей, но не могу понять, в чем ошибка. Нужно из файла прочитать данные в структуру а затем записать их в бинарный файл.

вот что получилось:

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

struct notebook
{   
    int x;
    int y;
    int f;
    float d;
    float hdd;
    char model[21];
};

int main()
{
    FILE *fi, *fo;
    notebook s;
    if((fi=fopen("input","r"))==0) {return 1;}
    if((fi=fopen("output","w+b"))==0) {return 1;}
    const int dl=80;
    char buff[dl];
    int kol = 0;

    while(fgets(buff,dl,fi))
    {
        strncpy(s.model,buff,20);
        s.model[20]='\0';
        s.f=atoi(&buff[21]);
        s.x=atoi(&buff[25]);
        s.y=atoi(&buff[30]);
        s.d=atof(&buff[35]);
        s.hdd=atof(&buff[40]);
        kol++;
    }
    fclose(fi);
    int i=0;
    for(i;i<kol;i++);
    {
        fseek(fo,(sizeof s)*i,SEEK_SET);
        fread(&s,sizeof s,1,fo);
        std::cout<<s.model<<" "<<s.f<<" "<<s.x<<"x"<<s.y<<" "<<s.d<<" "<<s.hdd<<"\n";
    }

    fclose(fo);
    return 0;
}

Входной файл: input.txt

[onewho]

начнём с того что в вашем файле квадратики откуда-то вместо переноса строки.

Сообщение от jack291

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

#include <cstdio> и т.д

Сообщение от jack291

if((fi=fopen("input","r"))==0) {return 1;}
if((fi=fopen("output","w+b"))==0) {return 1;}

1. надо писать имя файла с расширением ("input.txt", "output.bin")
2. правильно писать wb+ , и почему + ?, вам же просто wb
3. 2 раза открыли 1 файл

Сообщение от jack291

strncpy(s.model,buff,20);
s.model[20]='\0';
s.f=atoi(&buff[21]);
s.x=atoi(&buff[25]);
s.y=atoi(&buff[30]);
s.d=atof(&buff[35]);
s.hdd=atof(&buff[40]);
kol++;

бгг. самонадеянно) чиселки типа 20 35...
мм.. без разделителей будет не очень хорошо файл парсить.
было бы очень хорошо сделать разделители например
вместо строки "Acer Note Light 2699 1024x0768 2200 120."
сделать "Acer Note Light : 2699 : 1024x0768 : 2200 : 120."
в таком случае я бы советовал сделать так:

sscanf(buff,"%[^:] :  %d : %dx%d : %d : %d",s.model,&s.f,&s.x,&s.y,&s.d,&s.hdd);

1 строчка вместо всего гемора .

далее запись вообще сделана плохо. имхо проще считывать и сразу же записывать в вых. файл.
то есть вместо этого:

Сообщение от jack291

while(fgets(buff,dl,fi))
...
...
fseek(fo,(sizeof s)*i,SEEK_SET); // брр... жесть
fread(&s,sizeof s,1,fo); // почему fread ?? и забыли приведение типа
std::cout<<s.model<<" "<<s.f<<" "<<s.x<<"x"<<s.y<<" "<<s.d<<" "<<s.hdd<<"\n";
}

сделать примерно так (и счетчик ни к чему):

    while(fgets(buff,dl,fi))
    {
     sscanf(buff,"%[^:] :  %d : %dx%d : %d : %d",s.model,&s.f,&s.x,&s.y,&s.d,&s.hdd);
     fwrite((char*)&s,sizeof(s),1,fo);
    }

ну и отладочный вывод можете сделать))

[jack291]

начнём с того что в вашем файле квадратики откуда-то вместо переноса строки.

Пользуюсь линуксом, видимо из за этого проблемы.

1. надо писать имя файла с расширением ("input.txt", "output.bin")

опять таки линукс.

3. 2 раза открыли 1 файл

Ой, опечатался...

#include <cstdio> и т.д

QT Creator требует .h, #include <cstdio> не сущетвует.

бгг. самонадеянно) чиселки типа 20 35...

Забыл сказать, что текст в файле отморматирован, т.е. имя занимает 20 символов вместе с пробелами, и т.д.

за остальное спасибо) Всё работает чудненько