Удаление пользовательских векторов — C++

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

for ( std::vector<B*>::iterator it = vect.begin(); it != vect.end(); ++it)
    delete (*it);
vect.clear();

edit: если ваша программа ошибается в деструкторе, значит, ваш код неверен. Может быть, вы помещаете элемент стека по адресу в вектор, но для удаления объекта он должен быть в куче.

#include <iostream>
#include <vector>
#include <string>

class data {
public:
    std::string d;
    data(std::string _d) : d(_d) { }
};

class container {
public:
    std::vector<data*> c;
    container() { c.clear(); }
    void add (data *d) { c.push_back(d); }
    ~container() {
        for (std::vector<data*>::iterator it = c.begin(); it != c.end(); ++it)
            delete (*it); 
        c.clear();
    }
};

int main (int argc, char* argv[]) {

    typedef std::vector<std::string> sVec;
    typedef sVec::iterator sVecIter;

    std::vector<std::string> cmd (argv+1, argv+argc);

    {
    container k;            
    for (sVecIter it = cmd.begin(); it != cmd.end(); ++it)
        k.add(new data((*it)));

    }

    return 0;

}

Это работает без проблем.

Если A владеет вещами, на которые указывает vect, то он должен иметь возможность delete каждого элемента в пределах vect. Если при этом происходит сбой, значит, где-то в вашем коде есть ошибка.

Однако в целом вам лучше использовать умные указатели. ptr_vector Boost (часть Контейнер Boost.Pointer предназначен для вашего конкретного примера, но простой std::vector<std::tr1::shared_ptr<B> > также будет работать (хотя и с большими накладными расходами и более неудобным синтаксисом).

Да, если элементы типа B* указывают на объекты, размещенные в куче, то для каждого элемента вы должны вызывать для него удаление.

Да, вы бы зациклили вектор и удалили каждый элемент. Проблема в том, что вы забыли оператора полотенец в строке 42.

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

Если это единственный настоящий владелец и он гарантированно будет последним для очистки, я бы пошел с

std::vector<B> vect;

Если у вас есть разные ссылки и времена жизни, которые могут перекрываться, тогда лучше использовать shared_ptr (std::tr1 или boost в зависимости от компилятора).

std::vector< boost::shared_ptr<B> > vect;

Судя по вашему описанию, ваш векторный член класса A должен пожизненно владеть данными, указанными B *.

Я бы рекомендовал просто изменить это объявление на

vector< std::tr1::shared_ptr<B> > vect;

РЕДАКТИРОВАТЬ: заменить auto_ptr на std::tr1::shared_ptr