3

std::mapでカスタムクラスを使用する際に問題が発生します。クラスはメンバーに動的にメモリを割り当てます。クラスが割り当てられたすべてのメモリを確実に削除するようにしたいので、マップでポインタを使用したくありません。しかし、私が抱えている問題は、アイテムをマップに追加した後、そのコードブロックがスコープ外になると、オブジェクトがまだマップ上にあるにもかかわらず、オブジェクトデストラクタが呼び出されることです。以下に、私が何を意味するかを示す偽のコードを作成しました。出力は次のとおりです。問題は、なぜ最後のデストラクタが呼び出されるのかということです。事前に感謝し、長い質問をお詫び申し上げます。

Constructor Called Num:0034B7E8
Default Constructor Called Num:00000000
Copy Constructor Called Num:CCCCCCCC
Copy Constructor Called Num:CDCDCDCD
destructor called Num:CCCCCCCC
destructor called Num:00000000
destructor called Num:0034B7E8
Inserted Num:0034B7E8



class myClass
{
public:
  myClass(int num) 
  {
     mnNum = new int();
     cout << "Constructor Called Num:" << mnNum << endl;
   }

   myClass() : mnNum(NULL)
   {
      cout << "Default Constructor Called Num:" << mnNum << endl;
   }

   myClass(const myClass &copy) 
   {
      mnNum = new int(copy.mnNum);
      cout << "Copy Constructor Called Num:" << mnNum << endl;
   }

   ~myClass()
   {
      delete mnNum;
      mnNum = NULL;
   }

   int* mnNum;

 };

 map<string,myClass> mvMyMap;

 void testFunction()
 {
     myClass lcObj(1);

     mvMyMap["Test"] = lcObj;
 }


 int _tmain(int argc, _TCHAR* argv[])
 {
     testFunction();
     cout << "Inserted Num:" << mvMyMap["Test"].mnNum << endl;
 return 0;
  }
4

2 に答える 2

10

myClassコピー コンストラクターに加えて、カスタム代入演算子が必要です。したがって、割り当てを行うと、左側の元の値がリークし、最終的に右側の値が二重に削除されます。

于 2010-03-02T23:00:41.990 に答える
-1

コンストラクターはパラメーターを無視し、numパラメーターから初期化することはありませんmnNum。次のようになります。

myClass(int num) 
{
    mnNum = new int(num);
    cout << "Constructor Called Num:" << mnNum << endl;
}

また、次のようにコピー コンストラクターを調整する必要があります。

myClass(const myClass &copy) 
{
    mnNum = new int(*copy.mnNum);
    cout << "Copy Constructor Called Num:" << mnNum << endl;
}

編集

Derek Ledbetter は、代入演算子も必要だと指摘しました。そして、デストラクタを仮想にすることをお勧めします。

于 2010-03-02T22:56:24.513 に答える