c++ 析构函数 - 何时使用它们以及要删除的内容

Question

我在当前的项目中遇到了困难。在以前的版本中，我使用了 std::vectors，一切都很好，没有内存泄漏，没有错误，很好。

然后我改用了一些指针，因为它更优雅，而且我的性能有所提高。

所以我有类似的课程

class A
{
    private:
        std::string str;
    public:
        A() {str = "";}
};

class B
{
    private:
        std::string str;
        A* a;
    public:
        B() {str = ""; a = NULL; }
};

class C
{
    private:
        std::vector<B*> bs;
    public:
        C() { bs = std::vector<B*>(); }

};

我知道每次使用new，之后我都必须delete。所以我为class B 和class C 创建析构函数。

我认为它应该是这样的：

B::~B()
{
    if ( this->a != NULL )
        delete this->a;
}

C::~C()
{
    while ( (this->bs).size() > 0 )
    {
        if ( (this->bs).back() != NULL )
            delete (this->bs).back();
        (this->bs).pop_back();
    }
    (this->bs).clear(); // not necessary since bs has size 0?
}

但是，我遇到了各种错误，例如 valgrind 中的“无效读取大小为 4”。

我认为析构函数应该是什么样子的想法可以吗？ 我的代码还有什么问题？ 将私有std::string 成员更改为std::string-pointers 会更好吗？ 在析构函数中做str = "";是必要的还是好事？

附带说明：请理解我已经对“适当的析构函数”和类似关键字进行了大量搜索，但没有帮助。如果你认为这个问题被问得太频繁了：我不知道，因为我不明白。

Answer 1

Rob 所说的，你的析构函数也可以简化为：

B::~B()
{
    delete a;
}

C::~C()
{
    for (size_t i = 0; i < bs.size(); ++i)
        delete bs[i];
}