Thread: Teachers code aint right

The code is simple, and its not an assignment. She says theres nothing wrong with it, and i totally disagree. It outputs a second copy of the last digit entered, and i think it lies in the first while loop. Example, say i enter

1
2
3
-999

According to theory this places 1, 2, and 3 into the file lined as shown above, and -999 quits that while loop. This is where the problem is. Now if i remeber there is a marker of some sort added to the end of a file to mark the end, could this extra 3 be that marker? Or is the loop screwey? She says its supposed to do that, that she made it that way. I say the code is errorous but im not sure why. I've never used ifstream and ofstream this way before, so i really dont know.

My beef with this is shes teaching the other students to this code saying its right and i say its errorous, and it ........es me off that she cant explain to me how her design calls for this. She said, and i quote "it doesnt matter how u alter it, its c++ doing it".

Bull$$$$....please look at this and help me out, i cant prove her wrong alone lol. Also note the way we have been doing it is to NOT have a pre-created file when its run, just let it do it. The file also cannot be specified to .txt/.rtf and so on, no extensions.

Code:

#include "stdafx.h"
#include <iostream>
#include <fstream>

using namespace std;

const int SENTINEL = -999;

int main(int argc, char* argv[])
{
	int data;
	ofstream outfile;
	ifstream infile;

	outfile.open("c:\myfile");

	cout << "Enter an integer or -999 to quit: ";
	cin >> data;

	while (data != SENTINEL)
	{
		outfile << data << endl;
		cout << "Enter an integer or -999 to quit: ";
		cin >> data;
	}

	outfile.close();

	infile.open("c:\myfile");

	while (! infile.fail())
	{
		infile >> data;

		cout << data;
	}

	return 0;
}

I don't see anything wrong with either of the while loops. What part of the code is bothering you?

gg

Its not here, or at school. Two different versions of VC++, heres the output i get:

Code:

Enter an integer or -999 to quit: 1
Enter an integer or -999 to quit: 2
Enter an integer or -999 to quit: 3
Enter an integer or -999 to quit: -999
1233Press any key to continue

Notice that extra three?? it should be 123 not 1233....

Thanks, makes sense now : ). In school she has us writing it to the a:\ drive i just substituted it out for c:\.

Thanks again.

Yep, that one still burns me now and then.
Its better like this:

Code:

while (!(infile >> data).fail())
{
   cout << data;
}

gg

she told me the if way was "wrong" and did it this way:

Code:

infile.open("c:\myfile");

infile >> data; //prime it

	while (! infile.fail())
	{
	    cout << data << endl;
                    infile >> data;
	}

	return 0;

what gives?

why not just do it this way:

Code:

while(infile >> data) {
    cout << data << endl;
}

i showed her that too, she says her way is the standard...wtf is wrong with this woman.

Today i got kicked out for asking her to open a window. Its warm out, hot in there, and they were doing CONSTRUCTION in the area in back of the room, making it loud and dusty as $$$$. The dusts a health hazard and she kicked me out for my "cocky attitude". I SWEAR this woman CANT be WRONG....

God im so ........ed off....

a couple or so months, depending when you graduate. but, wow, she should have opened a window.

She teaches it like its a class and nothing more to her, yet she seems to enjoy the error-fixing end when its not her error. I honestly think she feels a little compromised by having someone whos really into c++ in there. You see shes used to epople just using it to fill a class slot, and not knowing anything. Then here i am someone whos right with her on most things, and usually quicker to realize a problem when debugging something.

I wasnt trying to prove her wrong in the sense that it first comes across as, i was trying to prove her wrong so that i could understand why it was(n't) right. See if she had just even tried to offer some sort of explanation as too why it was "c++'s fault" i probably would have let it go, but im there to learn, not to take some junk screwey answer as knowledge.

By working with you guys i have no learned a little bit more than the scope of her lesson offered. I'll never know why she wouldnt work with me here, but she wouldnt. I can live with that knowing that i got my answer as to why.

The only question i've left to ask is this: What reason, if any, places her fix on a higher scale then the one you guys showed me? And one u cant answer: If its c++'s fault, why did she go home and make a fix?

She's a doofus, that's why.
Once I was subbing at a local school. On my break, I was reading in the library. About 6 or 7 junior high kids were in there working on computers for some assignment. The kids hit the print key, which started printing a 30 or 40 page document.
Thecrabby old librarian got cranked over that, started yelling at the kids to not hit print, they don't need to print it, etc, really being nasty about it. She shut the printer off to stop it.
Well, of course when she turned it on it started printing again because all the print jobs were still in the print queue. She yelled at the kids again, and shut the printer off. This happened about 3 times.
Finally I went up to her and quietly and politely told her she'd have to go into printer setup and delete the print jobs. She gave me a rabid skanky look and hissed "I know that!" Then why the f**k didn't she do it?? I'm so glad my kids don't go to that school system.