3. Klasse in eine Binärdatei schreiben und wieder auslesen

Klasse in eine Binärdatei schreiben und wieder auslesen

  • T
     

    struct database
{

    char vorname[255];
    char nachname[255];
    char strasse[510];
    char ort[255];
    int hausnum;
    int plz;
    int gebtag;
    int gebmonat;
    int gebjahr;
    int id;

};

    und

    vector<database> Personendaten;

    Und personendaten wird an die entsprechende funktion übergeben..
    Hier der Prototyp:

    int show_eintrag(HANDLE stdOut, vector<database> daten, COORD xy);
    0
  • S

    Thunaer schrieb:

    ...

    while(i!=e)
...

    Hmmm, das sieht so ais, als würdest Du hier 2 Konzepte mischen: Iterator und Index.
    Dazwischen musst Du Dich entscheiden.
    (BTW: for-Schleifen sind ideal für sowas)

    Also entweder:

    for(size_t i=0; i<daten.size(); ++i) 
      cout<<daten[i].nachname<<"|"<<daten[i].vorname<<"|"<<daten[i].strasse<<" "<<daten[i].hausnum<<endl;

    oder

    for(vector<database>::const_iterator i=daten.begin(); i<daten.end(); ++i) 
      cout<<i->nachname<<"|"<<i->vorname<<"|"<<i->strasse<<" "<<i->hausnum<<endl;

    (Du siehst also: Iteratoren verwendet man wie Zeiger - genauergesagt sind "Zeiger" hier ein Sonderfall von Iteratoren)

    Noch ein paar Anmerkungen:
    1.) mach nicht zuviel "endl". kostet nur unnötig Performance. Zeilenumbruch geht prima mit '\n';
    2.) ich würde mir seeeeehr genau überlegen, ob ich WIRKLICH die Objekte im compilerspezifischen Binärformat ablegen will - denn das hat eine Menge Nachteile und IMHO nur sehr wenig Vorteile. Da gab's hier mal eine schöne Seite, auf die verwiesen wurde, aber den Link finde ich nicht mehr.
    3.) Denk daran, wie Parameter per value übergeben werden: Es wird eine Kopie angelegt, was bei größeren vector-en mächtig auf die Performance schlagen und bei ändernden Zugriffen zu unerwarteten Ergebnissen führen kann.
    4.) Wenn Du einen Parameter nicht ändern willst, deklariere ihn const. Das schafft Klarheit und Sicherheit.

    Gruß,

    Simon2.

    0
  • ?
     

    struct database
{

    char vorname[255];
    char nachname[255];
    char strasse[510];
    char ort[255];
    int hausnum;
    int plz;
    int gebtag;
    int gebmonat;
    int gebjahr;
    int id;

};

void show_eintrag(vector<database> const& daten){
	for( size_t i=0; i<daten.size();i++)
    {
         cout<<daten[i].nachname<<"|"<<daten[i].vorname<<"|"<<daten[i].strasse<<" "<<daten[i].hausnum<<endl;
    }
}

