c++ - 从 std::vector 中删除动态分配的对象

Question

这是正确的吗？

std::vector<Enemy*> enemies;
enemies.push_back(new Enemy());

Enemy* enemy = enemies[0];
enemies.erase(enemies.begin() + 0);
delete enemy;

Answer 1

是的，它确实有效，但这不是理想的方法。

首先，添加 0 只是噪音，您可以将其删除。但更好的是，只需使用 pop_front()。此外，无需中间步骤，您可以在删除之前删除。

但是 std::vector 不如从前面弹出那么好，尤其是当它很大时(因为需要移动剩余的元素来填充空白)。如果不需要连续内存，请改用 std::deque。或者，如果顺序无关紧要，您可以使用如下内容:

template <typename T, typename A>
void unordered_pop_front(std::vector<T, A>& vec)
{
    using std::swap;
    swap(vec.front(), vec.back()); // constant time

    vec.pop_back(); // constant time
}

它将前元素与后元素交换，然后将其弹出。当然，不会保留顺序。

另一个问题是您的内存管理方法。任何时候你有明确的清理代码，你就做错了什么。应该是done automatically .

使用 Boost's ptr_vector ，或 smart pointers 的 std::vector . (注意:不要在容器中使用 std::auto_ptr，它在这方面是错误的。)对于快速智能指针建议，使用 std::unique_ptr(如果你的编译器支持 C++0x)，或者 std::/boost::shared_ptr。