Teachers code aint right

This is a discussion on Teachers code aint right within the C++ Programming forums, part of the General Programming Boards category; RoD, I would say that her "fix" is the least good of the ones I've seen here. It most certainly ...

  1. #16
    Registered User
    Join Date
    Mar 2003
    Posts
    2
    RoD, I would say that her "fix" is the least good of the ones I've seen here. It most certainly ain't the standard way of doing it. The cleanest, most standard and best way do do it would be (as others have already pointed out) while(infile >> data) {...}or while(!infile.eof()) { infile >> data; ...}. In other words, don't listen to her.

  2. #17
    End Of Line Hammer's Avatar
    Join Date
    Apr 2002
    Posts
    6,231
    >>In other words, don't listen to her.
    >>while(!infile.eof()) { infile >> data; ...}
    I'm afraid your advice isn't much better. Controlling the loop with !eof is not the way to do things, as already highlighted.
    When all else fails, read the instructions.
    If you're posting code, use code tags: [code] /* insert code here */ [/code]

  3. #18
    RoD
    RoD is offline
    Redundantly Redundant RoD's Avatar
    Join Date
    Sep 2002
    Location
    Missouri
    Posts
    6,331
    >>In other words, don't listen to her.


    Out of the question. She is a teacher of a field i am trying to learn, anything she says has at least a certain degree of benifit or validitity that it carries with it.

  4. #19
    Registered User Codeplug's Avatar
    Join Date
    Mar 2003
    Posts
    4,676
    I went into every one of my CS classes having already learned the subject matter. What ever else you pick up from the class is a bonus.

    gg

  5. #20
    Registered User
    Join Date
    Mar 2003
    Posts
    2
    Hammer, about infile.eof(); I didn't notice that the file was ended with endl. If there hadn't been a newline charater at the end of the file eof() would work. Sorry about that, should have read the code more careful. However, eof() should be prefered over using fail() in a case like this (even if it, like you said, won't work in this particular case), don't you agree? It seems kind of dumb to wait for something to fail before you terminate it.
    RoD, I didn't suggest you should never listen to her (even though she deserves it after kicking you out of the class room), just not in this case since she is obviously wrong.
    Last edited by ArchMiffo; 03-16-2003 at 04:29 AM.

  6. #21
    Registered User
    Join Date
    Mar 2002
    Posts
    1,595
    IMO, any code that reads the last line of a file twice is wrong. Whether you use fail() or eof() or != flag, seems immaterial.

    As Hammer points out in his side post, using C code, but that's okay, fail() and eof() don't check the stream/streambuf directly, they check the state variable of the stream. In this case when the ifstream finds EOF it flips the EOF bit of the state variable; but finding EOF doesn't cause an istream to fail, therefore the fail bit isn't flipped. Therefore the loop continues for one last cycle, although in an undetermined way--often by repeating the last item read in, but you can't count on that.

    IMO, another reason the initial approach is wrong is that if the program continues beyond this snippet and you attempt to use the ifstream again, say to read in from another file, without clearing the fail bit using clear(), then the stream will contiue to fail indefinately.

Page 2 of 2 FirstFirst 12
Popular pages Recent additions subscribe to a feed

Similar Threads

  1. Enforcing Machine Code Restrictions?
    By SMurf in forum Tech Board
    Replies: 21
    Last Post: 03-30-2009, 07:34 AM
  2. Values changing without reason?
    By subtled in forum C Programming
    Replies: 2
    Last Post: 04-19-2007, 10:20 AM
  3. Updated sound engine code
    By VirtualAce in forum Game Programming
    Replies: 8
    Last Post: 11-18-2004, 11:38 AM
  4. Interface Question
    By smog890 in forum C Programming
    Replies: 11
    Last Post: 06-03-2002, 05:06 PM
  5. Replies: 0
    Last Post: 02-21-2002, 05:05 PM

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21