我正在尝试使用最小的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处理名为State
和Transition
的状态机。
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}插槽仍然未被调用。)
有什么想法吗?
答案 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可以透明地处理所有线程安全问题,但这会使你的状态机变慢。