istream::tellg bricht Schleife ab

Werner Salomon

Hallo Julius,

lass es mich noch mal zusammen fassen: Du behauptest, dass Dein Programm fuinktioniert, also mehr als einen Datensatz aus der Datei liest und ausgibt, wenn die Zeile 5 in Deinem ersten Listing auskommentiert ist. Aktivierst Du dann die Zeile 5

pos = datei.tellg();

, dann läuft die Schleife nur noch genau einmal durch - und das obwohl 'pos' den Wert von 236!=-1 annimmt.

Also um ehrlich zu sein - Du musst Dich vertan haben. Prüfe das, was ich geschrieben habe, bitte noch mal so nach, ob das auch passt.
Ansonsten füge folgende Zeilen ein:

while(datei >> tag >> monat >> jahr >> uhrzeit1 >> uhrzeit2 >> pause >> soll >> diffmin){
        if(monat==auswahl_monat && jahr==auswahl_jahr){
            cout << id << "\t" << tag << "\t" << uhrzeit1 << "\t" << uhrzeit2 << "\t" << pause << "\t" << soll << "\t" << diffmin << endl;
            id++;
            cout << "> datei.fail()=" << datei.fail() << endl; // <- neu
            pos = datei.tellg();
            cout << "> pos=" << pos << endl; // <- neu
            cout << "> datei.fail()=" << datei.fail() << endl; // <- neu
        }

    }

.. und berichte uns, wie die Ausgabe ist.

js707 schrieb:

Und ich bin Anfänger und verstehe deinen Post nicht richtig (Werner).

und wenn Du konkrete Fragen stellst, kann ich die auch gerne beantworten

Gruß
Werner

Hey Werner,

danke für die Hilfe.

Folgende Ausgabe kommt heraus:

> datei.fail()=0
> pos=236
> datei.fail()=0

Schöne Grüße

Julius

Werner Salomon

Hallo Julius,

.. dann darf das Einfügen oder Weglassen von Zeile 5 auch keinen Unterschied machen, was die Anzahl der Schleifendurchläufe betrifft. Da ist irgendwas anderes nicht ok, was Du uns noch nicht gesagt hast.

Gruß
Werner

pumuckl

Schau mal in SeppJs Post und Signatur. Oder in meine. Oder halte dich an den guten Ton im Netz und in Foren wie diesen und lies dir vorm Posten die mit wichtig gekennzeichneten Threads durch. Wird dich alles zu der gleichen Erkenntnis führen, dass man dir mit den paar Infos die du lieferst nicht helfen kann.

Belli

Vielleicht lohnt es sich, die Werte noch mal außerhalb der Bedingung auszugeben.

berniebutt

Die Abbruchbedingung in der while-Schleife erscheint mir zu kompliziert. Verlagere die Eingabe aus der Datei in die Schleife und prüfe dort auf Abbruch oder Fortsetzung. Entweder verwende eine logische Variable im while-Kopf oder break / continue innerhalb der Schleife.

Belli

berniebutt schrieb:

Die Abbruchbedingung in der while-Schleife erscheint mir zu kompliziert.

Häh? Die Abbruchbedingung ist einfach der Status des Streams. Solange der Stream okay ist, wird die Schleife ausgeführt.

berniebutt

Meine vielleicht unmassgebliche Meinung: Lieber ein paar Codezeilen mehr mit besser kontrollierbarem Ablauf des Programmes. Dann brauche ich keine solche Fragen hier zu stellen!

SeppJ

berniebutt schrieb:

Meine vielleicht unmassgebliche Meinung: Lieber ein paar Codezeilen mehr mit besser kontrollierbarem Ablauf des Programmes. Dann brauche ich keine solche Fragen hier zu stellen!

Wie würdest du es denn besser machen?

So?

while(true)
{
 datei >> tag >> monat >> jahr >> uhrzeit1 >> uhrzeit2 >> pause >> soll >> diffmin;
 if (!datei)
  break!

 rest_der_schleife();
}

Das wäre schon ein Antipattern.

js707

Moin,

ich habe jetzt zum Ausprobieren eine einfachere Bedingung im Schleifenkopf genommen und die Bedingung weggelassen.

Folgender Code:

while(id<4){
        datei >> tag >> monat >> jahr >> uhrzeit1 >> uhrzeit2 >> pause >> soll >> diffmin;
        pos = datei.tellg();
        cout << "ID = " <<id << " datei.fail() = " << datei.fail() << " pos = " << pos << endl;
        id++;
        }

Ergibt folgende Ausgabe:

ID = 1 datei.fail() = 0 pos = 44
ID = 2 datei.fail() = 1 pos = -1
ID = 3 datei.fail() = 1 pos = -1

Was bedeutet das?

Schöne Grüße

Julius

freaky

Das bedeutet, dass ID 2 und 3 bei dir fehlerhaft waren und die Position deshalb -1 ist, also so gesehen keine Positionen sind.

Belli

Das bedeutet, dass Du nur einmal ordentlich aus der Datei lesen konntest. Beim zweiten Leseversuch geht irgendwas schief.
Gib das auch noch mal vor dem tellg aus, damit man sieht, ob der Stream beim Lesen oder erst beim tellg - Aufruf auf fehlerhaft gesetzt wird.

