C / C++ / MFC

George_George wrote:

I agree that there is some issue in my code.

George_George wrote:

It is appreciated if you could provide some pseudo code to correct my code to implement the using swap technique to implement a exception safe assignment operator.

A& A::operator=(const A& other)
{
	A temp(other);
	swap(temp);
	return *this;
}

toxcct wrote:

hey, you want an exception safe copy ?

backup your object,

toxcct wrote:

Yes. but I believe that the state of the parameter only may throw an exception, if its state is corrupted.

toxcct wrote:

so if a is OK, then swapping it with this would be OK too...

George_George wrote:

We are talking about the state of current object instance (this object), why you are talking about the state of parameter?

toxcct wrote:

1) because : if the state of the parameter is corrupted, the state of the current object could have been corrupted too. but as long as we're assigning a temporary object first, the current object's assignment is exception-safe !