Destruktor Absturz beim Löschen eines Arrays von Zeigern in C++

Question

Ich machte eine Klasse mit dem Namen cell. Innerhalb dieser Klasse gibt es eine Reihe von cell Zeigern. Der Header sieht wie folgt aus:

class cell 
{  
public: 
    cell(); 
    cell *c[8]; 
    void creatcells(); 
    virtual ~cell(); 
    .. 

} 

und die cpp Datei sieht wie folgt aus:

cell::cell() 
{ 
//ctor 
for(int i=0;i<8;i++) 
{ 
    c[i]=NULL; 
} 

} 


void cell::creatcells() 

{ 
    cell c1,c2,c3,c4,c5,c6,c7,c8; 

    c[0]=&c1; 
    c[1]=&c2; 
    c[2]=&c3; 
    c[3]=&c4; 
    c[4]=&c5; 
    c[5]=&c6; 
    c[6]=&c7; 
    c[7]=&c8; 
} 

cell::~cell() 
{ 
    for(int i=0; i<8; i++) 
    { 
     if (c[i]!=NULL) 
     { 
        delete c[i]; 
     } 
    } 
    delete[] c; 

} 

Aber jedes Mal, endet das Programm, es stürzt ab, warum?
Ich versuchte ohne if (c[i]!=NULL), aber das hilft nicht. Nur ohne die for-Schleife endet der Code einwandfrei, aber ich weiß, dass dies auch gelöscht werden muss. Ich glaube, ich habe den Destruktor korrekt geschrieben, oder?

Answer 1

void cell::creatcells() 
{ 
    cell c1,c2,c3,c4,c5,c6,c7,c8; 

    c[0]=&c1; 
    c[1]=&c2; 
    ... 

Alle oben cell Objekte werden am Ende des createcells() .So delete c[i]; in destructor automatisch zerstört ist UB.What Sie wollen,

c[0]= new cell(); 
c[1]= new cell(); 

Answer 2

Sie versuchen, Zeiger auf und delete Variablen zu dereferenzieren, die automatische Speicherdauer hatten und nicht mehr im Geltungsbereich sind! Hat dich dein Compiler nicht davor gewarnt? Das Verhalten Ihres Programms ist daher undefined.

Sie nur je Paar ein delete[] mit einem new[] und ein delete mit einem new; obwohl Sie die delete (und sogar die new) an eine verwaltete Pointer-Klasse wie std::unique_ptr delegieren können.

Warum refaktorieren Sie nicht auf std::vector<cell> und verwenden Sie Rückgabewertoptimierung?

---

std::make_unique Verwendung.

Answer 3

In cell::createcells Funktion die Variablen lokale und den Gültigkeitsbereich verlassen und werden zerstört, sobald die Funktion zurückkehrt. Diese Objekte sind nicht vorhanden, wenn Sie versuchen, sie zu löschen. Dereferenzieren dieser Zeiger führt zu undefined behavior.

Nicht zu erwähnen, dass Sie nur delete was Sie new. Und da Sie nicht new alles tun delete auf den Zeigern wieder führt zu undefined behavior.

Die einfache Lösung ist ein verwenden vector von Objekte:

std::vector<cell> c; 

Dann fügen Sie einfach acht cell Objekte auf den Vektor:

void cell::creatcells() 
{ 
    c = std::vector<cell>(8); 
} 

nun der Vektor acht Standard ausgebaute enthalten cell Objekte. Keine Notwendigkeit, etwas im Konstruktor oder Destruktor zu tun. Tatsächlich empfehle ich Ihnen, den Konstruktor und den Destruktor vollständig zu entfernen, entsprechend the rule of zero.