我有一个管理器类，其中包含一个指向虚拟基类的指针向量，以允许将各种子类存储在那里。在这个管理器类的析构函数中，我希望它循环遍历它持有的所有指针并删除它们。但是，我尝试了许多我遇到的方法，程序在执行过程中不断崩溃。
我的当前代码如下所示：-
for (std::vector<GameState*>::iterator it = gamestates_.begin(); it != gamestates_.end(); ++it){
delete *it;
it = gamestates_.erase(it);
}
我还没有尝试过的一件事是使用 unique_ptr 但我确信这应该能够在不使用它们的情况下处理它。如果我错了，请纠正我。
编辑：我知道我应该在循环之后清除向量，但这是我在尝试了所有删除指针的正常方法后得出的结果。它似乎不喜欢删除命令。
【问题讨论】：
标签： c++ pointers vector delete-operator
从向量中删除一个元素会使迭代器失效，因此你不能在之后继续迭代。在这种情况下，我不会删除循环中的元素；之后我会清除向量：
for (auto it = gamestates_.begin(); it != gamestates_.end(); ++it){
delete *it;
}
gamestates_.clear();
虽然，如果这是在析构函数中并且向量即将被销毁，那么清除它也没有意义。
如果您确实需要在循环中擦除（可能是因为您只想擦除某些元素），那么您需要更加小心以保持迭代器有效：
for (auto it = gamestates_.begin(); it != gamestates_.end();){ // No ++ here
if (should_erase(it)) {
it = gamestates_.erase(it);
} else {
++it;
}
}
我还没有尝试过的一件事是使用
unique_ptr，但我确信这应该能够在不使用它们的情况下处理它。如果我错了，请纠正我。
如果您确实想通过这种方式管理动态对象，请确保遵循Rule of Three：您需要实现（或删除）复制构造函数和复制赋值运算符以防止“浅”复制留下两个试图删除相同对象的向量。您还需要注意删除任何其他删除或替换对象的位置。存储智能指针（或对象本身，如果您不需要指针来实现多态性）将为您处理所有这些事情，所以我总是建议这样做。
我知道我应该在循环之后清除向量，但这是我在尝试了所有删除指针的正常方法后得出的结果。它似乎不喜欢删除命令。
最可能的原因是您没有遵循三法则，并且在复制向量后不小心尝试删除相同的对象两次。也有可能GameState 是一个基类，而您忘记给它一个虚拟析构函数，或者指针已被其他代码破坏。
【讨论】：
delete 命令很好，如果它是一个有效的指针。您是否不小心复制了向量并删除了两次对象？ GameState 是基类，实际对象有不同的类型吗？如果是这样，请确保它具有虚拟析构函数。如果不是，你为什么要存储指针？
GameState 有一个，并确保管理器类不会意外复制向量。
您的迭代器在每个循环中更新两次：
it = gamestates_.erase(it);
和
it++
您只需要第一个 - 它已经指向容器中的“下一个对象”。
【讨论】：
从你的向量中删除一个元素会使迭代器失效。按元素删除对象指针，然后clear()向量的内容。
【讨论】：
去掉for循环头中的++it。
erase 已经为你提前了。
或者，迭代，删除，然后在迭代之后.clear()。
【讨论】：
更喜欢使用unique_ptr。你说你应该能够在不使用它们的情况下处理它，就好像让一个智能指针为你做这项工作是某种可怕的强加。
它们的存在是为了让您的生活更轻松，您不必为没有手工完成这些艰苦的工作而感到内疚。
使用您现有的代码，不要调用erase。无论如何，向量都会被破坏，对吧？它会自己处理所有这些。
【讨论】：
问题是您将it 增加了两次。首先，当您调用返回下一个元素的it = .erase(it) 时，然后在循环中调用++i。您可能会不经意间跳过最后，事情可能会出错，更不用说您只会删除向量的每个第二个元素。
一个简单的解决方法是不要在循环中更改it（不更改++it）。
更好的方法是从向量末尾实际删除数组，因为从向量内部擦除元素会导致其所有后续元素的昂贵移动。您当前的算法将在N^2 时间运行。
试试这样的：
while (!gamestates_.empty()) {
delete gamestates_.back();
gamestates_.erase(gamestates_.end()-1);
}
您也可以只遍历向量的所有元素，然后将其清除：
for (std::vector<GameState*>::iterator it = gamestates_.begin(); it != gamestates_.end(); ++it){
delete *it;
}
gamestates_.clear();
还要注意，向量的clear() 操作也在其析构函数中完成。如果删除过程是 gamestates_ 最终被销毁的某个销毁过程的一部分 - 您根本不必调用 clear()。
【讨论】：