从文本文件中读取,直到EOF重复最后一行

以下C++代码使用ifstream对象从文本文件(每行有一个数字)读取整数,直到它达到EOF.为什么它读取最后一行的整数两次？如何解决这个问题？

码:

#include 
#include 
using namespace std;

int main()
{
    ifstream iFile("input.txt");    // input.txt has integers, one per line

    while (!iFile.eof())
    {
        int x;
        iFile >> x;
        cerr << x << endl;
    }

    return 0;
}

input.txt:

10  
20  
30

输出:

10  
20  
30  
30

注意:我已跳过所有错误检查代码,以使代码段保持较小.在Windows(Visual C++),cygwin(gcc)和Linux(gcc)上可以看到上述行为.

1> wilhelmtell..：

---

只需密切关注事件链.

抢10

抢20

抢30

抓住EOF

看看倒数第二次迭代.你抓住30,然后继续检查EOF.你没有达到EOF,因为还没有读过EOF标记("二进制"说,它的概念位置就在30行之后).因此,您继续进行下一次迭代.x从前一次迭代开始仍然是30.现在你从流中读取并获得EOF.x仍为30,ios :: eofbit被提升.您输出到stderr x(这是30,就像在上一次迭代中一样).接下来,在循环条件下检查EOF,这次你不在循环中.

试试这个:

while (true) {
    int x;
    iFile >> x;
    if( iFile.eof() ) break;
    cerr << x << endl;
}

顺便说一句,你的代码中还有另一个错误.你有没有尝试在空文件上运行它？你得到的行为是出于同样的原因.

---

使用'while(iFile >> x)'.这将读取整数并返回流.当流用作bool值时,它会检查流是否有效.有效意味着eof()和bad()都是false.见http://stackoverflow.com/questions/21647/c-reading-from-text-file-until-eof-repeats-last-line#21666

---

与评论相同的精神:而不是`while(true)`,似乎更好写`while(file.good())`

2> 小智..：

---

我喜欢这个例子,现在,省略了你可以在while块中添加的检查:

ifstream iFile("input.txt");        // input.txt has integers, one per line
int x;

while (iFile >> x) 
{
    cerr << x << endl;
}

不确定它有多安全......

---

@harryngh如果我理解正确,`iFile >> x`返回流本身,而不是`x`.然后将流隐式转换为`bool`,测试EOF.

3> wilhelmtell..：

---

还有另一种方法:

#include 
#include 

// ...

    copy(istream_iterator(iFile), istream_iterator(),
         ostream_iterator(cerr, "\n"));

4> Solostaran14..：

---

如果不对原始代码进行太多修改,它可能会变成:

while (!iFile.eof())
{  
    int x;
    iFile >> x;
    if (!iFile.eof()) break;
    cerr << x << endl;
}

但我更喜欢上面的其他两种解决方案.

5> Brian Jack..：

---

EOF模式需要一个素数读取来"引导"EOF检查过程.考虑到第一次读取之前,空文件最初不会设置其EOF.素数读取将捕获此实例中的EOF并正确地完全跳过循环.

你需要记住的是,在第一次尝试读取文件的可用数据之前,你没有得到EOF.读取确切的数据量不会标记EOF.

我应该指出,如果文件是空的,你的代码会被打印,因为EOF会阻止在进入循环时将值设置为x.

0

因此,添加一个素数读取并将循环读取移动到结尾:

int x;

iFile >> x; // prime read here
while (!iFile.eof()) {
    cerr << x << endl;
    iFile >> x;
}