C++ newとdelete

Question

私はメモリを割り当てましたが、デストラクタを呼び出すと、セグメンテーション違反が発生します。 これがコードです。私は記憶を解放するために正しいものを使用していますか？

class plan { 
    char *symbol; 
    gro *grow; 
    public: 
    plan (int, char[]); //constructor 
    ~plan (); //destructor 
}; 

plan::plan (int num_of_sm, char sm[]){ 
    try { 
    symbol = new char [strlen(sm) + 1]; 
    } 
    catch (std::bad_alloc) { 
    symbol = NULL; 
    } 


    if (symbol != NULL) { 
    if (sm == NULL) { 
     strcpy (symbol, ""); 
    } 
    else { 
     strcpy (symbol, sm); 
    } 
    } 
    gro = new grow [num_of_sm]; 
} 

plan::~plan(){ 
    delete [ ] symbol; 
    delete [ ] gro; 
} 

Answer 1

std::stringを使用してください。

それが明らかでない場合には、問題が処理されます。

また、同様の問題を回避し、問題の詳細をもう少し理解するためには、3のルールを参照するか、C++ 11のルール5を参照してください。

Answer 2

メモリの問題は、プログラムが常にすぐにクラッシュするとは限らないため、非常に扱いにくいことがあります。たとえば、同じポインタを2回削除すると、プログラムがクラッシュする場所を削除するまで、プログラムは正常に続行されます。もしあなたがLinux上で走っているならば、valgrindを実行することをお勧めします（例えばvalgrindの後に通常のコマンドが続くと仮定します）。他の削除/解放文に問題がないか調べてみてください。また、num_of_smは妥当な値ですか？

Answer 3

私がデストラクタを呼び出すと、セグメンテーションエラーが発生します。もしそうなら

あなたは（怒鳴る）

int main() 
{ 
    plan a_plan(2, "hello"); 
    a_plan.~plan(); 
} 

をテストコードをもしかして、私はこの問題は、あなたが二回のポインタを削除していると思います。 plan ::〜plan（）で初めて、main（）を離れるときに2番目のプランです。 したがって、plan ::〜plan（）を呼び出さないでください。

そして、ルール3（またはルール5）、つまり、 を指定する必要があります。デストラクタを定義した場合、コピーコンストラクタと代入演算子を同時に定義する必要があります。