C++でポインタの配列を削除するとデストラクタがクラッシュする

Question

cellという名前のクラスを作成しました。このクラスの中にはcellポインタの配列があります。ヘッダーは次のようになります。

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

} 

とcppファイルには、次のようになります。

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; 

} 

しかし、プログラムが終了するたびに、それがなぜ、クラッシュ？
私はif (c[i]!=NULL)なしで試しましたが、これは役に立ちません。 forループがなければ、コードは完全に終了しますが、これも削除する必要があることを私は知っています。 デストラクタを正しく書いたと思いますか？

Answer 1

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

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

上記cellオブジェクトのすべてが自動的にデストラクタでcreatecells()。だからdelete c[i];の終わりに破壊されているが、あなたがしたいUB.Whatある

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

Answer 2

自動保存期間があり、もはや有効範囲にない変数deleteへのポインタを逆参照しようとしています。あなたのコンパイラはこれについてあなたに警告しませんでしたか？

したがって、プログラムの動作はで、定義されていないのはです。

あなただけこれまでペアnew[]とdelete[]とnewとdelete。 delete（さらにはnew ）をstd::unique_ptrのような管理されたポインタクラスに委任できます。

std::vector<cell>にリファクタリングして戻り値の最適化を使用するのはなぜですか？ std::make_uniqueを使用して

---

。

Answer 3

機能cell::createcellsで変数がローカルですされており、スコープの外に出て、関数が復帰すると破壊されます。これらのオブジェクトは、削除しようとすると存在しません。これらのポインタを参照解除するとundefined behaviorになります。

deleteあなただけがnewであることは言うまでもありません。あなたはnew何もしていないので、ののポインタにdeleteがあります。undefined behaviorにつながります。

シンプルなソリューションを使用することですvectorオブジェクトの：

std::vector<cell> c; 

そして、単にベクトルに8つのcellオブジェクトを追加します。ベクトルが含まれています今

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

8デフォルトで構築cellオブジェクト。コンストラクタやデストラクタで何もする必要はありません。実際にはこれを使って、コンストラクタとデストラクタをthe rule of zeroに従って完全に削除することをお勧めします。