关于内存泄漏的问题

Question

#include <QList>
class MyType{
//This has some data in it....
};

QList<MyType> f()
{
     QList<MyType> list;
     for(int i = 0; i<10; i++ )
     {
              MyType* item = new MyType();
              list << *item;
     }
     return list;
}

QList<MyType> temp_var = f();

当 temp_var 超出范围并销毁时，我们创建并添加到此列表中的项目会发生什么？ 会不会有内存泄漏？

谢谢。

Answer 1

是的，会有内存泄漏。作为一般规则，您的程序中的每个 new 都必须有一个 delete。

在您的具体情况下，错误逻辑发生的时间远早于temp_var 的破坏。您分配项目，然后将这些项目的副本存储在列表中。您应该立即销毁不再使用的原始物品。

你的 for 循环可能是：

 for(int i = 0; i<10; i++ )
 {
          MyType* item = new MyType(); // get me an item.
          list << *item;  // put copy of item in list
          delete item; // destroy my item
 }

当这样表达时，很明显我们根本不应该使用new！

for(int i = 0; i < 10; i++)
{
    MyType item;
    list << item;
}

假设MyType 本身没有任何内存管理错误，此版本不会泄漏。

---

编辑：顺便说一句，您的程序是否：

QList<MyType*> f() // List of POINTERS
{
     QList<MyType*> list;
     for(int i = 0; i<10; i++ )
     {
              MyType* item = new MyType();
              list << item;  // Storing a POINTER
     }
     return list;
}

那么，是的，您会遇到您所期望的内存泄漏。 QList 不会自动为指针类型提供 delete。

Answer 2

我看不出在您的代码中使用 new 有任何意义，因为您没有将指针存储在列表中，而是使用 new 创建的对象的副本，并且您没有删除它。所以是的，函数本身存在内存泄漏。

看到QList 不是指针列表，我可以说你不应该在代码中使用new：

QList<MyType> f()
{
     QList<MyType> list; //note : its not a list of MyType*
     for(int i = 0; i<10; i++ )
     {
              MyType item; //automatic variable
              list << item;
     }
     return list; 
}

Answer 3

当 QList 被销毁/超出范围时，它会随之销毁其 内容。在您的情况下，内容由对象的 副本 组成（由隐式复制构造函数构建），而不是对象本身。由于new MyType() 创建的原始对象将丢失其指针，因此内存将在 for 循环的每次迭代中泄漏，但仍将保持分配状态。

Answer 4

如果您不删除使用new MyType() 创建的所有项目，肯定会有泄漏！

在QList 的析构函数中，您需要遍历列表并在每个项目上调用 delete。