需要释放 QList 内容吗？

Question

我有一个充满动态创建的对象的 Qlist。在终止我的程序之前，我调用了 myqlist.clear()

我的问题是：这是否也会删除（释放）列表中包含的对象？ Valgrind 给了我一些丢失的块，我想知道我是否误解了 qlist clear 方法的工作原理。

或者，我是否需要遍历 qlist 并删除每个对象？

---

更新：我可以确认 mylist.erase(iterator) 正在从列表中删除项目，但没有释放动态分配的对象。 （对象是一个动态实例化的类）。很奇怪！我从 Qlist 切换到 QLinkedList 但结果相同。请记住，我的 QLinkedList 是 QLinkedList 而不是 QLinkedList

这是实际代码，以防有人发现我做错了什么：

// Here I define a couple important items.  Note that AMISendMessageFormat is a class
typedef QLinkedList<AMISendMessageFormat> TSentMessageQueue;
TSentMessageQueue m_sentMessageQueue;

// Here I create the message and append to my QLinkedList
AMISendMessageFormat *newMessage = new AMISendMessageFormat(messageToSend);
m_sentMessageQueue.append(*newMessage); 

// Here I delete
for (TSentMessageQueue::Iterator sMessagePtr = m_sentMessageQueue.begin(); sMessagePtr != m_sentMessageQueue.end(); )
{
    sMessagePtr = m_sentMessageQueue.erase(sMessagePtr);  
    qDebug() << "Sent size after erase: " << m_sentMessageQueue.size();  // Confirmed linked list is shrinking in size
}

在遍历列表并擦除之后，valgrind 显示每个 AMISendMessageFormat 对象都是丢失的块！

我怀疑这与使用迭代器在循环内擦除有关...但我无法理解这一点！

---

请参阅下面的详细解决方案...问题是 append 函数创建一个副本并将其添加到列表中...我虽然它正在添加实际对象（不是副本）...所以问题是'新的副本被泄露了。

Answer 1

这取决于您的对象。 Qt 有一个object ownership 的概念，它将对象组织成树。树的父级在超出范围后立即删除其所有子级。

如果您的对象不是由父对象管理的，您需要确保自己解除分配它们。 Qt 还附带了一组smart pointers 来简化此操作。

Answer 2

您正在泄漏newMessage 指向的实例。这与列表无关！您没有从列表中泄漏。解决方案：

// Best

m_sentMessageQueue << AMISendMessageFormat(messageToSend);

// Same, more writing

AMISendMessageFormat newMessage(messageToSend);
m_sentMessageQueue << newMessage;

// Rather pointless allocation on the heap

QScopedPointer<AMISendMessageFormat> newMessage(new AMISendMessageFormat(messageToSend));
m_sentMessageQueue << *newMessage; 

请注意，在每种情况下，您都将对象的副本存储到列表中。 重要提示：您必须验证 AMISendMessageFormat 是一个行为正确的 C++ 类，可以安全地复制构造和分配给它，而不会泄漏资源。

如果您没有定义复制构造函数和赋值运算符，那么您在此类中使用的所有数据成员都必须可以安全地复制和分配，而不会泄漏。所有 Qt 和 C++ 标准库类都不会在这种使用下编译，或者会正常运行。如果你使用裸指针，你已经在脚上开枪了，所以至少使用正确的QSharedPointer。

在编辑之前，你没有说你的对象是什么。

• 如果您将原始指针存储在列表中，那么当您执行clear() 时，您肯定会泄漏内存。 QList 将这些指针视为整数，并且不会对它们做任何特殊处理。在 C++ 中，原始指针的销毁，就像整数的销毁一样，是一个 NO-OP。

• 如果您将QSharedPointer 或std::shared_ptr 存储在列表中，那么您在执行clear() 时不会泄漏内存。以这种方式调用智能指针是有原因的 :)

• 如果您自己存储对象，并且它们是行为正确的 C++ 类，那么一切都很好。

您不能将QObject 直接存储在QList 中，因此您的“对象”不能是 QObjects - 它不会编译。

这工作得很好并且行为正常：

QList<QString> stringList1;
QList<QSharedPointer<QString> > stringList2;

stringList1 << "Foo" << "Bar" << "Baz";
stringList2 << new QString("Foo") << new QString("Bar") << new QString("Baz");

Q_ASSERT(stringList1.at(0) == *stringList2.at(0));
stringList1.clear();
stringList2.clear(); // no memory leaks

这会泄漏内存，你几乎不需要编写这样的代码：

QList<QString*> stringList3;
stringList3 << new QString("Foo") << new QString("Bar") << new QString("Baz");
stringList3.clear();

还要注意QList 和所有像样的 C++ 容器类型都是 RAII。这意味着他们将释放他们在销毁时使用的资源。这意味着您不需要在列表上调用clear()，除非您真的希望清除列表。此代码不会泄漏资源。列表的析构函数会在main()返回之前被调用，列表的析构函数会销毁所有的字符串，它们都会正确的释放它们分配的堆内存。

int main() {
    QList<QString> stringList1;
    stringList1 << "Foo" << "Bar" << "Baz";
    return 0;
}

Answer 3

qDeleteAll(list.begin(), list.end());