Overloading << & >> Operators 1

[DavidTheLazyStudent]

Each time I try to add overloaded input/output I get a compiler message about accessing the class's private data. If I REM out the statments it functions as expected.

My Friend statment is:
friend ostream& operator << (ostream &out, Meal &aMeal);

With Meal being the Class, aMeal is representing an Object of the Class. I believe this is my problem but can not find any solutions in my books. Is aMeal suspose to be only the object name and not just anything I use for refferencing inside my function?

*ostream& operator << (ostream &out, Meal &aMeal)
{
out << aMeal.entree << '\t' << aMeal.calorie << '\n';
return(out);
}
I have the exact same problem on the input.

 

[palbano]

Define the object reference as const and also dont change the return value in the definition signature

class Meal{
... other stuff
  friend ostream& operator <<( ostream& os, const Meal& oMeal);
};

// then in implementation file
ostream& operator <<( ostream& os, const Meal& oMeal){
  os << oMeal.calorie << endl;
  return os;
}

hope that helps
-pete

 

[Cagliostro]

Be attentive on symbols, look at * Ion Filipski

filipski@excite.com

 

[Foada]

FYI
I had a similar problem that was fixed if I used

<iostream.h>

instead of

<iostream>
using namespace std;

I then checked MSDN to find out that the problem was fixed with a VS srvPack. If you choose to battle wits with the witless be prepared to lose.

 

[DavidTheLazyStudent]

Thank You all... that was it foada. I did and it compiled fine. This has been driving me crazy.

(The * was a result of a careless /* not really in my program).

 

[palbano]

Wait a second... you got it to work with the code you posted? That's just not right!

you have the return type declared as ostream& and the definition declared as *ostream&

If that works there is a huge bug in the compiler!

-pete

 

[DavidTheLazyStudent]

No, that was a typo posting. The * was not in my program. I REM'ed it out to test that it would compile without those statements to isolate my problem. I had also made it a const as you had pointed out. But it still didn't compile, untill I used <iostream.h> instead of just plain <iostream> and using namespace std;

It works great, again thanks all.



Now if I could just do something about my short fat fingers.