为什么事件队列不处理我的Qt信号?

时间:2017-06-01 12:49:11

标签: c++ multithreading qt qt5.8

情况的快速描述

我正在尝试使用最小的GUI启动无休止的过程,通过CAN总线通过自定义协议进行通信。

根据我所阅读的内容here,我按如下方式编写了代码:

我一方面有一个处理我的GUI的类,有两个简单的按钮“start”和“stop”,即MainWindow。

另一方面,一个使用上面链接中描述的状态机管理我的自定义协议的类,即Worker。

在这些中间,我有一个控制器将整体链接在一起。此控制器在这里是因为处理了一些其他任务,但这不是本文的目的。

关于信号和插槽

我已将按钮信号(已发布())连接到来自控制器的信号。因此GUI不知道究竟是从什么开始的。

那些Controller的信号连接到Worker的插槽。这些插槽用于启动和停止该过程。

关于线程

Worker实例存在于自己的QThread中。可能涉及其他任务,所以我发现在自己的线程中处理每个任务会更好。

一开始,工作人员的过程通过信号/插槽来处理,这使得状态机在状态机上就过渡进行了演变。由于信号/插槽机制,如果我是正确的,线程的事件循环可以处理来自其队列的事件。

问题

我的启动信号被正确发送给工作人员,启动进程,从而启动状态机。该机器是循环的,直到用户请求停止信号。但是,当用户点击“停止”按钮时,不会调用关联的插槽。同时,机器继续无休止地运行,并没有看到停止请求(我已经放了一些调试消息来查看真正执行的内容)。

代码段

以下是代码段。 MainWindow.h

#ifndef MAINWINDOW_H
#define MAINWINDOW_H

#include <QMainWindow>

#include "controller.h"

class QPushButton;
class QWidget;
class QVBoxLayout;

class MainWindow : public QMainWindow
{
    Q_OBJECT

public:
    explicit MainWindow(Controller& controller, QWidget *parent = 0);
    ~MainWindow();

private:
    Controller& controller;

    QPushButton* startButton;
    QPushButton* stopButton;
    QWidget* centralWidget;
    QVBoxLayout* layout;
};

#endif // MAINWINDOW_H

MainWindow.cpp

#include "mainwindow.h"

#include <QWidget>
#include <QVBoxLayout>
#include <QPushButton>

MainWindow::MainWindow(Controller &controller, QWidget *parent) :
    QMainWindow(parent), controller(controller)
{
    centralWidget = new QWidget(this);
    setCentralWidget(centralWidget);

    layout = new QVBoxLayout();
    startButton = new QPushButton("START", this);
    stopButton = new QPushButton("STOP", this);

    layout->addWidget(startButton);
    layout->addWidget(stopButton);

    centralWidget->setLayout(layout);

    connect(startButton, SIGNAL(released()), &controller, SIGNAL(startSignal()));
    connect(stopButton, SIGNAL(released()), &controller, SIGNAL(stopSignal()));
}

MainWindow::~MainWindow()
{
    delete stopButton;
    delete startButton;
    delete layout;
    delete centralWidget;
}

或者Controller.h

#ifndef CONTROLLER_H
#define CONTROLLER_H

#include <QObject>
#include <QThread>

class MainWindow;
class Worker;

class Controller : public QObject
{
    Q_OBJECT
public:
    Controller();
    virtual ~Controller();

signals:
    void startSignal() const;
    void stopSignal() const;

private:
    MainWindow* mainWindow;

    QThread workerThread;
    Worker* worker;
};

#endif // CONTROLLER_H

Controller.cpp(继承公共QObject)

#include "controller.h"

#include "mainwindow.h"
#include "worker.h"

Controller::Controller()
{
    mainWindow = new MainWindow(*this);
    mainWindow->show();

    worker = new Worker();
    worker->moveToThread(&workerThread);
    connect(this, SIGNAL(startSignal()), worker, SLOT(startProcess()));
    connect(this, SIGNAL(stopSignal()), worker, SLOT(stopProcess()));
    workerThread.start();
}

Controller::~Controller()
{
    workerThread.quit();
    workerThread.wait();

    delete worker;
    delete mainWindow;
}

