no matching function for call to `std::basic_ofstream<char, std::char_traits<char> >::open(std::string&, const std::_Ios_Openmode&)'

Wo liegt mein Fehler?

#include <iostream>
#include <string>
#include <fstream>
using namespace std;
//Die Noten-Klasse
class noten
{
      public:
             int deutsch;
             int englisch;
             int mathe;
             int technik;
};
//Die Schüler-Klasse
class schueler
{
      public:
             string nachname;
             string vorname;
             string strasse;
             int plz;
             string ort;
             noten n;
};

int main(void)
{
    schueler a;
    char neu;
    do
    {
         //
         ifstream lesen;
         string dateiadresse;
         cout << "Bitte geben Sie die Adresse der Datei ein: ";
         cin >> dateiadresse;
         lesen.open(dateiadresse.c_str());
         cout << lesen.get();
         lesen.close();
         cout << "Nachname: ";
         cin >> a.nachname;
         cout << "Deutsch-Note: ";
         cin >> a.n.deutsch;

         //
         ofstream schreiben;
         schreiben.open(dateiadresse, ios_base::app);
         schreiben << a.nachname;
         schreiben << a.n.deutsch;
         schreiben.close();
         cout << "Moechten Sie einen weiteren Schueler hinzufuegen?" << endl 
              << "Antworten Sie bitte mit j fuer ja oder n fuer nein: ";
         cin >> neu;
    }while(neu=='j');
    return 0;
}

Wo liegt mein Fehler?

//
         ofstream schreiben;
         schreiben.open(dateiadresse, ios_base::app);
         schreiben << a.nachname;
         schreiben << a.n.deutsch;
         schreiben.close();
         cout << "Moechten Sie einen weiteren Schueler hinzufuegen?" << endl 
              << "Antworten Sie bitte mit j fuer ja oder n fuer nein: ";
         cin >> neu;
    }while(neu=='j');
    return 0;
}

schreiben.open(dateiadresse.c_str(), ios_base::app);

'c_str()'.. so wie oben auch

Gruß
Werner

lesen.open(dateiadresse.string::c_str());

Aber nun verweist er auf die Zeile

schreiben.open(dateiadresse, ios_base::app);

und sagt:

no matching function for call to `std::basic_ofstream<char, std::char_traits<char> >::open(std::string&, const std::_Ios_Openmode&)'

xxx.open(dateinadresse.c_str());

(beim Lesestream hattest du das bereits richtig, beim Schreibstream mußt du das noch korrigieren)

^* Wieso das so ist, weiß ich auch nicht genau, aber das Standardisierungskomitee wird schon seine Gründe gehabt haben.

Ich habe nun kurz die Zeile

cout << lesen.get();

editiert und die "Methode?" get(); weggelassen. Dann spuckt er zwar mehr als zwei Zeichen aus aber in Hex glaube ich. Ähnelt einer Speicheradresse: 0x22fe84 (wobei x ja kein Hex-Wert sein kann)

(btw, du solltest Trennzeichen zwischen die Namen und zugehörigen Noten setzen, das erleichter das Einlesen)

class schueler
{
      public:
             string nachname;
             string vorname;
             string strasse;
             int plz;
             string ort;
             noten n;
};

Die Klasse heißt ja schueler und die objekte sind nachname und eine Funktion in einer Klasse heißt Methode. Und wie heißt nun der datensatz, den ich übergebe? Ist das das Element? Und was ist der Ausdruck für noten n; in diesem fall, weil noten ja eine eigene klasse ist.

Mit dem "Element", meinst Du damit eine Instanz der Klasse? Dann nennt man das Objekt, oder Instanz.

schueler klaus; 
^klasse oder typ
         ^instanz oder objekt

Innerhalb der Klasse schueler gibt es ein Mitglied n, vom Typ noten (string ist übrigens auch ne Klasse, dasselbe gilt also für Nachname usw.)

while(lesen)
{
  schueler tmp;
  lesen>>tmp.name;
  lesen>>tmp.n.deutsch;
  cout<<tmp.name<<" "<<tmp.n.deutsch;
}

Und was du als Trennzeichen verwendest, bleibt dir überlassen - du mußt nur deine Einlese-Funktion so anpassen, daß sie dieses Trennzeichen erkennt (eine Möglichkeit wäre, Leerzeichen, '\t' oder ';' zur Trennung der einzelnen Elemente zu verwenden und '\n' zur Trennung der Datensätze)

Die Formatierung bekommst du hin, indem du nicht per get() jedes Zeichen einzeln ausliest, sondern über die Stream-Operatoren die Werte einliest und direkt in den richtigen Typ parsen lässt.

Ich verstehe den Zusammenhang aber nicht. Also tmp ist nun eine neue Instanz der schuelerklasse.

Genau richtig. Du könntest auch eine bereits vorhandene Instanz dafür verwenden.

Und was bringt mir das jetzt?? Ich verstehe nicht was du wozu vorhast - kann dir nicht folgen

Das ist ein selbständiges Objekt - d.h. du könntest es z.B. in einen vector packen, an andere Funktionen übergeben oder irgendwie weiterverarbeiten. Wichtig ist im Moment, daß du in diesem Objekt die Schülerdaten aus deiner Datei eintragen kannst.

t1m0n schrieb:

Ich verstehe den Zusammenhang aber nicht. Also tmp ist nun eine neue Instanz der schuelerklasse.

Genau richtig. Du könntest auch eine bereits vorhandene Instanz dafür verwenden.

Ich habe ja a als instanz genommen, möchte demnach mit a halt weitermachen weil es für mich als Anfänger ja klar erkennbar sein soll was was ist.

