Почему конструктор копирования не вызывается так, как я ожидаю, используя карту? - PullRequest
Новая
       5

Почему конструктор копирования не вызывается так, как я ожидаю, используя карту?

3 голосов
/ 03 марта 2010

У меня проблемы с использованием моего пользовательского класса с 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;
  }

Ответы [ 2 ]

10 голосов
/ 03 марта 2010

myClass требуется пользовательский оператор присваивания, в дополнение к конструктору копирования. Поэтому, когда вы делаете присваивание, вы теряете исходное значение слева и в конечном итоге дважды удаляете значение справа.

0 голосов
/ 03 марта 2010

Ваш конструктор игнорирует параметр 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;
}

1010 * редактировать *

Дерек Ледбеттер отметил, что нам также нужен оператор присваивания. И я бы предложил сделать виртуальный деструктор.

...