Checking end of file inside a loop condition bad practice

suggest change

eof returns true only after reading the end of file. It does NOT indicate that the next read will be the end of stream.

while (!f.eof())
{
  // Everything is OK

  f >> buffer;

  // What if *only* now the eof / fail bit is set?

  /* Use `buffer` */
}

You could correctly write:

while (!f.eof()) 
{  
  f >> buffer >> std::ws;

  if (f.fail())
    break;

  /* Use `buffer` */
}

but

while (f >> buffer)
{
  /* Use `buffer` */
}

is simpler and less error prone.

Further references:

Feedback about page:

Feedback:
Optional: your email if you want me to get back to you:


File I/O:
* Checking end of file inside a loop condition bad practice

Table Of Contents
8 Arrays
11 Loops
38 File I/O
39 Streams
51 Unions
56 Lambdas
60 SFINAE
62 RAII
67 Sorting
84 RTTI
87 Scopes
104 Profiling
107 Recursion
117 Iteration
125 Alignment
134 Semaphore
136 Debugging
139 Mutexes
142 decltype