Ist es sicher, das zu löschen?

Question

In meinen ersten grundlegenden Tests ist es vollkommen sicher, dies zu tun. Es ist mir jedoch aufgefallen, dass der Versuch, this später in einer Funktion zu manipulieren, dass delete s this ein Laufzeitfehler sein könnte. Ist das wahr und ist es normalerweise sicher zu delete this? oder gibt es nur bestimmte Fälle, in denen es sicher ist?

Answer 1

delete this ist legal und tut was Sie erwarten würden: es ruft den Destruktor Ihrer Klasse auf und befreit den zugrunde liegenden Speicher. Nach delete this zurückgibt, Ihr this Zeiger Wert nicht ändern, so dass es jetzt ein dangling Zeiger ist, sollte nicht dereferenziert werden. Dies schließt die implizite Dereferenzierung mit den Membervariablen der Klasse ein.

Es wird in der Regel in Bezug gezählt Klassen gefunden, wenn die Referenzzählung auf 0 verringert wird, die DecrementRefCount()/Release()/was auch immer Memberfunktion aufruft delete this.

delete this wird aus vielen Gründen als sehr schlechte Form angesehen. Es ist leicht, nach delete this versehentlich auf Mitgliedsvariablen zuzugreifen. Der Anrufercode erkennt möglicherweise nicht, dass sich Ihr Objekt selbst zerstört hat.

Auch delete this ist ein "Code-Geruch", dass Ihr Code möglicherweise keine symmetrische Strategie für Objektbesitz hat (wer zuweist und wer löscht). Ein Objekt konnte sich new nicht selbst zugewiesen haben, also bedeutet Aufruf delete this, dass Klasse A ein Objekt zuweist, aber Klasse B gibt es später frei [self].

Answer 2

Ja. Es sollte vollkommen in Ordnung sein. "Dies" ist nur ein Zeiger. Jeder Zeiger wird für das Löschen tun. Die Informationen zum Löschen eines Objekts sind in den Heap-Datensätzen enthalten. So wird IUnknown :: Release() normalerweise in COM-Objekten implementiert.

Answer 3

Es ist sicher, "this" zu löschen, solange es im Wesentlichen die letzte Operation in der Methode ist. In der Tat tun dies mehrere professionelle APIs (siehe die CComObject-Implementierung von ATL für ein Beispiel).

Die einzige Gefahr besteht darin, nach dem Aufruf von "delete this" auf andere Mitgliederdaten zuzugreifen. Dies ist sicherlich unsicher.

Answer 4

Read für eine ähnliche Diskussion. Ihr Verständnis ist richtig, dass es funktioniert, benötigt wird und gefährlich sein kann, da Sie danach nicht darauf zugreifen können.

Answer 5

löschen Dies kann zu einem Problem führen, wenn Sie Unterklassen des Objekts haben, das Sie löschen. Denken Sie daran, dass die Konstruktion von oben nach unten beginnt und die Löschung von unten beginnt. Wenn Sie also in der Mitte der Hierarchie löschen, haben Sie im Grunde alle Objekte unter dieser bestimmten Klasse verloren.

löschen Dies ist sehr praktisch, wenn Sie ein Objekt mit Referenzzählung implementieren, ein Beispiel dafür sind die COM-Klassen.

Answer 6

Löschen Sie dies ist völlig legal, wie andere bereits erwähnt haben. Es ist riskant für einen zusätzlichen Grund, der noch nicht erwähnt wurde - Sie gehen davon aus, dass das Objekt auf dem Heap zugeordnet wurde. Dies kann schwierig zu garantieren sein, obwohl im Falle von Referenzzählungsimplementierungen im Allgemeinen kein Problem besteht.

Answer 7

Wie von anderen angegeben, löschen Sie dies ist ein gültiges Idiom, aber damit es sicher ist, müssen Sie sicherstellen, dass das Objekt nie auf dem Stapel instanziiert wird.

Eine Möglichkeit, dies zu tun, ist sowohl der Konstruktor und der Destruktor privaten und erzwingen Objekterstellung durch eine Klasse Factory-Funktion zu machen, die auf dem Heap den das Objekt erstellt und gibt einen Zeiger auf sie. Die Klassenfabrik kann eine statische Elementfunktion oder eine Freundfunktion sein. Die Bereinigung kann dann über eine Delete() - Methode für das Objekt durchgeführt werden, das "delete this" ausführt. COM-Objekte funktionieren grundsätzlich so, außer dass sie zusätzlich mit dem "delete this" -Ziffer gezählt werden, der auftritt, wenn der Referenzzähler auf Null dekrementiert wird.

Answer 8

aber tun Sie es nicht im Destruktor!

Answer 9

Legal Ja
Sicher Nein

Answer 10

Wenn Sie von einer Basisklasse erben und gab diese in der Funktion der Basisklasse löschen, abgeleitete Klasse Zeiger verwendet, wird zu einem Absturz führen. Z. B:

class Base 
{ 

    virtual void Release() 
    { 
     delete this; 
    } 

} 

class Derived : public Base 
{ 

    void Foo() 
    { 
     ... 
    } 

} 

main() 
{ 

    Base *ptrDerived = new Derived(); 
    ptrDerived->release(); 
    ptrDerived->Foo() //Crash 

} 

Answer 11

Es gilt schlechte Praxisdelete this zu verwenden.

jedoch, falls verwendet, müssen folgende Punkte berücksichtigt werden:

1. Der delete Operator nur für Objekte arbeitet new Betreiber zugewiesen werden. Wenn das Objekt mit new erstellt wird, können wir delete this tun, sonst ist das Verhalten undefiniert.

class A { 
    public: 
    void fun(){ 
    delete this; 
    } 
    }; 

int main(){ 
    /* Following is valid */ 
    A *ptr = new A; 
    ptr->fun(); 
    ptr = NULL // make ptr NULL to make sure that things are not accessed using ptr. 


    /* And following is invalid: Undefined Behavior */ 
    A a; 
    a.fun(); 

    return 0; 
} 

2. Sobald delete this getan wird, wird jedes Mitglied des gelöschten Objekts sollte nicht nach dem Löschen zugegriffen werden.

class A { 
    int x; 
    public: 
    A() { 
     x = 0; 
    } 
    void fun() { 
     delete this; 

    /* Invalid: Undefined Behavior */ 
    cout<<x; 
    } 
};