Destruktor aus Memberfunktion aufrufen?

[destrict0r]

Ist folgendes erlaubt?

class T
{
   ~T() {foo;}

   bool hallo() {delete this; return true;}
};

[theta]

Ja.

EDIT
Mal abgesehen davon dass nirgens spezifiziert ist, was foo ist.

EDIT2
Und natürlich muss das Objekt auch mit new erzeugt worden sein.
Hier ein wenig ausführlicher:
http://www.parashift.com/c++-faq-lite/freestore-mgmt.html#faq-16.15

[asc]

destrict0r schrieb:

Ist folgendes erlaubt?

Auf Grund der potentiellen Probleme (z.B. delete auf Stackobjekt etc.) bleibt aber die Frage, ob es für deinen Fall nicht etwas besseres gibt. Vielleicht beschreibst du erst einmal wozu du es brauchst.

[destrict0r]

T wird mit new erzeugt.

T ist eine Spieler-Klasse, die mit "t->hallo()" geupdated wird. Wenn sie in "hallo" feststellt, dass sie z.B. mit dem Feind kollidiert ist, soll sie sich selbst löschen.

[Nexus]

destrict0r schrieb:

Wenn sie in "hallo" feststellt, dass sie z.B. mit dem Feind kollidiert ist, soll sie sich selbst löschen.

Ganz selbst löschen kann sie sich eh nicht, da sie wahrscheinlich in einem Container steckt oder durch eine andere Variable im Programm referenziert wird. Also musst du diesen Verweise noch entfernen. Und dann kannst du gleich das delete ausserhalb schreiben.

Generell sollte Anforderung und Allokation möglichst symmetrisch sein. Wenn jemand das Objekt mit new erstellt, sollte er (und nicht irgendwer sonst) es wieder zerstören. Falls du dann mal was änderst, musst du das nur an einer Stelle tun. Oder noch besser, du nimmst gleich Smart-Pointer und brauchst gar kein delete .

[destrict0r]

[quote="Nexus"]

destrict0r schrieb:

Und dann kannst du gleich das delete ausserhalb schreiben.

Und wo außerhalb? Und woher soll das außerhalb wissen, dass der Spieler getroffen wurde?

[destrict0r]

Ich dachte, dass ich einen CollisionObjectManager habe, über den alle Kollisionsobjekte erzeugt/gespeichert/gelöscht werden. Und alle Kollisionsobjekte haben eine "update"-Funktion. Könnte ich nicht aus der die Löschroutine des Managers mit this als Parameter aufrufen?

[fdfdg]

destrict0r schrieb:

Und wo außerhalb? Und woher soll das außerhalb wissen, dass der Spieler getroffen wurde?

boost::ptr_vector<T> myTs;
...
t->hallo();
if ( t->isDead() )
    myTs.erase( t );

Oder so. Irgendwo "außerhalb" hast du ja das Objekt noch gespeichert. Wenn du es in "hallo" destruierst, kriegt der Speichernde das ja nicht mit und versucht weiterhin, darauf zuzugreifen.

[destrict0r]

Oder soll ich lieber in der update-Routine ein Flag des CollisionObjekts "isDeleted" auf true setzen und dann von außen im Manager über alle CollsisionObjects iterieren und sie ggf. löschen?

[drakon]

isDeleted ist ungünstig. Nimm besser etwas, was den Zustand des Objektes beschreibt. Der, der das Objekt erzeugt hat kann dann entscheiden ob er es zerstören möchte oder doch lieber noch behalten.
Das Objekt soll seinem Besitzer nicht vorschreiben was er damit zu tun hat, sondern ihn nur darüber informieren, was mit ihm los ist. Was dann damit passieren soll liegt (sowieso) beim Besitzer.

[destrict0r]

Also sowas wie "isDestroyed"? Dann ist mein Vorschlag OK?

[volkard]

isDead oder isAlive sieht man alle Nase lang.

[fdfdg]

hasCollided könnte doch auch gehen, wenn das die Semantik ist. Manche Sachen überstehen ja vielleicht auch mehrere Kollisionen.

[volkard]

fdfdg schrieb:

hasCollided könnte doch auch gehen, wenn das die Semantik ist. Manche Sachen überstehen ja vielleicht auch mehrere Kollisionen.

Das verbessert nur wenig.

boost::ptr_vector<T> myTs;
...
t-> hallo( );
if (  t-> hasCollided( )  )
    if (  t-> isDead( )  )
        myTs. erase( t );

[drakon]

Collided ist auch nicht gut. Das ist eher ein Event. Denn normalerweise passieren ständig Kollisionen (mit dem Bode, einem Mitspieler, einem Objekt usw.). Das bedeutet in den meisten Fällen noch nicht, dass das Objekt gestorben ist. isDead, isAlive finde ich gut.

[destrict0r]

Was ist der Vorteil von ptr_vector<T> gegenüber vector<T*>?

[314159265358979]

Er ruft delete für dich auf.

[Nexus]

destrict0r schrieb:

Was ist der Vorteil von ptr_vector<T> gegenüber vector<T*>?

Automatische Aufräumung (kein delete notwendig), Exceptionsicherheit und direkte Dereferenzierung ( *itr ist T& und nicht T* ).

Ausführlich sind die Vorteile hier aufgelistet.

[destrict0r]

314159265358979 schrieb:

Er ruft delete für dich auf.

Wann?

[fdfdg]

destrict0r schrieb:

314159265358979 schrieb:

Er ruft delete für dich auf.

Wann?

Sobald du das Objekt aus dem Container entfernst, mit container.erase(iter) .