berniebutt

SeppJ schrieb:

while(true)
{
 datei >> tag >> monat >> jahr >> uhrzeit1 >> uhrzeit2 >> pause >> soll >> diffmin;
 if (!datei)
  break!
 
 rest_der_schleife();
}

Das wäre schon ein Antipattern.

Schöner Anticode zum Abgewöhnen!
Der Fragesteller scheint aber langsam dahin zu kommen, wo er hin will?

js707

Neuer Code:

while(id<4){
        datei >> tag >> monat >> jahr >> uhrzeit1 >> uhrzeit2 >> pause >> soll >> diffmin;
        cout << "VOR tellg:\tID = " <<id << " datei.fail() = " << datei.fail() << " pos = " << pos << endl;
        pos = datei.tellg();
        cout << "NACH tellg:\tID = " <<id << " datei.fail() = " << datei.fail() << " pos = " << pos << endl;
        id++;

    }

Ausgabe:

VOR tellg:    ID = 1 datei.fail() = 0 pos = 0
NACH tellg:   ID = 1 datei.fail() = 0 pos = 44
VOR tellg:    ID = 2 datei.fail() = 1 pos = 44
NACH tellg:   ID = 2 datei.fail() = 1 pos = -1
VOR tellg:    ID = 3 datei.fail() = 1 pos = -1
NACH tellg:   ID = 3 datei.fail() = 1 pos = -1

Danke.

Belli

Also beim zweiten Lesen geht etwas schief, da danach das failbit gesetzt ist ...
Gib bitte zusätzlich zu datei.fail() noch datei.eof() und datei.bad() aus.

js707

Neuer Code:

while(id<3){
        datei >> tag >> monat >> jahr >> uhrzeit1 >> uhrzeit2 >> pause >> soll >> diffmin;
        cout << "VOR tellg:\tID = " <<id << " datei.eof()=" << datei.eof() << " datei.bad()=" << datei.bad() <<" datei.fail()=" << datei.fail() << " pos = " << pos << endl;
        pos = datei.tellg();
        cout << "NACH tellg:\tID = " <<id << " datei.eof()=" << datei.eof() << " datei.bad()=" << datei.bad() <<" datei.fail()=" << datei.fail() << " pos = " << pos << endl;
        id++;

    }

Ausgabe:

VOR tellg:    ID = 1 datei.eof()=0 datei.bad()=0 datei.fail()=0 pos = 0
NACH tellg:   ID = 1 datei.eof()=0 datei.bad()=0 datei.fail()=0 pos = 44
VOR tellg:    ID = 2 datei.eof()=0 datei.bad()=0 datei.fail()=1 pos = 44
NACH tellg:   ID = 2 datei.eof()=0 datei.bad()=0 datei.fail()=1 pos = -1

Vielen Dank für die bisherige Hilfe!

js707

Ich wollte nur kurz mitteilen, dass ich gesehen habe, dass die Fehler nur unter bestimmten Bedingungen bzgl. des Inhalts der Datei kommen.

Ich versuche den Fehler mal zu finden und schreibe, wenn ich eine Lösung habe.

Belli

Okay, bad und eof sind nicht gesetzt, aber fail ist gesetzt. Ich würde vermuten, dass in der Datei etwas zu lesen ansteht, das nicht zu dem Datentyp passt, in den Du einlesen willst. Also möglicherweise ist die Variable tag vom Typ int, in der Datei folgt aber keine Ziffer, irgendsoetwas muss es sein.

js707

@Belli:

Wenn datei.tellg() nicht verwendet wird, funktioniert ja alles und wir richtig eingelesen und auch ausgegeben.

Ich sehe gerade, dass da einiges komisch läuft:

datei.tellg() z.B. in der ersten Zeile, die eingelesen und ausgegeben wird nimmt den Wert datei.tellg()+1 an, wenn eine weitere Zeile in die Datei geschrieben wurde und diese dann eingelesen und ausgegeben wird. Füge ich noch weiter Zeilen hinzu, so wird datei.tellg() an dieser Stelle immer um 1 erhöht.

Dann passiert noch was komisches: Wenn 2 Zeilen in der Datei sind, wird die 1. korrekt ausgegeben. Die 2. nicht.
Schreibe ich weitere Zeilen in die Datei, so werden nur die 1. und die letzte Zeile richtig eingelesen und ausgegeben (die Zeilen dazwischen nicht).

Wenn diffmin in einer Zeile negativ ist, so wird diese zwar wie vorher ausgegeben (als letzte Zeile), allerdings läuft der ganze Loop nur noch ein mal, wenn ich dann (nach dem negativen diffmin) noch eine weitere Zeile anhänge (mit positivem diffmin).

Belli

Das klingt alles sehr komisch. Ich schlage vor, Du kürzt Dein Programm auf ein paar Zeilen (max 100) so, dass es vollständig kompilierbar ist, und das Problem noch aufweist. Dann postest Du dieses Programm hier, und zusätzlich die ersten paar Zeilen Deiner Eingabedatei.
Vielleicht findet sich dann jemand, der das selbst mal ausprobiert und helfen kann.