クラステンプレートに問題があります

Question

私は宿題をしていますが、私はクラステンプレートであまり好きではありません。

タスクは次のとおりです。

次のクラスの実装における主要な問題はあります。あなたはそれを見つけることができますか？どうすれば の問題を解決できますか？ というクラスの要件仕様に応じて、複数のソリューションを提案できます。

template <class T> 
class Array 
{ 
private: 
    T *m_pData; 
    unsigned int m_nSize; 

public: 
    Array(unsigned int nSize) : m_nSize(nSize) 
    { 
    if(m_nSize > 0) 
     m_pData = new T[m_nSize]; 
    } 

virtual ~Array() 
    { 
    if(m_pData != NULL) 
     delete m_pData; 
    } 
bool Set(unsigned int nPos, const T& Value) 
    { 
    if(nPos < m_nSize) 
    { 
     m_pData[nPos] = Value; 
     return true; 
    } 
    else 
     return false; 
    } 

    T Get(unsigned int nPos) 
    { 
    if(nPos < m_nSize) 
     return m_pData[nPos]; 
    else 
     return T(); 
    } 
}; 

は私に関する限りメモリリークがあります。あなたは他のいくつかの問題を見つけますか？

Answer 1

コンストラクタにはelseのないifがあります。その結果を考えてみましょう。
新しい[]がdelete []に​​一致し、newがdeleteで一致していることを確認してください。
これは3のルール（またはC++ 0xの5）に違反します

これは私が見るものです。

Answer 2

1. コンストラクタ - あなたはif文がデストラクタで必要とされていないので、あなたはヌルポインタを削除することができます0
2. さnsizeの場合のイベントにm_pDataの値を設定する必要があります。削除は、[]

Answer 3

HERESにいくつかのことを削除する必要がありますが、私はそれらのいずれかが正解であることが十分に深刻であることを疑う：

1. get機能が無効なインデックスの値を返します。 。 type Tがintで、インデックスが大きすぎるとgetを使用した場合、得られる回答は0になります。値が0の場合はどうなりますか？ 0の値と無効なインデックス値の違いをどのように伝えますか？

2. いくつかの余分なコードがあります：あなたは何のコピーコンストラクタや代入演算子はありませんelse return

Answer 4

を必要といけません。このコードは非常に速くプログラムをクラッシュさせます。それが大きな欠陥です。

デストラクタを仮想にすることは奇妙な決定です。このクラスから派生する必要性や利点は明らかではありません。

私はスワップメソッドも実装します。