struct string odczyt pliku plik binrany

Question

cześć,

od paru godzin męczę się z programem, chodzi o odczyt z pliku binarnego. Zapis już zrobiłem ale nie mogę go odczytać: pokazuje się tylko jakiś ciąg znaków. 

#include<iostream>
#include<fstream>
#include<string>
#include<cstdlib>

using namespace std;

struct student
{
    string imie;
    string kierunek;
};

int main()
{
    int n;
    cout<<"Podaj rozmiar : ";
    cin>>n;
    cin.get();
    student s[n];

    fstream plik;
    plik.open("test.dat", ios_base::binary | ios_base::out | ios_base::app );

    for( int i = 0 ; i<n; i++)
    {
        cout<<"Podaj imie : ";
        getline(cin,s[i].imie);
        plik.write((char *) &s[i].imie, sizeof s[i].imie);

        cout<<"Kierunek : ";
        getline(cin, s[i].kierunek);
        plik.write((char *) &s[i].kierunek, sizeof s[i].kierunek);
    }
plik.close();


student p;
fstream plik1;
plik1.open("test.dat", ios_base::binary | ios_base::in );

if(plik1.is_open()==false)
{
    cout<<"ERROR!"<<endl;
    exit(0);
}
else
    cout<<"Odczyt!"<<endl;
string linia,linia1;

while(plik1)
{
    plik1.read((char *)&p.imie, sizeof (p.imie));
    plik1.read((char *)&p.kierunek, sizeof (p.kierunek));
    getline(plik1,linia);
    cout<<linia;


}
plik1.close();
    return 0;}

 

Comments

A ty widziałeś jakikolwiek poradnik czy cokolwiek jak się zapisuje stringa do pliku binarnie?

Answer 1

Nie możesz w ten sposób zapisywać i odczytywać obiektów klasy std::string.

 

Może zrobić to tak:

uint32_t n = p.imie.size();

plik1.write(static_cast<char*>(&n), sizeof(n));
plik1.write(p.imie.c_str(), n);

...


plik1.read(static_cast<char*>(&n), sizeof(n));
while(plik1 && n--) p.imie += plik1.get();

 

Answer 2

1. std::string to klasa, więc s[i].imie czy s[i].kierunek są obiektami. 
2. sizeof da ci rozmiar w bajtach obiektu. W skład obiektu std::string wchodzi m.in. wskaźnik do pamięci z ciągiem znaków. Wskaźnik, nie sama pamięć. Tzn. zakładając, że std::string składa się z 4-bajtowego unsigned inta oraz 8-bajtowego wskaźnika, to sizeof(s[i].imie) da ci 12 (lub ewentualnie może to być nieco więcej ze względu na jakiś padding między intem a wskaźnikiem - zależne od kompilatora, ale nie o tym mowa) - długość stringa nie ma tu nic do rzeczy
3. &s[i].imie da ci wskaźnik do obiektu std::string, nie do pamięci z ciągiem znaków

Zatem aby wpisać do pliku zawartość stringa:

plik.write(s[i].imie.c_str(), s[i].imie.length());

Podobna historia z odczytem. Nie możesz sobie po prostu zczytać do pamięci w której jest obiekt string. Do bufora stringa też nie o ile wcześniej nie wymusiłeś alokacji wystarczającej ilości pamięci. Najbezpieczniejszy wg mnie byłby taki sposób:

char buf[1u<<10]; //1024 bajty

streamsize sz = /*ile znakow?*/;
plik.read(buf, sz);
buf[sz] = 0; // wstawiamy terminating zero na koniec zaladowanego ciagu znakow aby operator przypisania ponizej wiedzial gdzie ma zakonczyc kopiowanie
p.imie = buf; // operator przypisania std::string skopiuje sobie znaki z `buf`

Musisz tylko wiedzieć ile znaków chcesz wczytać czego obecnie najwyraźniej nie wiesz. `sizeof(p.imie)` na pewno magicznie ci tego nie powie.