C++：デストラクタのif文が必要ですか？

Question

以下のVectorクラスのデストラクタには、使用するメモリをキャンセルするにはifステートメントが必要です。 arrにメンバーが1人いる場合はdelete arrとなります。 arrにメンバーが多い場合は、delete[] arrを使用する必要があります。

教えてもらえますか？

マイコード：

class Vector { 
    double * arr; 
    short dim; 

public: 
    Vector(short d = 0): dim(d) { 
     arr = NULL; 
     if (dim < 0) { 
      dim = 0; 
     } else { 
      arr = new double[dim]; 
     } 
    } 

    ~Vector() { 
     if (arr != NULL) { 
      if (dim == 1) { 
       delete arr; 
      } else { 
       delete[] arr; 
      } 

      arr = NULL; 
      dim = 0; 
     } 
    } 
}; 

Answer 1

だけでなく、それはそれだけで間違っている、必要はありません。 new []で作成されたものは、delete []でのみ削除できます。それ以外の場合は、未定義の動作が作成されます。

Answer 2

どこでもnew double;に電話することはないので、不要です。常にdelete [] arr;に電話してください。また、NULLを確認する必要もありません。

Answer 3

これは必須ではないだけでなく、new[]でメモリを割り当てるため、これは不正です。 newに電話する場合はdeleteが必要です。new[]に電話する場合はdelete[]が必要です。それらを混在させることは未定義の動作です。あなたのベクトルそれは次のようになります。今、私たちは正しいのコピーを持っているし、削除nullptr上の非OPであるか、または正しくコンストラクタから割り当てられたメモリを解放するか

class Vector { 
    double * arr; 
    short dim; 

public: 
    Vector(short d = 0): dim(d) { 
     if (dim > 0) 
      arr = new double[dim] 
     else 
      arr = nullptr; 
    } 

    Vector(const Vector& copy) : dim(copy.dim) { 
     if (dim > 0) { 
      arr = new double[dim] 
      // copy data here 
     } 
     else 
      arr = nullptr; 
    } 

    ~Vector() { 
     delete [] arr; 
    } 

    Vector & operator=(Vector rhs) { 
     // swap the contents of the copy. you can make a swap function to do this 
     double * temp = arr; 
     arr = rhs.arr; 
     rhs.arr = temp; 
     dim = rhs.dim; 
    } 

}; 

を。

Answer 4

これは必須ではありません。新しいキーワードを使用して割り当てられたメモリを作成するときは、大括弧を使わずにメモリを削除することを忘れないでください。