c++ - 如何正确清理对象指针 vector 中的元素

Question

我一直在研究动态分配并在 StackOverflow 上遇到了这个问题:

Deallocating objects stored in a vector?

其中一个被赞成的答案解释了如何在使用“指向对象的指针的 vector ”时手动管理内存:遍历调用 delete 的 vector 。

我的问题是关于如何删除 vector 的特定元素，而不是整个 vector 。在下面的示例程序中，我有一个对象指针 vector 。想象一下，这些对象的 x 变量随着时间的推移而递减......当一个对象的 x 值达到一个数字(比方说 3)时，我希望删除该对象；但是，我想始终按对象的 x 值保持 vector 可排序。

问题是，当我对 x 值达到 3 的对象调用 delete 时，该对象已被删除，但那里仍然有一个指向随机内存位置的指针，并且 vector 的大小也保持不变。

当我循环打印 x 值的 vector 时，我调用 delete 的元素仍然存在，但指向诸如 -53408995 之类的值。如何摆脱 vector 的指针元素以及对象？

调用 erase 不是一个选项，因为在我的实际程序(不是下面的最小示例)中， vector 会根据其他因素连续排序，这些因素会改变 x 值等价物。我无法跟踪他们的索引。当我遍历 vector 以检查 x 值时，我想同时删除对象和指针元素。

例子:

#include <iostream>
#include <vector>

class A
{
public:
    A(int i) { x = i; }
    int x;
};

int main()
{
    std::vector<A*> Vec;

    Vec.push_back(new A{ 5 });
    Vec.push_back(new A{ 4 });
    Vec.push_back(new A{ 3 });

    std::cout << "Size before = " << Vec.size() << std::endl; // 3

    for (auto& a : Vec)
    {
        std::cout << a->x << std::endl;

        if (a->x == 3) { delete a; }
    }

    std::cout << "Size after = " << Vec.size() << std::endl; // Still 3!

    for (auto& a : Vec)
    {
        std::cout << a->x << std::endl; // Prints 5, 4 and a random memory location like -34528374
    }

    return 0;
}

Answer 1

您在评论中提到对象将有其他指向它们的指针。这听起来像 std::shared_ptr<A>大部头书。这样你就可以得到指向你的A的其他指针没有内存泄漏问题的对象。 (std::shared_ptr 会带来很小的(!)性能成本，但您现在不必担心)。此外，我更改了您的段落以从 vector 中删除/删除您的元素。请注意 A如果有其他实例保持 std::shared_ptr<A>，对象仍然存在复制到它(但这是一件好事)。

代码如下:

#include <iostream>
#include <vector>
#include <memory>
#include <algorithm>

class A
{
public:
    A(int i) { x = i; }
    int x;
};

int main()
{
    std::vector<std::shared_ptr<A>> Vec;
    Vec.emplace_back(std::make_shared<A>(5));
    Vec.emplace_back(std::make_shared<A>(4));
    Vec.emplace_back(std::make_shared<A>(3));

    std::cout << "Size before = " << Vec.size() << std::endl;

    Vec.erase(
        std::remove_if(std::begin(Vec),std::end(Vec), [](auto&& ptr){ return ptr->x == 3;}),
        std::end(Vec));

    std::cout << "Size after = " << Vec.size() << std::endl;
    for (auto&& a : Vec)
    {
        std::cout << a->x << std::endl;
    }

    return 0;
}