Wichtig ist im Moment, daß du in diesem Objekt die Schülerdaten aus deiner Datei eintragen kannst.

Ja, das habe ich ja auch vor. Ich möchte den Inhalt auslesen (so wie er wirklich ist) und dann weiter beschreiben können. Warum hast du als bedingung bei while(lesen) lesen verwendet? Welchen wert liefert das?

vector sagt mir nichts..

Warum hast du als bedingung bei while(lesen) lesen verwendet? Welchen wert liefert das?

streams können implizit in bool umgewandelt werden - dabei entspricht "false", daß der Stream in irgendeinem Fehlerzustand gelandet ist (z.B. EOF oder falsches Zeichen am falschen Platz). D.h. die Schleife arbeitet so lange weiter, wie sie Daten auswerten kann.

vector sagt mir nichts..

Das ist eine Möglichkeit, viele gleichartige Daten auf einem Haufen zu speichern (analog zu Arrays, aber deutlich flexibler). Wenn du noch einen Tag wartest - in der nächsten Ausgabe des C++-Magazins kommt auch ein Artikel zu vector<> und anderen Containerklassen der STL.

Wenn du noch einen Tag wartest - in der nächsten Ausgabe des C++-Magazins kommt auch ein Artikel zu vector<> und anderen Containerklassen der STL.

Isat das Magazin denn OpenSource? Habe eine ältere Ausgabe im Fachhandel gesehen, kostete allerdings 10€ und das finde ich ist ein wenig viel oder?

Aber wozu benötige ich jetzt die Vektoren? Mit deiner schleife kann ich das doch alles ausgeben lassen oder?

Mir gibt die Konsole nun folgendes aus: 97509851100529949-1
Ich habe das einlesen so gestalltet:

//Auslesen der Datei
         ifstream lesen;
         lesen.open(dateiadresse.c_str());
         while (lesen)
         {
         cout << lesen.get();
         }
         lesen.close();

Als ich die Methode get weggelassen habe, gab es eine Endlosschleife, woran lag das eigentlich?

CStoll schrieb:

Wenn du noch einen Tag wartest - in der nächsten Ausgabe des C++-Magazins kommt auch ein Artikel zu vector<> und anderen Containerklassen der STL.

Isat das Magazin denn OpenSource? Habe eine ältere Ausgabe im Fachhandel gesehen, kostete allerdings 10€ und das finde ich ist ein wenig viel oder?

Das Magazin findest du hier an Bord - Magazin - die Artikel

Aber wozu benötige ich jetzt die Vektoren? Mit deiner schleife kann ich das doch alles ausgeben lassen oder?

Wenn du mehr machen willst als ein- und ausgeben (z.B. die Datensätze sortieren), mußt du deine Daten irgendwo intern zwischenspeichern.

Mir gibt die Konsole nun folgendes aus: 97509851100529949-1
Ich habe das einlesen so gestalltet:

//Auslesen der Datei
         ifstream lesen;
         lesen.open(dateiadresse.c_str());
         while (lesen)
         {
         cout << lesen.get();
         }
         lesen.close();

[/quote]get() gibt einen int zurück (-1 steht übrigens für EOF), wenn du etwas lesbareres haben willst, mußt du das zumindest nach char casten. Oder du liest dir den Thread nochmal aufmerksam durch und denkst etwas länger über operator>> nach.

tmp ist ein Objekt vom Typ "schueler" - dort kannst du alle eingelesenen Daten des Schülers zusammenfassen (nächste Steigerung des Codes wäre es übrigens, für "schueler" und "noten" einen eigenen operator>> und operator<< zu definieren).

Die Formatierung bekommst du hin, indem du nicht per get() jedes Zeichen einzeln ausliest, sondern über die Stream-Operatoren die Werte einliest und direkt in den richtigen Typ parsen lässt.

Ich glaube du willst darauf hinaus. Ich kenne aber nicht solche Operatoren. Ich habe das übrigens nun so umgeschrieben

schreiben << a.nachname << " " << a.n.deutsch;

Dort habe ich ja auch den << Operator. Hängt das irgendwie zusammen mit dem worauf du hinaus willst??

lesen >> a.nachname >> a.n.deutsch;

(und bei der Ausgabe würde ich noch ein "\n" hinterherschieben, dann sieht deine Datei übersichtlicher aus)

Aber woher soll mein Programm denn beim Einlesen wissen was a.nachname oder a.n.deutsch ist? Ich gehe davon aus, das die Laufzeit beendet ist und die Datei nach x-Tagen wieder eingelesen wird.

Dazu mußt du beim Schreiben die Reihenfolge festlegen, in der die Daten in der Datei landen und später in der selben Reihenfolge wieder auslesen (das Leerzeichen und Newline werden von op>> als Trennzeichen zwischen den einzelnen Werten angesehen).

(im Klartext: DU weißt, daß in der Datei erst der Name und dann die zugehörige Deutsch-Note kommt. Also mußt du die Elemente auch in genau dieser Reihenfolge wieder lesen)

ifstream lesen;
         lesen.open(dateiadresse.c_str());
         while (lesen)
         {
         lesen >> a.nachname >> a.n.deutsch;
         }
         lesen.close();

Und was das Schreiben angeht, habe ich das so gemacht:

//Schreiben der Datei
         ofstream schreiben;
         schreiben.open(dateiadresse.c_str(), ios_base::app);
         schreiben << a.nachname << " " << a.n.deutsch << "\n";
         schreiben.close();

Das Schreiben funktioniert einwandfrei. In der txt steht alles drinne wie es soll, aber auslesen tut er rein garnix.