Warum C++ ifstream die Datei mit doppelten Datensätzen lesen

Question

Mögliche Duplizieren:
Reading from text file until EOF repeats last line
c++ EOF running one too many times?

ich eine einfache c schreibe ++ Code eine binäre Datei lesen und schreiben, aber ich festgestellt, dass der ifstream den Datensatz zweimal lesen wird. Ich weiß nicht, wie dies geschieht, aber ich versuche, den Code mit mingw32 in Windows und Linux, den gleichen Fall

ofstream outfile; 
int data[5]; 
outfile.open("abc.out", ios::out | ios::trunc | ios::binary); 
data[0] = data[1] = 1; 
data[2] = data[3] = 2; 
data[4] = -1; 
cout << "ORIGINAL DATA:" << data[0] << " " << data[1] << " " << data[2] << " " << data[3] << " " << data[4] << endl << endl; 
outfile.write((char *)&data[0], 5*sizeof(int)); 
outfile.close();  


ifstream infile; 
infile.open("abc.out", ios::in | ios::binary); 
data[0] = data[1] = data[2] = data[3] = data[4] = 0; 
while (!infile.eof()) 
{ 
    infile.read((char *)&data[0], 5*sizeof(int)); 
    cout << data[0] << " " << data[1] << " " << data[2] << " " << data[3] << " " << data[4] << endl; 
} 

hier zu kompilieren ist die Ausgabe

ORIGINALDATEN: 1 1 2 2 -1

1 1 2 2 -1

1 1 2 2 -1

Answer 1

nicht immer .eof() oder .good() als Schleifenbedingung verwenden. Es erzeugt fast immer fehlerhaften Code (wie in diesem Fall).

Die idiomatische Muster zum Lesen von Daten ist C++ ist das:

while (infile.read((char *)&data[0], 5*sizeof(int))) 
{ 
    cout << data[0] << " " << data[1] << " " << data[2] << " " 
     << data[3] << " " << data[4] << endl; 
} 

Answer 2

@BoPersson Wie bereits erwähnt, ist das Problem ergibt sich aus while (!infile.eof()) verwenden, welche praktisch immer ein Fehler vorliegt. Stattdessen möchten Sie das Ergebnis des Lesens der Daten überprüfen. Eine erste Annäherung würde wie folgt aussehen:

// prefer 1-step initialization -- supply parameters to ctor. 
ifstream infile("abc.out", ios::in | ios::binary); 

while (infile.read((char *)&data[0], 5*sizeof(int)) 
    cout << data[0] << " " 
     << data[1] << " " 
     << data[2] << " " 
     << data[3] << " " 
     << data[4] << endl; 

Answer 3

Da ein Strom schlecht geht, wenn ein Lese ausfällt und ein schlecht Strom kann als falsch Bool zu sehen ist, und da Lese liefert den Strom istelf, Sie können tun

while(infile.read((char *)&data[0], 5*sizeof(int))) 
    cout << data[0] << " " << data[1] << " " << data[2] << " " << data[3] << " " << data[4] << endl; 

damit die Schleife nur auf den Lesefehler verlassen.