如何解决这会彻底清除类对象动态分配的结果吗?
void Inventory::createNewInventoryItem(std::string itemName,unsigned int maxQuantity,unsigned int orderThreshold,double price)
{
InventoryItems* newItem = new InventoryItems;
newItem->createInventoryItem(itemName,maxQuantity,orderThreshold,price);
m_inventory.push_back(newItem);
}
存储在此向量中的内容:std::vector<InventoryItems*> m_inventory;
然后一旦对象使用完毕,deleteInventoryItem
被调用,其定义为:
void Inventory::deleteInventoryItem(int posInVector)
{
m_inventory.at(posInVector)->~InventoryItems();
m_inventory.erase(m_inventory.begin() + posInVector);
}
我的问题是:这个deleteInventoryItem
函数会成功释放createNewInventoryItem
中动态分配使用的内存吗? (这让我对在调用类解构器时是否需要 delete[]
感到困惑)
抱歉不是“关键任务”问题,只是想确保我不会继续不良做法,因为我真的在努力尽可能完美地编程......
解决方法
这个deleteInventoryItem
函数会成功释放createNewInventoryItem
中动态分配使用的内存吗?
不,您的代码确实会泄漏内存。你在这里做什么,
m_inventory.at(posInVector)->~InventoryItems();
显式调用容器元素的析构函数。这确实会清除特定对象持有的任何资源,但不会清除托管对象本身的内存资源。您必须手动调用对象的析构函数的情况非常非常少(我能想到的唯一情况是使用位置 new
,而您很少需要这样做)。
您可以做的是使用 delete
来清理内存和调用被擦除元素的析构函数。但是作为这种手动内存管理的更安全的替代方案,我建议您选择以下两个选项之一
-
如果应该存储在容器中的对象可以通过值复制(即不是多态类型并且您想存储基类引用),则只需按值存储:
std::vector<InventoryItems> myData;
-
否则,存储一个
std::unique_ptr
为您处理所有内存管理:std::vector<std::unique_ptr<InventoryItems>> myData;
不,它会导致内存泄漏。您显然是在调用析构函数,但没有调用 delete
的对应物 new
。 new
后跟类型首先分配内存,然后隐式调用析构函数。 delete
运算符还做两件事,首先调用析构函数,然后释放内存。
很少需要显式调用析构函数,专门用于清理通过new
placement syntax.创建的实例,如果不调用~
,这种情况下如果实例包含动态hold会导致泄漏通过成员或基类绑定资源。
根据经验,请记住这些对:
-
obj = new Class;
需要delete obj;
-
obj = new (buffer) Class
需要obj->~Class();
-
array = new Class[count];
需要delete [] array;
...而且,作为集合中最重要的,自动对象:
-
Class obj;
需要(nothing)
但是封闭范围的结尾
最后一个“对”显示对(默认)构造函数(它可以是 Class
定义提供的任何其他构造函数)的隐式调用,并以析构函数的隐式调用结束(在右大括号处),顺便说一下,C++ 语言将类的析构函数限制为最多一个的原因是:编译器依赖于这种唯一性能够自动关闭作用域。这种自动性也构成了 std::unique_ptr 和 std::shared_ptr 的基础。
版权声明:本文内容由互联网用户自发贡献,该文观点与技术仅代表作者本人。本站仅提供信息存储空间服务,不拥有所有权,不承担相关法律责任。如发现本站有涉嫌侵权/违法违规的内容, 请发送邮件至 dio@foxmail.com 举报,一经查实,本站将立刻删除。