【问题标题】:Worker threads with shared resources in Qt applicationQt 应用程序中具有共享资源的工作线程
【发布时间】:2019-01-21 03:21:30
【问题描述】:

我正在开发一个涉及与一个或多个设备进行串行通信的 Qt 应用程序。有不同的程序可以同时执行,每个程序可以向设备发送一个或未知数量的命令,并可以接收数据作为响应。为了更清楚,这里是场景的图形说明:

点击一个按钮会触发相应过程的执行。 因此,当用户在短时间内单击两个或多个按钮时,可能会同时运行两个或多个不同的程序。实际上,它们之间唯一可以共享的是与单个设备的串行通信;否则它们大多是相互独立的。下面是两个伪代码示例,说明程序的外观:

程序 A:

begin
write command a1 on serial port
wait for one second
perform some computations
write command a2 on serial port
wait for one second
end

程序 B:

begin
while true:
    write command b1 on serial port
    read the response from serial port
    perform some computations
    if a condition holds return, otherwise continue
end

我的解决方案及其问题:

为了简化情况,我们只需要与一台设备进行通信。由于程序可以同时执行(一次只有一个程序可以通过串行端口与设备通信),我为每个程序创建了一个线程和一个工作人员类,并将工作人员移动到相应的线程。为了在访问串行端口时同步程序,我创建了一个互斥锁:

MainWindow.h

class MainWindow : public QMainWindow {

public:
    //...
    QSerialPort*    serial_;
    QMutex      serial_mutex_;

private:
    //...
    ProcAWorker*    proca_worker;
    ProcBWorker*    procb_worker;
    ProcCWorker*    procc_worker;
    ProcDWorker*    procd_worker;

    QThread     proca_thread;
    QThread     procb_thread;
    QThread     procc_thread;
    QThread     procd_thread;

}

MainWindow.cpp

void MainWindow::onConnectButtonClicked()
{
    serial_ = new QSerialPort();
    // configure serial port settings

    serial_->open(QIODevice::ReadWrite);
}

void MainWindow::onButtonAClicked()
{
    proca_worker = new ProcAWorker(0, this);   // pass a pointer to this class to be able to access its methods and members
    proca_worker->moveToThread(&proca_thread);

    // setup worker-thread connections: started, quit, finished, etc.

    proca_thread.start();    // triggers `proccess` slot in proca_worker
}

// same thing for other buttons and procedures

ProcAWorker.cpp

void ProcAWorker::ProcAWorker(QObject *parent, QMainWindow *wnd) :
    QObject(parent), wnd_(wnd)
{

}

void ProcAWorker::process()
{
    wnd_->serial_mutex_->lock();
    wnd_->serial_->write('Command a1');   // Warning occurs in this line
    bool write_ok = client_->serial_->waitForBytesWritten(SERIAL_WRITE_TIMEOUT);
    wnd_->serial_mutex_->unlock();

    QThread::sleep(1);
    // perform some computations

    wnd_->serial_mutex_->lock();
    wnd_->serial_->write('Command a2');
    bool write_ok = client_->serial_->waitForBytesWritten(SERIAL_WRITE_TIMEOUT);
    wnd_->serial_mutex_->unlock();

    if (write_ok) {
        // signal successful to main window
        emit success();
    }
}

但是,当在串行端口(即wnd_->serial_->write('Command a1');)上执行写操作时,会显示以下警告:

QObject:无法为不同的父级创建子级 线。 (父级为 QSerialPort(0x18907d0),父级线程为 QThread(0x13cbc50),当前线程为QThread(0x17d8d08)

我的问题:

1) 我已经在 Stackoverflow 上查看了有关此警告的其他问题,但他们的回答只提到应该使用信号/插槽。我熟悉使用信号/插槽与工作线程进行通信。但是,我无法弄清楚如何使用信号/插槽来实现我的特定场景(同时运行具有串行端口等共享资源的程序),或者如何修改我当前的解决方案来解决这个问题? 请注意,应允许这些过程并行运行(除非在它们想要与设备通信的那些时刻)。显然可以按顺序运行这些程序(即一个接一个),但我不是在寻找这样的解决方案。

2) 实际上还有一个“Halt”按钮,可以停止所有正在运行的程序并向设备发送停止命令。但我也想不出实现这个功能(设置标志、发送退出信号等)。在这方面您能否也给我一些提示?

【问题讨论】:

  • 关于错误消息"QObject: Cannot create children for...",您可以使用qInstallMessageHandler 安装您自己的消息处理程序并找出消息的确切来源。
  • @G.M.谢谢你提到这一点。但是您认为这会有所帮助吗,因为目前我知道它是由wnd_->serial_->write('Command a1'); 行提出的(或者我错了吗?!)?
  • 您可以拥有一个管理函数指针列表(或一般函子)的线程,而不是为每个按钮单击创建一个新线程。当列表不为空时,它可以从列表中删除一个函数然后调用它,这样您的onButtonAClicked() 函数只需在按下某个键时将相关函数指针推到该列表上。
  • @today 是的,但消息可能不是直接来自write 电话。这实际上只是为了提供有关该消息的根本原因的更多信息。
  • 请注意,大多数QIODevice 类的设计方式是您确实不需要使用线程。只需连接到他们的各种signals 并以您需要的任何方式处理它们。

