std::vector<State*> states;
states.push_back(new MenuState(window));
states.erase(states.begin());
这是我的代码。我没有在擦除到矢量之前使用删除。 new MenuState(window) 是一个指针。如果我不使用删除会发生什么？我应该使用吗？指针会一直占用内存吗？我该如何解决这个问题？
【问题讨论】：
标签： c++ pointers memory new-operator delete-operator
如果你不delete 存储在vector 中的指针，你会泄漏内存。您将分配内存并失去对它的控制，它将继续保持分配状态，直到您的程序结束。
在这个简单的示例中，您应该在擦除 vector 之前检查您的 vector 和 delete 每个指针。更好的是不使用“naked new”并将std::unique_ptrs 存储在vector 中，这样当它们被销毁时，它们会将分配的内存返回给系统。
【讨论】：
您可以使用智能指针：std::shared_ptr 或 std::unique_ptr。
对于std::shared_ptr，您可以使用std::make_shared()：
#include <memory> //for std::shared_ptr
#include <vector>
int main() {
std::vector<std::shared_ptr<State>> states;
states.push_back(std::make_shared<MenuState>(window));
states.erase(states.begin());
return 0;
}
或者，std::unique_ptr：
int main() {
std::vector<std::unique_ptr<State>> states;
states.push_back(std::make_unique<MenuState>(window));
states.erase(states.begin());
return 0;
}
【讨论】：
是的，对于你使用new创建的任何对象，你都需要在使用完毕后对其调用delete，否则它会泄漏并继续占用内存。
std::vector<State*> states;
states.push_back(new MenuState(window));
...
delete *states.begin();
states.erase(states.begin());
如果push_back() 失败并引发异常，您也会面临泄漏的风险，除非您捕获到异常：
std::vector<State*> states;
State *new_state = new MenuState(window);
try {
states.push_back(new_state);
}
catch (...) {
delete new_state;
throw;
}
避免这两个问题的最佳方法是根本不在原始State* 指针上使用new/delete。请改用std::unique_ptr<State> 智能指针，让它为您处理内存管理，例如：
std::vector<std::unique_ptr<State>> states;
states.push_back(std::make_unique<MenuState>(window));
...
states.erase(states.begin());
【讨论】：