Sign in to follow this  
atRoot

Can't overload the operator

Recommended Posts

atRoot    146
As the title says, I'm having a problem overloading the << operator. It occurs in this header:
#ifndef COORD_H
#define COORD_H

// Class Coord
// Descr:    A coordinate.
// Members:  value -> The value of the coordinate.
// Func:    -Assignment through a Coord and an integer.
//          -Equality check based on the values of the coordinates.

#include <iostream>

class Coord {
  public:
	  // Constructor: initializes 'value'
	Coord(int v = 0): value(v) {}
	  // Copy constructor
	Coord(const Coord& c);
	  // Assigns 'c.value' to 'value'
	Coord& operator=(const Coord& c);
	  // Assigns 'i' to 'value'
	Coord& operator=(const int i);
	  // Returns whether 'c.value' equals 'value'
	bool operator==(const Coord& c) const;    
	  // Writes 'value' into 'os' (for testing purposes)
	std::ostream& operator<<(std::ostream& os, const Coord& c) const;   

	  // Getter for 'value'
	int Value() const;
	  // Setter for 'value'
	void SetValue(int i);

  private:
	int value;
};

#endif

When I try to compile, I get the following error: error C2804: binary 'operator <<' has too many parameters I understand what the error message is telling me, but I don't get why it's even there. My book appears to overload the operator in the exact same way. So why does this error occur and how do I go about fixing it?

Share this post


Link to post
Share on other sites
Washu    7829
I doubt your book has it as a class member, but more likely as a free function. As it is, your current overload is taking 3 arguments, one implicit and two explicit.

Share this post


Link to post
Share on other sites
Brother Bob    10344
You must make it a free function, not a member fuction. It cannot be a member function of the Coord class, because it will be called on the stream object.

foo << myCoord

The above is equal to one of these two, depending on which one is defined.

foo.operator <<(myCoord); // (1)
operator <<(foo, myCoord); // (2)

Since (1) requires modification of the stream class, you cannot do it that way. Only the free function form remains. Move the declaration you have at the moment out of the class, and make it friend if necessary.

edit: Some fixes... and too slow.

Share this post


Link to post
Share on other sites

Create an account or sign in to comment

You need to be a member in order to leave a comment

Create an account

Sign up for a new account in our community. It's easy!

Register a new account

Sign in

Already have an account? Sign in here.

Sign In Now

Sign in to follow this