首先,我动态分配一个对象:
Object* someObject = new Object();
然后我把这个指针的地址插入到一个向量中:
std::vector<Object**> someVector0;
someVector0.push_back(&someObject);
然后,我将位于someVector0
的地址插入到另一个向量中:
std::vector<Object***> someVector1;
someVector1.push_back(&someVector0[0]);
最后,我将前面提到的someVector1
的地址插入到另一个向量中:
std::vector<std::vector<Object***>*> someVector2;
someVector2.push_back(&someVector1);
然后我使用得到的向量,现在我需要释放分配的内存:
//Do some stuff
//Free memory
for (int i = 0; i < someVector2.size(); i++) {
for (int j = 0; j < (*someVector2[i]).size(); j++) {
delete *(*(*someVector2[i])[j]);
delete *(*someVector2[i])[j];
delete (*someVector2[i])[j];
}
delete someVector2[i];
}
我曾经怀疑这段代码足够释放了,但是,我在释放的时候遇到了一些问题。我怀疑我可能解引用了一个nullptr
,但是我想不出其他的东西,我删除指针的方式是否有错误?
注:这是我在一个大规模程序中遇到的一个问题的简化版本,前面提到的向量都是不必要的,我不想处理任何副本,也不想改变向量的类型。
编辑:
嗨,在研究了更多的东西并听到了评论之后,我已经恢复了理智,即使我不认为有一个灵魂会发现我的代码是明智的,我强烈建议任何可能认为这是做任何事情的有效方法的人不要这样做。正如一个评论已经指出的,我在解决方案上的尝试表明我不知道我在做什么,所以请不要这样做;它实际上是不可维护的,相反,正如另一个评论已经指出的那样,使用smart pointers或者真正了解代码中发生了什么。
2条答案
按热度按时间5f0d552i1#
对于一个元素,你调用了
new
一次,但是调用了delete
四次,这是错误的,你只能对你用new
创建的东西调用delete
,并且一旦删除就不能再删除。q7solyqu2#
对于初学者来说,你应该重新设计你的代码,因为它非常混乱和不清楚。
由于
Object
类型的每个对象仅动态分配一次,因此对于每个操作符new
,您仅需要调用一个操作符delete
,下面是一个演示程序,说明如何做到这一点。
程序输出为
由于向量someVector2包含指向向量的指针,因此基于此范围的for循环
提供存储在向量中的指针类型
std::vector<Object ***> *
的每个元素。取消引用此范围中基于for循环的指针
你会得到指针类型为
Object ***
的元素,所以要释放一个Object类型的对象,你需要通过两次解引用元素来得到一个指向它的指针