3

我有一个更大的项目，有一个 GUI，我想在后台管理一些文件。我已经为此任务实现了一个新线程，并且在运行时，一切都很好。但是一旦我退出应用程序，就会visual-leak-detector发现 3-7 个内存泄漏。

我分离了我的线程代码并创建了一个新项目以使用最小的代码示例进行检查，但我仍然无法解决我的问题。

我认为这与主程序的事件循环有关。也许循环没有处理最后一个事件来删除我的线程类和线程本身。因为我在析构函数中停止并退出线程。但我不确定这一点。

这是我的最小代码：threadclass.hpp：

#include <QObject>
#include <QDebug>

class ThreadClass : public QObject {
    Q_OBJECT

public:
    explicit ThreadClass() {}
    virtual ~ThreadClass(){
        qDebug() << "ThreadClass Destructor";
    }

signals:
    // emit finished for event loop
    void finished();

public slots:
    // scan and index all files in lib folder
    void scanAll(){
        for(long i = 0; i < 10000; i++){
            for (long k = 0; k < 1000000; k++);
            if(i%500 == 0)
                qDebug() << "thread: " << i;
        }
    }
    // finish event loop and terminate
    void stop(){
        // will be processed after scanall is finished
        qDebug() << "STOP SIGNAL --> EMIT FINSIHED";
        emit finished();
    }
};

线程处理程序.hpp：

#include <QObject>
#include <QThread>
#include "threadclass.hpp"

class ThreadHandler : public QObject {
    Q_OBJECT

public:
    explicit ThreadHandler(QObject *parent = 0) : parent(parent), my_thread(Q_NULLPTR) {}

    virtual ~ThreadHandler() {
        // TODO Check!
        // I think I don't have to delete the threads, because delete later
        // on finish signal. Maybe I just have to wait, but then how do I
        // check, if thread is not finished? Do I need to make a bool var again?

        if (my_thread != Q_NULLPTR && my_thread->isRunning())
        {
            emit stopThread();
            //my_thread->quit();
            my_thread->wait();
            //delete my_thread;
        }

        qDebug() << "ThreadHandler Destructor";
        my_thread->dumpObjectInfo();
    }

    void startThread(){
        if (my_thread == Q_NULLPTR)
        {
            my_thread = new QThread;
            ThreadClass *my_threaded_class = new ThreadClass();
            my_threaded_class->moveToThread(my_thread);

            // start and finish
            QObject::connect(my_thread, &QThread::started, my_threaded_class, &ThreadClass::scanAll);
            QObject::connect(this, &ThreadHandler::stopThread, my_threaded_class, &ThreadClass::stop);

            // finish cascade
            // https://stackoverflow.com/a/21597042/6411540
            QObject::connect(my_threaded_class, &ThreadClass::finished, my_threaded_class, &ThreadClass::deleteLater);
            QObject::connect(my_threaded_class, &ThreadClass::destroyed, my_thread, &QThread::quit);
            QObject::connect(my_thread, &QThread::finished, my_thread, &QThread::deleteLater);

            my_thread->start();
        }
    }

signals:
    void stopThread();

private:
    QObject *parent;
    QThread* my_thread;
};

main.cpp 很糟糕，但似乎很好地模拟了我的主程序的行为：

#include <QCoreApplication>
#include <QTime>
#include <QDebug>
#include "threadhandler.hpp"

#include <vld.h>

int main(int argc, char *argv[]) {
    QCoreApplication a(argc, argv);

    ThreadHandler *th = new ThreadHandler();
    th->startThread();

    // wait (main gui programm)
    QTime dieTime= QTime::currentTime().addSecs(5);
    while (QTime::currentTime() < dieTime) {
        QCoreApplication::processEvents(QEventLoop::AllEvents, 100);
    }

    qDebug() << "DELETE TH";
    delete th;
    qDebug() << "FINISH ALL EVENTS";
    QCoreApplication::processEvents(QEventLoop::AllEvents, 500);
    qDebug() << "QUIT";
    QCoreApplication::quit();
    qDebug() << "CLOSE APP";
    // pause console
    getchar();
//    return a.exec();
}

这是 VLD 的输出：

WARNING: Visual Leak Detector detected memory leaks!
...
turns out this is very boring and uninteresting
...
Visual Leak Detector detected 3 memory leaks (228 bytes).
Largest number used: 608 bytes.
Total allocations: 608 bytes.
Visual Leak Detector is now exiting.
The program '[8708] SOMinimalExampleThreads.exe' has exited with code 0 (0x0).

更新1：我添加qDebug() << "ThreadClass Destructor";到析构函数中ThreadClass，我的新输出如下所示：

...
thread:  9996
thread:  9997
thread:  9998
thread:  9999
ThreadHandler Destructor
FINISH ALL EVENTS
CLOSE APP

现在很明显，我的线程类的析构函数从未被调用，因此丢失在 void 中。但是为什么这不起作用呢？

QObject::connect(my_threaded_class, &ThreadClass::finished, my_threaded_class, &ThreadClass::deleteLater);

更新 2：我在 ThreadHandler 中发现了一个问题：

