Question

我有一个工人阶级在后台进行图像采集。

void acq::run ()
{
    while (m_started)
    {
        blocking_call();
    }
    emit workFinished();
}

void acq::start ()
{
    m_started = true;
    run();
}

void acq::stop ()
{
    m_started = false;
}

start (); stop ()是广告位，workFinished是信号。

所以在我的UI类中，我启动了worker并将信号连接到插槽：

m_thread = new QThread;
m_worker = new acq();

m_worker->moveToThread(m_thread);

// When the thread starts, then start the acquisition.
connect(m_thread, SIGNAL (started ()), m_worker, SLOT (start ()));

// When the worker has finished, then close the thread
connect(m_worker, SIGNAL(workFinished()), m_thread, SLOT(quit()));

m_thread->start();

此时，我实现了广告位closeEvent

void UIClass::closeEvent (QCloseEvent *event)
{
    m_worker->stop(); // tell the worker to close
    m_thread->wait(); // wait until the m_thread.quit() function is called
    event->accept(); // quit the window
}

不幸的是，m_thread->wait()正在阻止。即使信号quit()被发送

由于

编辑：

我添加了这两个连接：

connect(m_worker, SIGNAL(workFinished()), m_worker, SLOT(deleteLater()));
connect(m_thread, SIGNAL(finished()), m_thread, SLOT(deleteLater()));

和Qdebug进入acq::~acq()

打印的消息证明，调用了stop，发出了workFinished，发出了deleteLater（）。

Answer 1

不同线程上的对象之间的正常信号/槽连接要求接收方对象的线程运行事件循环。

您的接收器线程理论上运行其事件循环，但事件循环正忙于执行start()槽，因为run()永远不会返回。

您需要取消阻止接收方事件循环或使用Qt::DirectConnection调用停止位置。

执行后者时，您需要知道现在在发送方线程的上下文中调用了插槽，您需要保护m_started以防止并发访问。

除了使用自己的旗帜外，您还可以使用QThread::requestInterruption()和QThread::isInterruptionRequested()

Answer 2

添加

QCoreApplication::processEvents();

到你的循环，它会工作。

你死锁的原因是对acq::run()的调用阻塞了并且没有留出时间让acq::stop()在工作线程上执行。

Answer 3

在Ralph Tandetzky和Kevin Krammer的帮助下，我终于找到了解决方案。

我没有使用m_worker->stop();关闭线程，而是在worker事件循环中使用QMetaObject::invokeMethod(m_worker, "stop", Qt::ConnectionType::QueuedConnection);和QCoreApplication::processEvents();。行为不会改变，但我希望它能防止竞争或其他问题。
我没有使用：connect(m_worker, SIGNAL(workFinished()), m_thread, SLOT(quit()));，而是使用自定义广告位：
```
connect(m_worker, &Acq::workFinished, [=]
{
    std::this_thread::sleep_for(std::chrono::milliseconds(100));
    QMetaObject::invokeMethod(m_thread, "quit", Qt::ConnectionType::DirectConnection);
});
```
我们使用DirectConnection，因为我们在无限循环之外，因此不处理该事件。
有了这个，我有一个最后的问题。 m_thread->wait正在阻止，我将读取事件，否则，我的自定义插槽将永远不会被调用。因此，我的UI类QEventLoop m_loop添加了一个事件循环就在m_thread->wait()之前，我写了m_loop.exec(); 最后，在我的自定义广告位中，我添加了m_loop.quit()
```
connect(m_worker, &Acq::workFinished, [=]
{
    std::this_thread::sleep_for(std::chrono::milliseconds(100));
    QMetaObject::invokeMethod(m_thread, "quit", Qt::ConnectionType::DirectConnection);
    m_loop.quit();
});
```
m_loop.exec()进程事件，直到调用quit m_loop.quit()为止。使用该方法，我甚至不需要m_thread->wait()，因为在m_loop.quit()发出时会调用workFinished。我不再需要QMetaObject::invokeMethod(m_thread, "quit", Qt::ConnectionType::DirectConnection);

现在它就像一个魅力

编辑：这个解决方案非常沉重和丑陋，Qt（https://www.qtdeveloperdays.com/sites/default/files/David%20Johnson%20qthreads.pdf）在我的情况下使用子类和requestInteruption。

正确终止QThread

3 个答案: