3. Array mithilfe von getline() füllen

Array mithilfe von getline() füllen

  • M

    Du machst einmal zu oft push_back(). Denn eof wird erst beim nächsten Leseversuch nach Erreichen des Dateiendes gesetzt.

    Und überprüf erst mal, ob die Größe deines Vektors mindestns 5 ist, bevor du auf das fünfte Element zugreifst.

    0
  • *

    Ich weiss Eure Hilfe wirklich zu schätzen aber könnte jemand mir bitte sagen, nicht nur was ich falsch mache, sondern auch wie man's richtig macht? um das Problem mit der Länge des Vektors zu umgehen, habe ich die for-Schleife eingebaut. Wenn ich das richtig verstanden habe, Michael, dann kommt das pushback nach do-while? würde also so aussehen:

    [...]
	quesFile.seekg (0, ios::beg);
    do
    {
		getline(quesFile,inhalt);
    }
	while (!quesFile.eof());
	line.push_back(inhalt);
	for (unsigned int i = 0; i < inhalt.size(); i++) cout << "5. Array: " << inhalt[i] << endl;
	cin.get();
[...]

    und folgenden Output produzieren, was eigentlich nur die Zeichen in der letzten Zeile sind:

    5. Array: 2
5. Array: 0
5. Array:
5. Array: -
5. Array: -
5. Array: -
5. Array: i
5. Array: n
5. Array: h
5. Array: a
5. Array: l
5. Array: t
5. Array: -
5. Array: -
5. Array: -

    Da kommt zwar keine fehlermeldung, aber das ist nicht das, was ich haben möchte.

    0
  • M

    Nein, so verwirfst du ja alle Zeilen und nur für die letzte rufst du push_back auf.

    Eher so:

    fstream quesFile("E:\\C++\\Microsoft Visual Studio 9.0\\MyProjects\\Quizzle\\Quizzle\\fragen.txt", ios::in);  // spar dir den Aufruf von open()
    vector<string> line;
    string inhalt;
    // quesFile.seekg (0, ios::beg);  überflüssig
    while(getline(quesFile, inhalt))
        line.push_back(inhalt);
    0
  • P

    [Rewind] schrieb:

    Wenn ich das richtig verstanden habe, Michael, dann kommt das pushback nach do-while?

    Nein, das hast du falsch verstanden. Das push_back muss schon in die Schleife, aber eben nur wenn der vorangegangene Leseversuch erfolgreich war. ich würds so machen:

    vector<string> fileContent; //
  string line;
  ifstream questionFile("E:\\C++\\Microsoft Visual Studio 9.0\\MyProjects\\Quizzle\\Quizzle\\fragen.txt");  

  getline(questionFile, line);
  while (!questionFile.eof())
  {
    fileContent.push_back(line);
    getline(questionFile, line);
  }

    ich hab die Variablen mal etwas umbenannt, weil du 4 Unschönheiten drin hattest:
    - deutsche und englische Namen gemischt verwirren nur
    - aussagenlose Variablennamen (inhalt? wessen inhalt?) machen den Code weniger klar als er sein könnte
    - falsche Variablennamen (der vector enthält alle Zeilen der datei, sollte also nicht line heißen) sind irreführend und verwirren unnötig
    - Abkürzungen (quesFile) machen den Code nur unnötig "holprig" zu lesen - und den Tippaufwand nimmt dir deine IDE bei Namensvervollständigung eh ab, egal obs 8 oder 15 Zeichen sind. Niemand liest gern "mkNewPtr2prvRefObj"

    0
  • U

    nur aus reiner neugier:
    geht das nich iwie (eleganter) mit std-mitteln? ^^

    ich hab jz aber auf die schnelle nichts hinbekommen - kanidaten wären zwar imho std::istream_iterator und std::back_inserter , aber kein Plan - ich hab nichts zu Stande bekommen : D

    bb

    0
  • M

    http://www.c-plusplus.net/forum/viewtopic-var-t-is-240556-and-postdays-is-0-and-postorder-is-asc-and-start-is-7.html

    0
  • U

    ty : >

    0
  • *

    Ok, ich habe es endlich zum Laufen gebracht und es funktioniert einwandfrei. Vielen Dank, unskilled, dein Vorschlag hat prima funktioniert. Die Variante von pumuckl spuckte bei mir nur einzelne Zeichen der letzten Zeile aus (der Fehler kann auch bei meiner Umsetzung liegen, aber ich bezweifle's). Und danke pumuckl für die Tipps. Ich werd versuchen, sie im Kopf zu behalten, wenn ich nächstes Mal um deine Hilfe bitte 🙂

    0
  • P

    [Rewind] schrieb:

    Die Variante von pumuckl spuckte bei mir nur einzelne Zeichen der letzten Zeile aus (der Fehler kann auch bei meiner Umsetzung liegen, aber ich bezweifle's).

    Ich bezweifle es meinerseits 😉
    Du musst natürlich die Schleife mit dem output über den vector (fileContext) laufen lassen, nicht über den String (line) - im String ist nämlich am Ende tatsächlich nur die letzte Zeile enthalten, und dort mit der Schleife drüber zu laufen gibt dann tatsächlich nur einzelne Zeichen aus.

    0
  • U

    [Rewind] schrieb:

    Vielen Dank, unskilled, dein Vorschlag hat prima funktioniert.

    Danke für die Blumen - nur hab ich in dem ganzen thread genau 2 posts geschrieben - eine frage und einmal nen danke, weil jmd anderes die forensuche richtig bedienen kann : D

    bb

    0
Anmelden zum Antworten