rozliczeń wektor wskaźników

Question

Załóżmy, że zdefiniowaliśmy klasę tak:

class foo { 
private: 
    std::vector< int* > v; 
public: 
    ... 
    void bar1() 
    { 
     for (int i = 0; i < 10; i++) { 
     int *a = new int; 
     v.push_back(a); 
     } 
    }; 

    void bar2() 
    { 
     std::vector<int>::iterator it = v.begin(); 
     for (; it != v.end(); it++) 
     std::cout << (*it); 
     v.clear(); 
    } 
}; 

Krótko mówiąc, odepchnąć pewne wskazówki w wektorze, później wyczyścić wektor. Pytanie brzmi, czy ten kod ma wyciek pamięci? Chodzi mi o to, oczyszczając wektor, czy wskaźniki są właściwie usunięte?

Answer 1

Tak, kod ma wyciek pamięci, chyba że usuniesz wskaźniki. Jeśli klasa foo jest właścicielem wskaźników, to jest odpowiedzialna za ich usunięcie. Powinieneś to zrobić przed usunięciem wektora, w przeciwnym razie stracisz uchwyt do pamięci, którą musisz zwolnić.

for (std::vector<int>::iterator it = v.begin() ; it != v.end(); ++it) 
    { 
    delete (*it); 
    } 
    v.clear(); 

Można uniknąć tego problemu całkowicie zarządzania pamięcią za pomocą std::vector odpowiedniego smart pointer.

Answer 2

Nie wyczyścisz tylko pamięci wektorowej. Przydzielona pamięć z "nowym" nadal tam jest.

for (int i =0; i< v.size();i++) 
    { 
    delete (v[i]); 
    } 
    v.clear(); 

Answer 3

Można użyć for_each:

std::vector<int*> v; 

template<typename T> 
struct deleter : std::unary_function<const T*, void> 
{ 
    void operator() (const T *ptr) const 
    { 
    delete ptr; 
    } 
}; 

// call deleter for each element , freeing them 
std::for_each (v.begin(), v.end(), deleter<int>()); 
v.clear(); 

Answer 4

myślę najkrótsza i oczywistym rozwiązaniem byłoby:

std::vector<Object*> container = ... ; 
for (Object* obj : container) 
    delete obj; 
container.clear();