QThread 正在造成内存泄漏

经过对我的 QT 应用程序进行大量测试和更改后，Visual Leak Detector 识别出了令人讨厌的泄漏源（8 字节）。 VLD 报告 QT 应用程序是干净的，除了QThread*指针。

一些实现背景：该应用程序由 Jeffrey Holmes 建模为解决方案的混合体，使用Qt批量下载网页 https://stackoverflow.com/questions/3910105/bulk-download-of-web-pages-using-qt。感谢 Jeffrey 提供的早期解决方案！

问题：

• Why is QThread*当工作线程完成其工作时不会销毁自身？

• 我怎样才能强制QThread*工作完成后删除线程和工作对象？

• Should QThread以不同的方式实施？

Code:

void vqMDIChildDialog::processWorkQueue(bool bIsBC)
{

if (m_listOfTables.isEmpty() && currentReplicationThreads == 0)
{
}
else if (!m_listOfTables.isEmpty())
{
    for (int i = 0; i < maxReplicationThreads && !m_listOfTables.isEmpty();i++)
    {
        QThread *thread = new QThread;
        QPointer<vcSharedDataQt> worker = new vcSharedDataQt();
        worker->moveToThread(thread);
        QString tmpTableName (m_listOfTables.dequeue());
        worker->setParentObject(this);
        //   
        // set properties on the worker object.
        //  
        connect(thread,  SIGNAL(started()),  worker, SLOT(process()));
        connect(worker,  SIGNAL(finished()), thread, SLOT(quit()));
        connect(worker,  SIGNAL(finished()), worker, SLOT(deleteLater()));
        connect(thread,  SIGNAL(finished()), thread,SLOT(deleteLater()));

        connect(worker,
            SIGNAL(updateMessageFromThread(  const QString&,
                             const QString&,
                             const QString&,
                             const QString&,
                             const QString&,
                             const QString&,
                             const QString&)), 

            this,
            SLOT(UpdateStatusBarFromThread( const QString&,
                            const QString&,
                            const QString&,
                            const QString&,
                            const QString&,
                            const QString&,
                            const QString&)));

        thread->setObjectName(worker->getUniqueKey());

        thread->start();
        currentReplicationThreads ++;
    }
}

}

Stack 不允许我回答这个问题，所以：

该功能受到保护QMutex:

mutex.lock();
processWorkQueue();
mutex.unlock();

这导致了内存泄漏。QThread显然无法在工作线程完成时销毁。我删除了互斥锁，VLD 报告没有内存泄漏QThread.

1. A QThread并不真正知道它的工作何时完成。您可以通过思考如何实现这一点来回答您自己的问题。

   Its run()方法只是旋转一个事件循环。由于事件循环可以知道是否有任何事件发布到它，因此您可以合理实现的唯一条件是在没有进一步的事件时退出线程。这会使线程立即退出，所以它根本没有帮助。

   也许您希望在没有更多线程时终止线程QObject以线程作为线程的 s。这当然可以作为可选行为来实现QThread，但我不知道这个改变是否会被接受。它不能是一种默认行为，因为在许多情况下，不断地销毁和重新创建线程根本就是一种浪费——人们可能希望保留一个没有对象的线程。

   最终只有您知道线程的工作何时完成。您的工作对象可以调用thead()->quit()或者它可以在完成时发出信号 - 就像您已经做的那样。

2. A QThread工作完成后不能自行销毁，因为线程是可重新启动的。您可以完全控制线程的生命周期，因此你当然可以当它的工作完成时就摧毁它，你已经这样做了，只是你做错了。

3. 你的问题实际上是按照你希望事情发生的顺序排列的。这deleteLater操作由事件循环执行。如果线程的事件循环没有运行，deleteLater是一个NO-OP。

   因此，首先，您的连接应该形成一个只能以明确定义的顺序执行的级联：

   connect(thread, SIGNAL(started()),  worker, SLOT(process()));
   connect(worker, SIGNAL(finished()), worker, SLOT(deleteLater()));
   connect(worker, SIGNAL(destroyed()), thread, SLOT(quit()));
   connect(thread, SIGNAL(finished()), thread, SLOT(deleteLater()));
   

   然后，您必须确保您所在的线程processWorkQueue方法运行不会被阻止并且有机会its事件循环继续进行。正是这个事件循环将处理线程的删除。

   正如 AlexP 所指出的，这在 Qt 4.7 或更早版本中不起作用，因为所有这些版本在 QThread 的实现中都有一个错误。 “QThread 的行为已经改变”是“有一个丑陋的错误最终被修复”的委婉说法。

4. 您的连接过于冗长。您可以从签名中删除空格和引用/常量引用。第三个参数也是可选的，如果它是this。它应该看起来像：

   connect(worker,                                    
           SIGNAL(updateMessageFromThread(QString,QString,QString,QString,
                                          QString,QString,QString)),
           SLOT(updateStatusBarFromThread(QString,QString,QString,QString,
                                          QString,QString,QString)));