删除指针矢量时避免双重免费

Question

我尝试了以下内容，但是在运行时获得了分割故障和“在TCACHE2中的双免费检测到”：

class A {
... 
};

class B {
    public:
        B(...) {...}
        ~B() {
        for(int i=0;i<As.size();i++) {
            if(As[i] != nullptr) {
                delete As[i];
                As[i] = nullptr;
                }
    
            }
        }
        std::vector<A*> As;
};

int main()
{   
    A* a1 = new A(...);
    A* a2 = new A(...);
    
    B* b1 = new B(...);
    b* b2 = new B(...);
    
    b1.As.push_back(a1);
    b1.As.push_back(a2);
    
    b2.As.push_back(a1);

    delete b1;
    delete b2;
    return 0;
}

有什么办法可以保证我只删除这些指针一次？

Answer 1

好的，您正在将相同的指针存储在两个阵列中。是的，您正在删除两次。这就是为什么在现代C ++中劝阻原始指针的原因。改用智能指针。

std::unique_ptr<A> a1 = std::make_shared<A1>();

之后，您可以像其他任何指针一样使用它们，除了类型不同，您不必删除它们。他们参考计数。如果需要，只需将值设置为nullptr即可。

a1 = nullptr;  // Will free if this is the last copy.