Нужно ли освобождать указатели объектов в векторе? - PullRequest
0 голосов
/ 08 апреля 2019

Я не понимаю, как работает освобождение векторной памяти. Для примера ниже,

vector<Object*> vec;

for(int i = 0; i < 10; i++){
  Object* obj = new Object();
  vec.push_pack(obj);
}

//DEALLOCATE CODE HERE//

Что я должен сделать, чтобы правильно освободить vec? Программа вроде бы работает нормально, но я не уверен.

Ответы [ 2 ]

1 голос
/ 08 апреля 2019

избегайте использования new / delete:

std::vector<std::unique_ptr<Object>> vec;

for(int i = 0; i < 10; i++)
{
  vec.push_pack(std::make_unique<Object>());
}

unique_ptr позаботится об удалении

0 голосов
/ 08 апреля 2019

как освободить

например сделать

for(auto o : vect){
  delete o;
}
vect.clear();

Обратите внимание, что вы написали push_pack вместо push_back для заполнения вектора


создание полной программы:

#include <vector>
using namespace std;

class Object{};

int main()
{
  vector<Object*> vec;

  for(int i = 0; i < 10; i++){
    Object* obj = new Object();
    vec.push_back(obj);
  }
  for(auto o : vec){
    delete o;
  }
  vec.clear();
}

Компиляция и выполнение под valgrind :

pi@raspberrypi:/tmp $ g++ v.cc
pi@raspberrypi:/tmp $ valgrind ./a.out
==9157== Memcheck, a memory error detector
==9157== Copyright (C) 2002-2017, and GNU GPL'd, by Julian Seward et al.
==9157== Using Valgrind-3.13.0 and LibVEX; rerun with -h for copyright info
==9157== Command: ./a.out
==9157== 
==9157== 
==9157== HEAP SUMMARY:
==9157==     in use at exit: 0 bytes in 0 blocks
==9157==   total heap usage: 16 allocs, 16 frees, 20,358 bytes allocated
==9157== 
==9157== All heap blocks were freed -- no leaks are possible
==9157== 
==9157== For counts of detected and suppressed errors, rerun with: -v
==9157== ERROR SUMMARY: 0 errors from 0 contexts (suppressed: 6 from 3)

вся выделенная память была освобождена

Добро пожаловать на сайт PullRequest, где вы можете задавать вопросы и получать ответы от других членов сообщества.
...