内存释放冻结了我的程序

Question

这不是问题，只是我想回答的一个问题。

我正在制作具有粒子的2D应用程序。在单击处理我已经写了这个代码：

Particle *tempp = new Particle(); 
tempp->setPosition(mx, my); 
particles.push_back(tempp); // typeof particles = std::list<Particle*> 
delete tempp; // <- this line is the problem 

当我点击，一个粒子将在鼠标的位置创建。大约一秒后，它应该消失，这工作正常。消失后，我可以再次点击创建一个新的粒子。

但是，当我点击屏幕上仍然有一个粒子时，我的程序冻结并停止工作。

我析构函数Particle类和它的母公司析构函数都是空的。

没有调用delete我的程序运行良好，即使有多个粒子一次，甚至每帧多个。我只是想知道是什么导致这个冻结的问题。

Answer 1

根据发布的代码，particles容器将包含悬挂指针。任何解除引用的尝试都是未定义的行为。我假设他们后来被使用，否则他们的存​​储似乎毫无意义。

调用push_back()不会复制指向对象，而是复制指针的值（动态分配的对象的内存地址）。如果Particle是便宜的复制，是可复制和多态的行为是不需要的只是将Particle存储在容器中。否则，建议使用智能指针（如std::unique_ptr）在从容器中移除时自动销毁Particle。

Answer 2

当您将指针推入列表中时，您只需按下实际指针的副本，但不要复制它指向的内容。这意味着push_back之后你有两个指针指向相同内存。

如果你然后释放该内存，那么你有一个指向空闲内存的指针，该指针现在是无效的。

Answer 3

您正在将pointer放入容器中，然后删除它是一个问题。 push_back将复制pointer的值而不是pointer的内容，因此当您拨打delete时，容器中的pointer不再有效。所以现在你有一个悬挂的指针，当你解除引用时，这将是未定义的行为，但很可能是崩溃。

Answer 4

我相信这个解决方案在于你的第三行代码。请注意，粒子是指向粒子的指针的向量？那么，在第三行中，您将创建指针的副本并将其插入到列表中。在下一行中，您将释放该指针指向的内存。 该列表不存储您的粒子 - 它仅存储粒子的内存地址因此，当您删除粒子时，您告诉编译器重新使用包含有效数据的内存。

所以，在评论第4行时，问题就消失了，它并没有真的在那里崩溃 - 没有理由这样做。在有可能的情况是，它试图做与曾经是你记忆东西时崩溃，但在第3行

你可以榜上无名店实际颗粒作为修复被释放。

Answer 5

当我看到new，然后几行后delete，我宁愿看到使用的堆栈。除非Particle是巨大的（我怀疑），你可以更改您的代码如下：

Particle tempp; 
tempp.setPosition(mx, my); 
particles.push_back(tempp); // change particles to std::list<Particle> 

普雷斯托。你写的代码少，你不会炸毁。