<![CDATA[eigenes Objekt überschreiben]]>Hi,

ich habe eine Bildklasse die ein rotate() Methode hat. Da die rotate-Funktion nicht in-place ist, hab ich mir gedacht ich erstell ein neues Bildobjekt, schreib da das neue Bild rein und überschreib dann das aktuelle Objekt mit dem neuen.
Ungefähr so:

void Bild::rotate(int deg)
{
   //rotieren...
   Bild NeuesBild;
   //rotieren...

   //this = &NeuesBild; //error: non-lvalue in assignment
   *this = NeuesBild; //kein Kompilierfehler
}

Allerdings bin ich mir jetzt nicht sicher, was für Fehler so auftreten können.
Oder macht man sowas ganz anders?

MfG Caster

]]>https://www.c-plusplus.net/forum/topic/231308/eigenes-objekt-überschreibenRSS for NodeFri, 25 Sep 2026 21:48:28 GMTThu, 08 Jan 2009 17:21:49 GMT60<![CDATA[Reply to eigenes Objekt überschreiben on Thu, 08 Jan 2009 17:21:49 GMT]]>Hi,

ich habe eine Bildklasse die ein rotate() Methode hat. Da die rotate-Funktion nicht in-place ist, hab ich mir gedacht ich erstell ein neues Bildobjekt, schreib da das neue Bild rein und überschreib dann das aktuelle Objekt mit dem neuen.
Ungefähr so:

void Bild::rotate(int deg)
{
   //rotieren...
   Bild NeuesBild;
   //rotieren...

   //this = &NeuesBild; //error: non-lvalue in assignment
   *this = NeuesBild; //kein Kompilierfehler
}

Allerdings bin ich mir jetzt nicht sicher, was für Fehler so auftreten können.
Oder macht man sowas ganz anders?

MfG Caster

]]>https://www.c-plusplus.net/forum/post/1641951https://www.c-plusplus.net/forum/post/1641951Thu, 08 Jan 2009 17:21:49 GMT<![CDATA[Reply to eigenes Objekt überschreiben on Thu, 08 Jan 2009 17:49:50 GMT]]>Sofern der Kopierkonstruktor keine Probleme macht, sollte das so schon funktionieren. Auch wenn ich es ein wenig übertrieben finde jedesmal, wenn du eine Rotation machst ein neues Bild zu erzeugen. (Kommt allerdings auf deine Möglichkeiten an, ob du das wirklich so machen musst..)

]]>https://www.c-plusplus.net/forum/post/1641967https://www.c-plusplus.net/forum/post/1641967Thu, 08 Jan 2009 17:49:50 GMT<![CDATA[Reply to eigenes Objekt überschreiben on Thu, 08 Jan 2009 20:19:56 GMT]]>Jetzt hab ich aber immer noch ein Problem. Das eigentliche Bild ist eine Bitmap in einem dynamisch angelegtem Speicher. Ich hab keinen Kopier- bzw. in dem Fall eher Zuweisungsoperator (oder?) implementiert, da ich mir gedacht hab, dass es langt wenn ich den Zeiger auf den Speicher kopier, da ich das neu angelegte Objekt ja nicht mehr benötige. Allerdings wird nach nach der Zuweisung (der Überschreibung des alten Objektes) der Destruktor des neuen Objektes aufgerufen, da dieses ja innerhalb dieser Funktion instanziert wurde. Der Destruktor "free'd" jetzt aber den Speicher auf den der (kopierte) Zeiger weist und das neu erzeugt Bild ist somit weg.
Kann ich den Aufruf des Desktruktors irgendwie verhindern?

MfG Caster

EDIT: Mir fällt grad ein, ich könnte ja einfach die Instanzen zählen und erst nach der letzen den Speicher freimachen.

]]>https://www.c-plusplus.net/forum/post/1642032https://www.c-plusplus.net/forum/post/1642032Thu, 08 Jan 2009 20:19:56 GMT<![CDATA[Reply to eigenes Objekt überschreiben on Thu, 08 Jan 2009 20:08:57 GMT]]>Caster schrieb:

