Удаление объекта в C++

Question

Вот пример кода, который у меня есть:

void test() 
{ 
    Object1 *obj = new Object1(); 
    . 
    . 
    . 
    delete obj; 
} 

я запустить его в Visual Studio, и он выходит из строя в соответствии с «удалить OBJ;». Разве это не нормальный способ освободить память, связанную с объектом? Я понял, что он автоматически вызывает деструктор ... это нормально?

---

Вот фрагмент кода:.

if(node->isleaf()) 
    { 
     vector<string> vec = node->L; 
     vec.push_back(node->code); 
     sort(vec.begin(), vec.end()); 

     Mesh* msh = loadLeaves(vec, node->code); 

     Simplification smp(msh); 

     smp.simplifyErrorBased(errorThreshold); 

     int meshFaceCount = msh->faces.size(); 

     saveLeaves(vec, msh); 

     delete msh; 
    } 

loadleaves() это функция, которая считывает сетку с диска и создает Mesh объект и возвращает его (вспомните vec и node->code только информация о файл должен быть открыт)

Следует ли удалить строку delete msh;?

Answer 1

Разве это не нормальный способ освободить память, связанную с объектом?

Это обычный способ управления динамически распределенной памятью, но это не очень хороший способ сделать это. Этот тип кода является хрупким, потому что он не является безопасным для исключения: если при создании объекта возникает исключение, и когда вы его удаляете, вы пропустите этот объект.

Лучше использовать контейнер интеллектуальных указателей, который вы можете использовать для получения управления ресурсами с ограничением по объему (его обычно называют resource acquisition is initialization или RAII).

В качестве примера автоматического управления ресурсами:

void test() 
{ 
    std::auto_ptr<Object1> obj1(new Object1); 

} // The object is automatically deleted when the scope ends. 

В зависимости от вашего случая использования auto_ptr не может обеспечить семантику вам нужно. В этом случае вы можете использовать shared_ptr.

Что касается причины сбоя вашей программы при удалении объекта, вы не дали достаточного кода, чтобы кто-либо мог ответить на этот вопрос с какой-либо определенностью.

Answer 2

если он падает на линии delete, то вы почти наверняка каким-то образом испортили кучу. Нам нужно будет увидеть больше кода для диагностики проблемы, так как представленный вами пример не имеет ошибок.

Возможно, у вас есть переполнение буфера на куче, которая испортила структуры кучи или даже что-то такое же простое, как «double free» (или в случае C++ «двойное удаление»).

Кроме того, как отметил Fuzz, у вас может быть и ошибка в вашем деструкторе.

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

Answer 3

Разве это не нормальный способ освободить память, связанную с объектом?

Да, это так.

Я понял, что он автоматически вызывает деструктор ... это нормально?

Yes

Убедитесь, что вы не double delete ваш объект.

Answer 4

Ваш код действительно использует обычный способ создания и удаления динамического объекта. Да, это нормально (и действительно гарантировано языковым стандартом!), Что delete вызовет деструктор объекта, как и new должен вызвать конструктор.

Если вы не инстанцирование Object1 непосредственно, но некоторый его подкласс, я бы напомнить вам, что любой класс предназначен для наследования от должен виртуального деструктора (так что деструктор Правильного подкласса может быть использован в тех случаях, аналогичных к этому), но если ваш образец кода действительно представляет ваш фактический код, это не может быть вашей текущей проблемой - должно быть что-то еще, может быть, в коде деструктора, который вы нам не показываете, или какой-то кучевой коррупции в код, который вы не показываете внутри этой функции или те, которые он вызывает ...?

BTW, если вы всегда собираетесь удалить объект непосредственно перед тем, как выйти из функции, которая его создает, нет смысла делать этот объект динамическим - просто объявите его как локальный (класс хранения auto, как и default) переменной указанной функции!

Answer 5

saveLeaves(vec,msh);
Я принимаю указатель msh и помещает его в vec. Поскольку msh - это просто указатель на память, если вы удалите его, он также будет удален внутри вектора.