标签: c++ multithreading qt serial-port serial-communication


【解决方案1】:

首先,您不需要显式多线程(它是可选的),其次您不需要任何手动管理的同步原语。

然后,使用状态机对每个过程进行建模。希望通信协议允许每个程序识别对自己命令的响应,这样即使您将传入数据复制到所有程序,它们也会忽略与它们无关的数据。

This answer 有一个解决方案的草图,它完全符合您的要求,没有多路复用。当您通过local pipes 公开QIODevice 时,多路复用QIODevice 是微不足道的:从端口传入的所有内容都写入一个或多个本地管道的一端。从管道传入的所有内容都写入端口。只要您以Unbuffered 模式打开它们的程序端,管道就会保持数据包的完整性。这样每个write 将作为一个连续的字节块到达串行端口,并以相同的方式写入端口。

您将如何复用?像这样:

class IODeviceMux : public QObject {
  Q_OBJECT
  QVector<QPointer<AppPipe>> m_portPipes;
  QVector<QPointer<AppPipe>> m_userPipes;
  QPointer<QSerialPort> m_port;
public:
  IODeviceMux(QObject *parent = {}) : QObject(parent) {}
  void setPort(QIODevice *port) {
    if (m_port) {
      disconnect(m_port.get(), 0, this, 0);
      m_userPipes.removeAll({});
      for (auto pipe : qAsConst(m_userPipes))
        disconnect(m_port.get(), 0, pipe.get(), 0);
    }
    m_port = port;
    connect(m_port.get(), &QIODevice::readyRead, this, &IODeviceMux::onPortRead);
  }
  AppPipe *getPipe() {
    QScopedPointer<AppPipe> user(new AppPipe(QIODevice::ReadWrite | QIODevice::Unbuffered));
    auto *port = new AppPipe(QIODevice::ReadWrite | QIODevice::Unbuffered, this);
    user->addOther(port);
    connect(port, &QIODevice::readyRead, this, &IODeviceMux::onPipeRead);
    connect(m_port.get(), &QIODevice::bytesWritten, user.get(), &QIODevice::bytesWritten);
    connect(user, &QObject::destroyed, port, &QObject::deleteLater);
    m_userPipes.push_back(user.get());
    m_portPipes.push_back(port);
    return user.take();
  } 
private:
  void onPortRead() {
    if (!m_port) return;
    auto data = m_port->readAll();
    m_portPipes.removeAll({});
    for (auto pipe : qAsConst(m_portPipes))
      pipe->write(data);
  }
  void onPipeRead() {
    auto *pipe = qobject_cast<AppPipe*>(sender());
    QByteArray data;
    if (pipe) data = pipe->readAll();
    if (m_port) m_port->write(data);
  }
};

每个过程都将getPipe() 并将管道视为串行端口设备。管道中的每个write 都会在端口上忠实执行。端口上的每个readyRead 都被忠实地转发,相同的数据量可以立即读取。甚至端口的bytesWritten 也被转发。但是bytesToWrite 不起作用——它总是返回零。这可以通过向AppPipe 添加一个选项来查询该值来解决。

我想这就是让它工作所需的一切。

【讨论】:

  • 谢谢。我考虑将程序实现为状态机,但我不确定并认为必须有更好的解决方案(似乎没有)。所以那部分没有问题。但是,对于串行通信,我有另一个我不确定的设计:创建一个串行工作类并将其移动到一个新线程。然后创建像write_data 和read_data 这样的信号/槽,串行工作者和过程工作者使用它们相互通信。这样可以保证一次只有一个程序可以访问串行端口(因为事件循环>>>
  • >>> 的串行工作者一一处理事件)。这样共享资源的问题就解决了。你提到的只有一个问题:通信协议没有区分不同程序的响应。为了解决这个问题,我认为我们可以向每个过程发出的写入信号传递一个额外的参数,例如唯一的过程 id。这样,串行工作者作为另一个信号发回的响应也可以将此 id 作为其参数之一,并且将被不相关的程序忽略。你觉得这个设计怎么样?
  • @today 当串行通信完全异步处理时,它将对 gui 影响最小。当然,您希望它脱离 UI 线程,但它可以在另一个处理一般通信的线程中,并且绝对不需要每个过程或每个端口或类似的任何线程。
  • 这样想:一个 UI 对每个可能发生的事件都有许多过程(响应)——用户按下按钮、时间在动画中传递等。你的过程做同样的事情事物:它们响应事件,即数据变得可用或时间流逝等。它们完成大部分工作并将控制权返回给事件循环。他们从不阻止。
  • 长时间计算应该使用QtConcurrent::run同时完成,这样您就不会手动执行线程管理来处理批量计算作业。机器更擅长 :) 计算完成后,未来观察者的信号使程序再次向前推进。为所有通信设置一个专用线程是有意义的,但是仅仅因为您偶尔需要长时间运行的计算而拥有其中的许多线程是一种浪费——这些线程大部分都处于空闲状态,而这只是无缘无故地浪费 RAM。
猜你喜欢
  • 1970-01-01
  • 1970-01-01
  • 1970-01-01
  • 1970-01-01
  • 1970-01-01
  • 1970-01-01
  • 1970-01-01
  • 2012-09-28
  • 1970-01-01
相关资源
最近更新 更多