Jetzt hab ich aber immer noch ein Problem. Das eigentliche Bild ist eine Bitmap in einem dynamisch angelegtem Speicher. Ich hab keinen Kopier- bzw. in dem Fall eher Zuweisungsoperator implementiert, da ich mir gedacht hab, dass es langt wenn ich den Zeiger auf den Speicher kopier, da ich das neu angelegte Objekt ja nicht mehr benötige. Allerdings wird nach nach der Zuweisung (der Überschreibung des alten Objektes) der Destruktor des neuen Objektes aufgerufen, da dieses ja innerhalb dieser Funktion instanziert wurde. Der Destruktor "free'd" jetzt aber den Speicher auf den der (kopierte) Zeiger weist und das neu erzeugt Bild ist somit weg.
Kann ich den Aufruf des Desktruktors irgendwie verhindern?

MfG Caster

Statt dessen solltest du eher einen korrekten Copy-Ctor/Assignment Operator implementieren, der tatsächlich die Daten auch kopiert. Dein Ansatz mit dem verhindern des Destruktoraufrufs ist völlig falsches Design.

]]>https://www.c-plusplus.net/forum/post/1642040https://www.c-plusplus.net/forum/post/1642040Thu, 08 Jan 2009 20:08:57 GMT<![CDATA[Reply to eigenes Objekt überschreiben on Thu, 08 Jan 2009 20:29:21 GMT]]>Aber so würde ich umsonst ein vl. 1MB großes Bild kopieren.
Und zu der Sache mit dem Instanzen zählen: da müsste ich das inkrementieren des Zählers ja auch in dem Zuweisungsoperator machen und den Standard-Zuweisungoperator wieder selber implementieren... oh man.

]]>https://www.c-plusplus.net/forum/post/1642048https://www.c-plusplus.net/forum/post/1642048Thu, 08 Jan 2009 20:29:21 GMT<![CDATA[Reply to eigenes Objekt überschreiben on Thu, 08 Jan 2009 20:48:06 GMT]]>Wenn Du ein natürliches Design ohne Referenzzählung haben möchtest, wie wäre folgendes:

  • eine Methode void Bild::swap(Bild& other) , die nur den Bilddatenzeiger mit other tauscht
  • void rotate(Bild& bild, int deg) ist eine freie Funktion, die ein neues Bild anlegt, das Rotationsergebnis dort ablegt und dann bild.swap(neuesBild) macht (den swap mit this finde ich - komisch)
  • (opt.) der Copy-Ctor kopiert das gesamte Bild (tiefe Kopie)
  • (opt.) der Zuweisungsoperator nutzt Copy-and-Swap (s.a. Google)

Die letzten beiden Punkte würden das Gesamtild imho noch abrunden, natürliches Kopierverhalten eben (es seidenn Du möchtest Referenzzählung, dann nimm am besten gleich std::tr1::shared_ptr 🙂 ).

]]>https://www.c-plusplus.net/forum/post/1642064https://www.c-plusplus.net/forum/post/1642064Thu, 08 Jan 2009 20:48:06 GMT<![CDATA[Reply to eigenes Objekt überschreiben on Thu, 08 Jan 2009 21:50:00 GMT]]>Ich hab mit jetzt mal den Quellcode von "CxImage" angeschaut und die machen das eigentlich auch nicht viel anders als ich; nur sie rufen manuell eine Destroy-Methode auf anstatt den Speicher im Destruktor freizugeben.

@LordJaxom: Deine Idee ist aber auch nicht schlecht.

]]>https://www.c-plusplus.net/forum/post/1642097https://www.c-plusplus.net/forum/post/1642097Thu, 08 Jan 2009 21:50:00 GMT