Diese Schleife führt mehr Iterationen als erwartet aus

Question

Ich habe Probleme mit dem folgenden Code. Was ich erwarte ist, dass die do-while Schleife 4 Mal ausgeführt wird, einmal für jede Zeile der Textdatei, in der sie eingelesen wird, aber in Wirklichkeit führt sie fünf Mal aus, was später zu einem Segmentfehler im Programm führt. Was mache ich hier falsch, was dazu führt, dass es die zusätzliche Iteration ausführt? Ich habe versucht, er do-while mit einer einfachen while Schleife zu ersetzen, aber das Ergebnis ist das gleiche.

int count = 0; 
string devices[4]; 
string line; 
ifstream DeviceList; 
DeviceList.open("devices/device_list.txt"); 
do 
{ 
getline(DeviceList, line); 
devices[count] = line; 
count ++; 
} while(!DeviceList.eof()); 

device_list.txt enthält folgende Komponenten:

WirelessAdaptor 
GPU 
CPU 
Display 

Answer 1

eof() nicht true zurück zu ignorieren, bis getline das Ende verbraucht. Es tut dies nicht, bis die getline Anruf nach die letzte Zeile lesen. Sie müssen überprüfen, ob eof wahr ist sofort nach getline Anruf:

while(true) 
{ 
    getline(DeviceList, line); 
    if(DeviceList.eof()) 
    break; 
} 

Answer 2

eof() nicht wahr zurück, bis Sie versuchen, mehr Daten zu lesen, als es gelassen wird.

Answer 3

Oberhalb der Linie getline(DeviceList, line); einfügen cout << line.length() << endl; und sagen Sie uns, was passiert.

Answer 4

Ich denke, Ihre Schleife wahrscheinlich mehr soll wie folgt aussehen:

Edit: Hinzugefügt Scheck Leerzeilen

while (getline(DeviceList, line)) 
{ 
    if (line.length() > 0) 
    { 
     devices[count] = line; 
     ++count; 
    } 
} 

Answer 5

Ihre Textdatei wahrscheinlich einen Zeilenvorschub nach der letzten Zeile enthält, so getline liest eine leere Zeichenfolge, bevor die Schleife tatsächlich endet.