0

快速描述情况

我试图让一个最小的 GUI 启动一个无休止的进程，该进程通过 CAN 总线通过自定义协议进行通信。

根据我在这里阅读的内容，我的代码结构如下：

一方面，我有一个类使用 2 个简单的按钮“开始”和“停止”来处理我的 GUI，即 MainWindow。

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

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

关于信号和插槽

我已将按钮信号 (released()) 连接到来自控制器的信号。所以 GUI 不知道到底是什么开始了。

这些控制器的信号连接到来自 Worker 的插槽。这些插槽用于启动和停止该过程。

关于线程

Worker 实例存在于它自己的 QThread 中。可能涉及其他任务，所以我认为最好在自己的线程中处理每个任务。

一开始，worker 的进程是通过信号/槽来处理的，这些信号/槽使状态机在关于转换的状态之间进化。由于信号/槽机制，如果我是正确的，线程的事件循环可以处理其队列中的事件。

问题

我的启动信号已正确发送给工作人员，启动进程并因此启动状态机。这台机器是循环的，直到用户请求停止信号。但是，当用户单击“停止”按钮时，关联的插槽不会被调用。同时，机器继续无休止地运行并且看不到停止请求（我已经放了一些调试消息来查看真正执行了什么）。

代码片段

这是代码片段。主窗口.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

主窗口.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;
}

控制器.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 处理带有枚举State的状态机。Transition工人.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;
    }
}

。轮廓

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 是否正确发送了信号并且该槽确实已执行。但该stopProcess()槽仍未被调用。）

有什么想法吗 ？

4

1 回答 1

2

正如 Oktalist 所指出的，问题在于您永远不会在工作线程中返回 Qt 的事件循环。默认情况下，Qt 使用 a Qt::AutoConnection，Qt::DirectConnection如果接收者位于同一个线程中，则为 a 。因此，QtprocessEvent以无限的方式递归调用。

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

正如您所建议stopProcess的，Controller直接调用可能会解决您的问题，但不是线程安全的。您可以定义stopRequested为volatile，但这仅适用于 windows，并且可能适用于其他情况。

更好的方法是将它定义为std::atomic好像 C++11 是您的选择。

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

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

这种方法的优点是所有线程安全问题都由 Qt 透明地处理，但这会使您的状态机稍微变慢。

于 2017-06-01T14:15:48.730 回答