Overloading !

[GGWPBOT]

Hello
I need help with overloading '==' and '*'
i cant seem to find the right way to do it, can somebody help please
here is my code:

 #include <iostream>
using namespace std;

class Complax
{
private:
	double real;
	double imag;

public:
	Complax()
	{
		real = 0;
		imag = 0;
	}

	Complax(double x, double y)
	{
		real = x;
		imag = y;

	}

	void setReal(double x)
	{
		real = x;
	}

	void setImag(double x)
	{
		imag = x;
	}

	double getReal()
	{
		return real;
	}

	double getImag()
	{
		return imag;
	}

	Complax operator =(Complax &x)
	{
		real = x.getReal();
		imag = x.getImag();
	}

	Complax operator +(Complax &x)
	{
		Complax temp;
		temp.setReal(x.getReal() + real);
		temp.setImag(x.getImag() + imag);
		return temp;

	}

	Complax operator -(Complax &x)
	{
		Complax temp;
		temp.setReal(x.getReal() + real);
		temp.setImag(x.getImag() + imag);
		return temp;

	}

};



int main()
{
	Complax x(1, 2);
	Complax y(2, 3);
	Complax z, w, v;

	z = x + y;
	w = x - y;
	v = x * y;

	if (z == w)
		cout << " z = w" << endl;
	else
		cout << " z != w" << endl;


	system("PAUSE");
	return 0;
}

[magnum pi]

You weren't specific on what these operators should do. IIRC, Complex numbers multiply/compare equal by doing said operation with their real component and imaginary component. For operator==, make a global function like this:

bool operator==(const Complax& a, const Complax& b) {
   return( a.getReal() == b.getReal() &&
              a.getImag() == b.getImag() );
}

For operator*, see if you can figure it on your own ;) but hint: It's going to look a lot like and follow the same principle as your operators - and +.

Btw, your operator-() is doing a + of your operands. Lastly your operators *, -, +, and = should accept a const reference, that way you can pass const objects and temporary objects, and it prevents you from accidently modifying your param.

[GGWPBOT]

I added some stuff to the code but it's showing me errors in the code and there's no obvious one

#include <iostream>
using namespace std;

class Complax
{
private:
	double real;
	double imag;

public:
	Complax()
	{
		real = 0;
		imag = 0;
	}

	Complax(double x, double y)
	{
		real = x;
		imag = y;

	}

	void setReal(double x)
	{
		real = x;
	}

	void setImag(double x)
	{
		imag = x;
	}

	double getReal()
	{
		return real;
	}

	double getImag()
	{
		return imag;
	}

	Complax operator =(Complax &x)
	{
		real = x.getReal();
		imag = x.getImag();
	}

	Complax operator +(Complax &x)
	{
		Complax temp;
		temp.setReal(x.getReal() + real);
		temp.setImag(x.getImag() + imag);
		return temp;

	}

	Complax operator -(Complax &x)
	{
		Complax temp;
		temp.setReal(x.getReal() + real);
		temp.setImag(x.getImag() + imag);
		return temp;

	}
	Complax operator *(Complax &x)
	{
		Complax temp;
		temp.setReal(x.getReal() * real);
		temp.setImag(x.getImag() * imag);
		return temp;

	}

	bool operator ==(const Complax& a, const Complax& b) {
	
		return (a.getReal() == b.getReal() && a.getImag() == b.getImag());
	}

};



int main()
{
	Complax x(1, 2);
	Complax y(2, 3);
	Complax z, w, v;

	z = x + y;
	w = x - y;
	v = x * y;

	if (z == w)
		cout << " z = w" << endl;
	else
		cout << " z != w" << endl;

	cout << "z= " << z << endl << "w= " << w << endl << "v= " << v;
	system("PAUSE");
	return 0;
}

[magnum pi]

The operator== needs to be a *global* function, ie, outside the body of the class, such as where line 82 is currently. You'll also need a global operator<<(ostream&, const Complax&) function in order to send out z, w, and v on line 100. It could look something like this:

//Return an ostream& to allow chaining of << operators.
ostream& operator<<(ostream& out, const Complax& c)
{
   out << c.getReal() << '+' << c.getImag() << 'i';  //Just my guess on how it should look, feel free to change this if you want it to look different.
   return( out );  //This allows the chaining.
}

And don't forget what I said previously about your operator-(), and the const refs! ;)

[GGWPBOT]

Thank you so much,

it helped me a lot, but i have a question about '=' overload, it says it has to return value, what value it should return ? or my format is wrong ?

Complax operator =(Complax &x)
	{
		real = x.getReal();
		imag = x.getImag();
	}

[cire]

    Complax& operator =(const Complax &x)
    {
        real = x.getReal();
        imag = x.getImag();
        return *this;
    }

Of course, getReal and getImage should be const (as well as operator+ and operator-.)

[GGWPBOT]

Worked!
Thanks a lot guys