Файловый эхо-цикл с дополнительной финальной итерацией

Question

Почему я получаю дополнительную итерацию (дополнительная строка), когда этот код завершается? Должна ли быть дополнительная новая линия в EOF? Я бы предпочел не добавлять лишние/специальные символы, чтобы отметить EOF.

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

int main(){ 
    ifstream infile("dictionary.txt"); // one word per line 
    string text; 
    while(infile){ 
     infile >> text; 
     cout << text << endl; 
    } 
    infile.close(); 
    return 0; 
} 

Answer 1

попробовать

while(infile>>text) cout << text << endl; 

вместо этого.

Answer 2

С вашим условием while вы проверяете, находится ли поток в хорошем состоянии. Затем вы читаете из потока, который может или не может быть успешным. Затем вы выводите значение текста. То, что вы должны сделать, это:

while(infile >> text){ 
    cout << text << endl; 
} 

Answer 3

В конце файла, infile еще может вычисляться true, но после удаления слова с infile >> text терпит неудачу. Даже если это не удается, вы все равно распечатываете строку. Лучший способ сделать это было бы позволить проверку контура, а для успешного извлечения:

string text; 
ifstream infile("dictionary.txt"); // one word per line 
while (infile >> text) { 
    cout << text << endl; 
} 
infile.close(); 

Answer 4

Входной поток не не выявлять конец файла до после вы пытаетесь прочитать мимо него. Когда вы читаете последнее слово в файле, входной поток все еще действителен; в следующем цикле infile >> текст пытается прочитать прошлый EOF и не удается, но следующая строка все равно выполняется.

Петля должна выглядеть следующим образом:

while (infile >> text) 
    cout << text << endl; 

Таким образом EOF будет обнаружен, прежде чем он пытается записать на выход.

Answer 5

Вы уже получили ряд поправок, но, возможно, немного отличается один стоит учесть:

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

using namespace std; // not really a good idea, but harmless enough for now. 

int main() { 
    ifstream infile("dictionary.txt"); 
    copy(istream_iterator<string>(infile), istream_iterator<string>(), 
     ostream_iterator<string>(cout, "\n")); 
    return 0; 
}