0

I have a while (!feof(infile)) loop, so that exits only when the eof is reached. And it closes all files upon exit. Here's the code:

// scan input file
while (!feof(infile))
{
    // read four bytes
    BYTE byte[4];
    fread(&byte, sizeof(BYTE), 4, infile);
Improve this question
2
  • 1
    I put through a suggested edit to remove most of the code, due to the Honor Code. Thanks. – curiouskiwi♦ Jun 11 '14 at 8:47
  • Sorry, I forgot about that. Thanks. – Pranav Ballaney Jun 11 '14 at 12:44
5

feof is only set after fread tries to read the file and fails. When you reach the end of the file, fread will run one more time and your loop will carry on and write one more block (the last one, again). This will cause your last file to have an extra block in it, and fail check50.

Try to think of another way to know you have reached the end of the file.

Improve this answer
5
  • 1
    I added another later of error checking! Now, inside the while loop, before reading the file, it checks if the eof has been reached. Works like a charm! Thanks! – Pranav Ballaney Jun 11 '14 at 13:16
  • THANK YOU!! Works for me as well :-) – julio Oct 1 '14 at 1:41
  • Phew. Thank you too! – Andrew Venezia Dec 18 '14 at 21:09
  • Super helpful! Thanks! – Justin Goff Jun 30 '15 at 14:07
  • I think I have the same problem, can you please elaborate? – Sushant Bansal Jul 2 '15 at 9:33

You must log in to answer this question.

Not the answer you're looking for? Browse other questions tagged .