emit stopThread();
my_thread->quit(); // <-- stops the event loop and therefore no deletelater
my_thread->wait();

我删除my_thread->quit()了，现在调用了 ThreadClass 的析构函数，但my_thread->wait()从未完成。

4

2 回答 2

1

我发现该my_thread->wait()函数阻止了事件循环，因此退出和deleteLater级联从未完成。我用另一种等待完成线程的方法解决了这个问题：

removed

这是 Mike 新实施的解决方案。这很容易实现，我只需要my_threaded_class::stop在threadhandler课堂上更改连接即可。

#include <QObject>
#include <QThread>
#include "threadclass.hpp"

class ThreadHandler : public QObject {
    Q_OBJECT

public:
    explicit ThreadHandler(QObject *parent = 0) : parent(parent), my_thread(Q_NULLPTR) {}

    virtual ~ThreadHandler() {

        if (my_thread != Q_NULLPTR && my_thread->isRunning())
        {
            my_thread->quit();
            my_thread->wait();
        }

        qDebug() << "ThreadHandler Destructor";
    }

    void startThread(){
        if (my_thread == Q_NULLPTR)
        {
            my_thread = new QThread;
            ThreadClass *my_threaded_class = new ThreadClass();
            my_threaded_class->moveToThread(my_thread);

            // start and finish
            QObject::connect(my_thread, &QThread::started, my_threaded_class, &ThreadClass::scanAll);
            // https://stackoverflow.com/questions/53468408
            QObject::connect(my_thread, &QThread::finished, my_threaded_class, &ThreadClass::stop);

            // finish cascade
            // https://stackoverflow.com/a/21597042/6411540
            QObject::connect(my_threaded_class, &ThreadClass::finished, my_threaded_class, &ThreadClass::deleteLater);
            QObject::connect(my_threaded_class, &ThreadClass::destroyed, my_thread, &QThread::quit);
            QObject::connect(my_thread, &QThread::finished, my_thread, &QThread::deleteLater);

            my_thread->start();
        }
    }

signals:
    void stopThread();

private:
    QObject *parent;
    QThread* my_thread;
};
于 2018-11-25T19:13:36.463 回答
1

问题描述：

当从主线程ThreadHandler发出的析构函数发出时，Qt通过将事件发布到工作线程的事件循环（也称为排队连接）来调用连接的插槽（）。这意味着当此信号发出时，worker 的事件循环需要准备好接收新事件（因为您依赖它来执行适当的清理）。但是，正如您已经发现的那样，调用可能会导致事件循环提前退出（在工作线程开始调用之前，因此不会发出信号）。您可能想要检查这个重现我正在谈论的行为的最小示例的输出：stopThread&ThreadClass::stopthread->quit()ThreadClass::stopThreadClass::finished

#include <QtCore>

/// lives in a background thread, has a slot that receives an integer on which
/// some work needs to be done
struct WorkerObject : QObject {
  Q_OBJECT
public:
  using QObject::QObject;
  Q_SLOT void doWork(int x) {
    // heavy work in background thread
    QThread::msleep(100);
    qDebug() << "working with " << x << " ...";
  }
};

/// lives in the main thread, has a signal that should be connected to the
/// worker's doWork slot; to offload some work to the background thread
struct ControllerObject : QObject {
  Q_OBJECT
public:
  using QObject::QObject;
  Q_SIGNAL void sendWork(int x);
};

int main(int argc, char *argv[]) {
  QCoreApplication a(argc, argv);

  QThread thread;
  WorkerObject workerObj;
  workerObj.moveToThread(&thread);
  // quit application when worker thread finishes
  QObject::connect(&thread, &QThread::finished, &a, &QCoreApplication::quit);
  thread.start();

  ControllerObject controllerObj;
  QObject::connect(&controllerObj, &ControllerObject::sendWork, &workerObj,
                   &WorkerObject::doWork);

  for (int i = 0; i < 100; i++) {
    QThread::msleep(1);
    // call thread.quit() when i is equal to 10
    if (i == 10) {
      thread.quit();
    }
    controllerObj.sendWork(i);
  }
  return a.exec();
}

#include "main.moc"

在我的机器上，这是一个可能的输出：

working with  0  ...
working with  1  ...
working with  2  ...
working with  3  ...

请注意，尽管在thread.quit()主线程的第十次迭代中调用，工作线程可能不会处理退出调用之前收到的所有消息（并且我们将值3作为工作人员处理的最后一个值）。*

解决方案：

实际上，Qt 提供了一种退出工作线程并执行必要清理的规范方法，因为信号QThread::finished以特殊方式处理：

当这个信号发出时，事件循环已经停止运行。除了延迟删除事件之外，线程中不会再处理任何事件。该信号可以连接到 QObject::deleteLater()，以释放该线程中的对象。

这意味着您可以使用thread->quit()（与您相同的方式），但您只需要添加：

connect(my_thread, &QThread::finished, my_threaded_class, &ThreadClass::stop);

到你的并从析构函数startThread中删除不必要的。emit stopThread();

*我在文档中找不到任何详细解释此行为的页面，因此我提供了这个最小示例来解释我在说什么。

于 2018-11-25T21:49:35.313 回答