Предположим, что я определил такой класс:
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();
}
};
Короче говоря, я отталкиваю некоторые указатели в векторе, позже я очищаю вектор. Вопрос в том, имеет ли этот код утечку памяти? Я имею в виду, очистив вектор, правильно ли удалены указатели?
Да, в коде есть утечка памяти, если вы не удалите указатели. Если класс foo владеет указателями, он обязан удалить их. Вы должны сделать это перед очисткой вектора, в противном случае вы потеряете дескриптор памяти, которую нужно отменить.
for (auto p : v)
{
delete p;
}
v.clear();
Вы можете полностью избежать проблемы управления памятью, используя std::vector подходящего интеллектуального указателя.
Я думаю, что самое короткое и ясное решение было бы:
std::vector<Object*> container = ... ;
for (Object* obj : container)
delete obj;
container.clear();
Нет, вы просто очищаете векторное хранилище. Выделенная память с "новым" все еще существует.
for (int i =0; i< v.size();i++)
{
delete (v[i]);
}
v.clear();
Вы можете использовать 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 ();