Introduction to Programming with c++ 13-7 eof的置位时机

问题

考虑如下的问题，文件内容的复制。

思路

枚举字符的方式实现。
有下面的代码：

int copy_file1( const std::string& src, const std::string& tgt ){    if( src == "" || tgt == "" )    {           std::cerr << "Invalid argumnts!" << std::endl;        return -1;    }    std::ifstream fin;    fin.open( src.c_str() );    if( !fin.is_open() )    {        std::cerr << "Can not open the file!" << std::endl;        return -1;    }    std::ofstream fout;    fout.open( tgt.c_str() );    if( !fout.is_open() )    {        std::cerr << "Can not open the file!" << std::endl;        return -1;    }    char ch;    while( fin.get(ch) ) fout.put(ch);    fin.close();    fout.close();    return 0;}

给出源文件：input.dat

abcd

可得如下的输出文件：output.dat

abcd

---

但是，如果修改上面的代码为下面的代码：

int copy_file2( const std::string& src, const std::string& tgt ){    if( src == "" || tgt == "" )    {           std::cerr << "Invalid argumnts!" << std::endl;        return -1;    }    std::ifstream fin;    fin.open( src.c_str() );    if( !fin.is_open() )    {        std::cerr << "Can not open the file!" << std::endl;        return -1;    }    std::ofstream fout;    fout.open( tgt.c_str() );    if( !fout.is_open() )    {        std::cerr << "Can not open the file!" << std::endl;        return -1;    }    char ch;    while( !fin.eof() ) // 循环结束的条件修改    {        ch = fin.get();        fout.put(ch);    }    fin.close();    fout.close();    return 0;}

则会得到如下的输出文件：output.dat

abcd

也就是会多一个原本没有的字符出现。这是为什么？我对上面的代码增加了文件指针位置的显示，输出如下图。

我们可以看到，文件指针的位置和我们的想法一样。初始为0，最后一个字符读完之后是4。但是问题是，文件指针明明已经位于文件结束的位置，为什么循环还是会执行一次？

这就是说到了eof置位的时机，在最后一个字符读取完毕之后，文件指针是位于文件结束的位置。但是如果测试eof函数，还是返回false.只有当试图再次读取数据时，发现在文件结束位置进行读取，此时才会eof函数才会置true。

所以，上面的第二段代码之所以会多读入一个非法字符就是因为，最后一个字符读取完毕之后，eof还是false。所以，尝试再次读取，此时eof置true。由于没有判断，直接写入，才导致错误。

第一段代码没错是因为，最后一个字符读取完毕之后。循环的条件里面再次进行了读取，所以置位导致无法读取。

所以，再次强调。eof函数的置位时机并不是当文件指针位于文件结束位置时，而是在文件结束位置进行读取时才会置位。

下面给出全部代码实现

#include <iostream>#include <fstream>#include <string>int copy_file(  const std::string& src,  const std::string& tgt );int copy_file1(  const std::string& src,  const std::string& tgt );int copy_file2(  const std::string& src ,  const std::string& tgt );int main( void ){    std::string input_file_name;    std::string output_file_name;    std::cout << "Enter a source file name: ";    std::cin >> input_file_name;    std::cout << "Enter a target file name: ";    std::cin >> output_file_name;    copy_file2( input_file_name, output_file_name );    std::cout << "Copy Done!" << std::endl;    return 0;}int copy_file( const std::string& src,  const std::string& tgt ){    if( src == "" || tgt == "" )    {           std::cerr << "Invalid argumnts!" << std::endl;        return -1;    }    std::ifstream fin;    fin.open( src.c_str() );    if( !fin.is_open() )    {        std::cerr << "Can not open the file!" << std::endl;        return -1;    }    std::ofstream fout;    fout.open( tgt.c_str() );    if( !fout.is_open() )    {        std::cerr << "Can not open the file!" << std::endl;        return -1;    }    std::string line;    while( std::getline( fin, line ) )     {        fout << line;        if( !fin.eof() )            fout << std::endl;    }    fin.close();    fout.close();    return 0;}int copy_file1( const std::string& src, const std::string& tgt ){    if( src == "" || tgt == "" )    {           std::cerr << "Invalid argumnts!" << std::endl;        return -1;    }    std::ifstream fin;    fin.open( src.c_str() );    if( !fin.is_open() )    {        std::cerr << "Can not open the file!" << std::endl;        return -1;    }    std::ofstream fout;    fout.open( tgt.c_str() );    if( !fout.is_open() )    {        std::cerr << "Can not open the file!" << std::endl;        return -1;    }    char ch;    while( fin.get(ch) ) fout.put(ch);    fin.close();    fout.close();    return 0;}int copy_file2( const std::string& src, const std::string& tgt ){    if( src == "" || tgt == "" )    {           std::cerr << "Invalid argumnts!" << std::endl;        return -1;    }    std::ifstream fin;    fin.open( src.c_str() );    if( !fin.is_open() )    {        std::cerr << "Can not open the file!" << std::endl;        return -1;    }    std::ofstream fout;    fout.open( tgt.c_str() );    if( !fout.is_open() )    {        std::cerr << "Can not open the file!" << std::endl;        return -1;    }    while( !fin.eof())     {        std::cout << "file pointer before read : " << fin.tellg() << std::endl;;        char ch = fin.get();        fout.put(ch);        std::cout << "file pointer after read : " << fin.tellg() << std::endl; ;    }    fin.close();    fout.close();    return 0;}