Unendlicher Loop bei Auslesen aus Datei

[philishake]

Hey Leute,

aus irgendeinem Grund, hört er nicht auf die Letzte Zeile meiner .txt-File auszugeben, obwohl klar definiert wann er aufhören soll.

values.txt:

Laufspeed 10
Rennspeed 11
Sprintspeed 12
health 13
armor 14
ammo 15
EOF

ext_values.cpp:

    // Deklariere Variablen
    int zeilen = 0;
	char str[50];

    // Öffne Datei
    ifstream b_file (dateiname);
  
    // Gebe Dateiname aus
    cout << "\nfilename: " << dateiname << "\n\n";
    
    // Starte Zeilenweises einlesen der Daten
    while(str != "EOF")
    {
      b_file >> str;
      cout << "content(str): " << str;
      b_file >> str;
      cout << " = " << str << "\n";
    }

Danke schon im voraus

 

[Ryu1991]

Vermutlich ist str nicht "EOF" sondern "EOF\n", aber es gibt da eh bessere methoden, um ne datei zeilen weise aus zu lesen, da nimm man in der regel den getline befehl.

#include <iostream>
#include <fstream>
#include <string>
using namespace std;

int main(){
  ifstream b_file("Dein\\File");
  string str;

  str.clear();

  while(getline(b_file, str)){
    cout << str << endl;
  }
  cin.get();
  return 0;
}

 

[philishake]

Alles klar. Danke ich probiers mal so aus.

 

[higret]

Hi

du vergleichst mit

if (str == "EOF")

nicht den String der in str steht, sondern nur die Adresse von str mit der Adresse von "EOF" und die beiden Adressen werden nie gleich sein.

Zum Vergleichen von C-Strings (char-Arrays) gibt es dir strcmp Funktion

http://www.cppreference.com/stdstring/strcmp.html

lg

 

[Ryu1991]

So wie er das gemacht hat, müsste das eigentlich gehen. Man müsste mal mit dem Debugger in den Inhalt von STR reinschauen, die Problematik die du beschrieben hast gibt es bei std::string nicht, sondern Bei Char-Arrays.

 

[devDevil]

Korrekt. DAS ist aber der wahre Grund, warum der Code des Topicerstellers nicht funktioniert.

std::ifstream file_stream(file_name);
std::clog << "Dateiname: " << file_name << "\n\n"; 

char string[50];
std::memset(string, 0, sizeof(char) * 50);

while (std::strcmp(string, "EOF") != 0)
{
    file_stream >> string;
    std::clog << "Content: " << string;
}

das ist ja soweit in Ordnung. Na gut man macht es anders. Falls EOF nicht vorhanden ist, brichst du damit nie das Lesen ab. Ich glaub du meinstest eher den State des Streams eof.


Zu deinem Code Ryu:

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


int main()
{
    std::ifstream file_stream("C:\\text.txt");

    std::string string;
    while (std::getline(file_stream, string)) std::clog << string << "\n";
    
    std::cin.ignore();
}

ignore, weil du das Zeichen nicht brauchst, was da noch steht. Du überprüfst nicht, ob die Datei geöffnet wurde, sonst aber i.O.

Besser:

#include <fstream>
#include <algorithm>
#include <iterator>
#include <iostream>

int main()
{
    std::ifstream file_stream("C:/Test.txt");
    if (!file_stream) { std::cerr << "FEHLER: Datei konnte nicht geöffnet werden!"; return 1; }

    std::copy(std::istream_iterator<char>(file_stream), std::istream_iterator<char>, std::ostream_iterator(std::clog, "");
}

Und für ultimo nen kleiner Trick:

#include <fstream>
#include <iostream>

int main()
{
    std::ifstream file_stream("C:/Test.txt");
    if (!file_stream) { std::cerr << "FEHLER: Datei konnte nicht geöffnet werden!"; return 1; }

    std::clog << file_stream.rdbuf();
}

 

[philishake]

Abermals ein dickes Danke für die vielen verschiedenen Möglichkeiten und Ansätze