QIODevice报错‘device not open’：请检查QSerialPort操作代码

问题复现 #

按钮触发的串口操作槽函数执行时，抛出错误：QIODevice::read (QSerialPort): device not open，原代码如下：

void MACH3::on_pushButton_clicked()
{
    serial = new QSerialPort(this);

    serial->setBaudRate(QSerialPort::Baud115200);
    serial->setDataBits(QSerialPort::Data8);
    serial->setParity(QSerialPort::NoParity);
    serial->setStopBits(QSerialPort::OneStop);
    serial->setFlowControl(QSerialPort::NoFlowControl);

    serial->open(QIODevice::ReadWrite);

    QString time = QTime::currentTime().toString();
    QString elde = ui->data->toPlainText();
    QString eldetut = ui->data->toPlainText();
    QByteArray inBytes;
    const char *cStrData;
    inBytes = elde.toUtf8();
    cStrData = inBytes.constData();
    QString qtStrData;
    qtStrData = QString::fromUtf8(cStrData);

    QString gelen = serial->readAll();

    if(gelen.length()>0)
    {
        QString eldetut = ui->data->toPlainText();
        QString time2 = QTime::currentTime().toString();
        ui->data->setText(eldetut+"\n"+time2 + "\n" + "received-> "+gelen + "\n");
    }
}

问题分析 #

1. 未指定目标串口：仅创建QSerialPort对象，未设置要打开的串口设备名（如Windows的COM3、Linux的/dev/ttyUSB0），系统无法定位硬件串口，导致open()调用失败。
2. 未校验打开结果：调用open()后不检查返回值，直接执行readAll()，失败后读取操作触发设备未打开的错误。
3. 内存泄漏：每次点击按钮都会新建QSerialPort实例，旧实例未释放，多次点击会造成堆内存泄漏。
4. 冗余无效代码：字符串转码的循环操作（QString→QByteArray→const char*→QString）完全无意义，属于无效代码。
5. 读取时机错误：串口刚打开就立即读取，此时硬件还未传输数据，几乎不可能读取到内容。

修复后的代码 #

// 建议在头文件中将serial声明为类成员变量：QSerialPort *serial;

void MACH3::on_pushButton_clicked()
{
    // 避免重复创建对象，先检查串口是否已打开
    if (serial && serial->isOpen()) {
        ui->data->append("串口已处于打开状态");
        return;
    }

    if (!serial) {
        serial = new QSerialPort(this);
    }

    // 设置要打开的串口设备名，根据实际硬件修改
    serial->setPortName("COM3"); // Linux环境改为"/dev/ttyUSB0"这类路径

    // 配置串口参数
    serial->setBaudRate(QSerialPort::Baud115200);
    serial->setDataBits(QSerialPort::Data8);
    serial->setParity(QSerialPort::NoParity);
    serial->setStopBits(QSerialPort::OneStop);
    serial->setFlowControl(QSerialPort::NoFlowControl);

    // 尝试打开串口并检查结果
    if (!serial->open(QIODevice::ReadWrite)) {
        ui->data->append(QString("串口打开失败：%1").arg(serial->errorString()));
        return;
    }

    ui->data->append(QTime::currentTime().toString() + " 串口已成功打开");

    // 连接readyRead信号，有数据时触发读取
    connect(serial, &QSerialPort::readyRead, this, [this](){
        QByteArray receivedData = serial->readAll();
        if (!receivedData.isEmpty()) {
            ui->data->append(QTime::currentTime().toString() + "\nreceived-> " + QString(receivedData) + "\n");
        }
    });
}

// 析构函数中释放串口资源
MACH3::~MACH3()
{
    if (serial && serial->isOpen()) {
        serial->close();
    }
    delete serial;
    delete ui;
}

关键修复点说明 #

• 指定串口设备名：必须通过setPortName()明确要操作的串口，设备名需匹配实际连接的硬件。
• 校验打开状态：调用open()后判断返回值，失败时输出错误信息，避免后续无效操作。
• 避免重复创建：将serial改为类成员变量，重复点击按钮时先检查状态，防止内存泄漏。
• 信号槽异步读取：通过readyRead信号触发读取操作，确保只有串口有数据时才执行读取，避免无效调用。
• 清理冗余代码：移除无意义的字符串转码操作，简化逻辑。

内容的提问来源于stack exchange，提问作者TIMBO 2023