if (*this != value)

oder

if (this != &value)

? Ersteres vergleicht wenn ich das richtig sehe die Objekte miteinander sofern der == bzw. != operator vorhanden ist und letzteres schaut ob die Adresse der Objekte identisch ist?

MyClass& MyClass::operator=(const MyClass& value)
{
	if (*this != value)
	{
        ...
	}
	return *this;
}

T& operator = (T other)
{
    swap(other);
    return *this;
}

Don't pessimize prematurely

Du möchtest bei jeder Zuweisung immer und überall eine Zusatzabfrage einbauen, damit du im Sonderfall, der eigentlich gar nicht vorkommen sollte, dir eine Kopie sparen kannst?

Da ist ja statistisch gesehen die Performance mehr am Leiden, da es mehr kostet 3000 mal unnötig zu prüfen als einmal eine Kopie ein paar Skalarer Datentypen durchzuführen. cmon.

a = a;

, aber

*p = *q;

kann schon mal passieren, auch wenn p und q auf das selbe Objekt zeigen. Es ist ja auch nicht so, dass eine Abfrage die Welt kostet - obwohl wenn ich mir manchmal eine portable Variante von gccs __builtin_expect wünsche.

Was man auch nicht unterschlagen sollte: das Copy&Swap-Idiom ist performancetechnisch dann eine gute Sache, wenn Swaps billig zu haben sind, sonst aber recht teuer. Wenn man seine Datentypen generell verpimplt, mag das trivial erscheinen, aber ich habe auch schon Typen gebaut, die ein paar Kilobyte lokal vorliegen hatten.