Worker处理名为StateTransition的状态机。 Worker.h

#ifndef WORKER_H
#define WORKER_H

#include <QObject>

class Worker : public QObject
{
    Q_OBJECT
public:
    enum State { IDLE, STATE_1, STATE_2 };
    enum Transition { OK, ERROR };
    enum Mode { MODE_1, MODE_2 };
    explicit Worker();

    void read();

public slots:
    void startProcess();
    void stopProcess();

    void processEvent(const Transition& transition);

signals:
    void sendSignal(const Transition& transition) const;

private:
    State currentState;
    Mode selectedMode;
    bool stopRequested;
};

#endif // WORKER_H

Worker.cpp(继承公共QObject)

#include "worker.h"

#include <QDebug>
#include <QThread>

Worker::Worker() : QObject()
{
    stopRequested = false;
    currentState = IDLE;

    connect(this, SIGNAL(sendSignal(Transition)), this, SLOT(processEvent(Transition)));
}

void Worker::read()
{
    qDebug() << "Reading...";
    QThread::msleep(500);
    emit sendSignal(OK);
}

void Worker::startProcess()
{
    qDebug() << "Start requested";
    selectedMode = MODE_1;
    stopRequested = false;
    emit sendSignal(OK);
}

void Worker::stopProcess()
{
    qDebug() << "Stop requested";
    stopRequested = true;
}

void Worker::processEvent(const Worker::Transition &transition)
{
    qDebug() << "Process event";
    switch(currentState) {
    case IDLE:
        switch(selectedMode) {
        case MODE_1:
            currentState = STATE_1;
            read();
            break;
        case MODE_2:
            currentState = STATE_2;
            break;
        }
        break;
    case STATE_1:
        if (!stopRequested) {
            if (transition == OK) {
                read();
            } else {
                currentState = IDLE;
                // No emission. The state machine stops on error
            }
        }
        break;
    case STATE_2:
        // Not implemented yet
        break;
    }
}

.pro文件

QT       += core gui

greaterThan(QT_MAJOR_VERSION, 4): QT += widgets

TARGET = sample_project
TEMPLATE = app

DEFINES += QT_DEPRECATED_WARNINGS

SOURCES += main.cpp\
        mainwindow.cpp \
    controller.cpp \
    worker.cpp

HEADERS  += mainwindow.h \
    controller.h \
    worker.h

免责声明代码无法正常退出。最好在你的IDE中启动它,这样你就可以轻松杀死它。

这些代码片段是使用Qt5.8.0 MinGW 32位构建的。 要重现该问题,只需点击“开始”,调试消息就会出现在控制台中。然后点击“停止”,消息不断出现,不要按原样停止。

我找到了一种解决方法,直接从stopProcess()调用Controller而不是使用信号。这样做会正确设置stopRequested并停止该过程。

虽然,我想知道为什么事件队列永远不会处理来自Controller的信号?即使使用信号/插槽处理状态机,也允许事件队列在事件到达时处理它们。

(我已经尝试在Controller中放置一个中间槽,将信号发送到Worker以查看GUI是否正确发送了信号并确实执行了此槽。但{{1}插槽仍然未被调用。)

有什么想法吗?

1 个答案:

答案 0 :(得分:2)

正如Oktalist所指出的,问题是你永远不会回到你的工作线程中的Qt事件循环。默认情况下,Qt使用Qt::AutoConnection,如果接收器位于同一个线程中,则为Qt::DirectConnection。所以,Qt以无限的方式递归地调用processEvent

解决方案1:从两个线程中写入/读取stopRequested

正如您的建议,直接从stopProcess调用Controller可以解决您的问题,但不是线程安全的。您可以将stopRequested定义为volatile,但这只会work on windows并且会probably work in other situations

如果C ++ 11是您的选择,更好的方法是define it as std::atomic

解决方案2:避免递归函数调用

您可以指定QObject::connect的第五个参数,您需要哪种类型的连接。选择Qt::QueuedConnection会破坏您的递归操作。通过这种方式,Qt将能够处理您的stopRequested信号。

这种方法的优点是Qt可以透明地处理所有线程安全问题,但这会使你的状态机变慢。