int main()
{
	vector<database> Personendaten;
	show_eintrag(Personendaten);
    return 0;
}

    kompiliert bei mir ohne probleme

    0
  • S

    ohje schrieb:

    ...kompiliert bei mir ohne probleme

    Ich glaube ja auch nicht, dass sein Problem bei dem Typ von "daten" sondern bei dem von "i" liegt (siehe mein letztes Posting).

    Gruß,

    Simon2.

    0
  • T

    Hm ok danke!
    Die Eingabe und die Ausgabe funktioniert wunderbar bisher....
    Wenn ich aber versuche die Einträge so abzuspeichern gibt der Compiler mir einen Fehler aus....

    FILE *datei;
    if((datei = fopen("data.db", "wb")) == NULL)
    {
         fflush(stdin);
         cout<<"Datei konnte nich geoeffnet werden!"<<endl;
         cin.get();
    }
    else
    {
         fwrite(daten, sizeof(struct database), daten.size(), datei); 
    }
    fclose(datei);

    Der Fehler:

    cannot convert `std::vector<database, std::allocator<database> >' to `const void*' for argument `1' to `size_t fwrite(const void*, size_t, size_t, FILE*)'

    Wäre nett wenn ihr mir mal wieder helfen könntet

    mfg

    Edit:
    Hm, warum funktioniert das nich:

    database *tmp;
   size_t i;
   int x = 0;
   for(i=0; i<daten.size(); ++i)
    {
         strcpy(tmp[x].nachname, daten[i].nachname);
         strcpy(tmp[x].ort, daten[i].ort);
         strcpy(tmp[x].strasse, daten[i].strasse);
         strcpy(tmp[x].vorname, daten[i].vorname);
         tmp[x].gebjahr = daten[i].gebjahr;
         tmp[x].gebmonat = daten[i].gebmonat;
         tmp[x].gebtag = daten[i].gebtag;
         tmp[x].hausnum = daten[i].hausnum;
         tmp[x].id = daten[i].id;
         tmp[x].plz = daten[i].plz;
         x++;
    }
    0
  • T

    Achso was ich noch sagen wollte:
    Der Compiler gibt mir beim letzteren Code keinen Fehler aus, sondern die *.exe schmiert an der Stelle des ersten strcpy ab....

    0
  • S

    Hi,

    mein Tipp: Lass' es !!

    (das "Binärspeichern") 😉
    Du hast eigentlich scohn alles, was Du brauchst: So wie Du Deine Objekte auf Konsole ausgibst, kannst Du sie auch in eine Datei ausgeben und wieder einlesen.
    Das einzige, was Du tun musst, ist, mit dem operator<<() in einen ofstream statt nach cout zu "schieben"....

    Glaub mir: Es hat zahllose Vorteile und (für Dich) keine Nachteile....

    Gruß,

    Simon2.

    0
  • T

    Simon2 schrieb:

    Hi,

    mein Tipp: Lass' es !!

    (das "Binärspeichern") 😉
    Du hast eigentlich scohn alles, was Du brauchst: So wie Du Deine Objekte auf Konsole ausgibst, kannst Du sie auch in eine Datei ausgeben und wieder einlesen.
    Das einzige, was Du tun musst, ist, mit dem operator<<() in einen ofstream statt nach cout zu "schieben"....

    Glaub mir: Es hat zahllose Vorteile und (für Dich) keine Nachteile....

    Gruß,

    Simon2.

    Hm ja.
    Das Problem ist nur, dass es ein Projekt für die Schule ist, welches benotet wird und der Lehrer Binärdateien fordert....
    Was wir machen konnten wir uns aussuschen, es sollten aber aufjedenfall:
    1. Daten in Binärdateien geschrieben ausgelesen werden
    2. Sortier und Suchalgorythmen vorhanden sein

    Ich würde es mit Binärdateien ja liebend gerne lassen, aber es geht ja leider nicht....

    0
  • T

    Hab meinen fehler schon selber gefunden, war wirklich dumm irgendwie ^^

    Das hat gefehlt:

    tmp = new database[daten.size()];
    0
  • S

    Thunaer schrieb:

    ...
    Hm ja.
    Das Problem ist nur, dass es ein Projekt für die Schule ist, welches benotet wird und der Lehrer Binärdateien fordert...

    OK - da kann man nichts machen.

    Dein Ansatz mit fwrite() ist an sich schon nicht schlecht, aber Du musst genau betrachten, was Du speichern willst:
    - das vector-Objekt (also alle Verwaltungsinformationen),
    - die einzelnen Elemente hier also die "database-Objekte".
    Stell Dir einfach mal eine Simpelimplementation von vector vor:

    class vector {
   database* elemente;
   size_t anzahl;
public:
   database& operator[](size_t i) { return elemente[i]; }
   size_t size() { return anzahl; }
};

    Wenn Du nun diesen vector abspeicherst, kommt in die Datei lediglich ein Zeiger und ein size_t ... was Dir wahrscheinlich nicht reichen wird.
    (mit string hättest Du dasselbe Problem - bei Deiner database-Klasse hast Du das "Glück", dass sie ihre Daten als char-Arrays halten).
    Wenn Du das also "binär" speichern willst, solltest Du die Elemente einzeln durchgehen und jedes einzelne "weg-fwrite-n".

    Gruß,

    Simon2.

    P.S.: Vielleicht kannst Du mal Deinen Lehrer auf die Probleme mit dem "Binärspeichern" hinweisen ... natürlich erst, wenn Du Deine Note eingesackt hast. 😉

    0
Anmelden zum Antworten