为什么打开串口会发送两次指令（加强版）

之前代码：

void MainWindow::on_btnSend_clicked()
{
    if (!modbusRTU || !modbusRTU->isOpen()) {
        QMessageBox::critical(this, "错误", "请先打开串口");
        return;
    }

    ui->btnSend->setEnabled(false); // 禁用按钮防止重复点击

    // 异步发送避免阻塞
    QTimer::singleShot(0, this, [this]() {
        bool ok;
        QString input = ui->txtSend->toPlainText().trimmed();
        uint16_t pressure = input.toUShort(&ok);

        if (!ok) {
            QMetaObject::invokeMethod(this, [this]() {
                QMessageBox::critical(this, "错误", "无效的压力值");
                ui->btnSend->setEnabled(true);
            });
            return;
        }

        modbusRTU->clearBuffers();
        bool success = modbusRTU->writeSingleRegister(0x0002, pressure);

        QMetaObject::invokeMethod(this, [this, success, pressure]() {
            if (success) {
                ui->txtRead->append(QString("[发送成功] 压力: %1 kPa").arg(pressure));
            } else {
                ui->txtRead->append("[发送失败] 请检查设备连接");
            }
            ui->btnSend->setEnabled(true);
        });
    });
}

之后代码：

    if (!modbusRTU || !modbusRTU->isOpen()) {
        QMessageBox::critical(this, "Error", "Serial port is not open.");
        return;
    }

    bool ok;
    QString input = ui->txtSend->toPlainText();
    uint16_t pressure = input.toUInt(&ok);

    if (!ok || input.isEmpty()) {
        QMessageBox::critical(this, "Error", "Invalid pressure value. Please enter a valid number.");
        return;
    }

    // 清理缓冲区
    modbusRTU->clearBuffers();

    // 禁用按钮，防止重复点击
    ui->btnSend->setEnabled(false);

    // 发送指令
    if (modbusRTU->writeSingleRegister(0x0002, pressure)) {
        ui->txtRead->append(QString("Set pressure to: %1 kPa").arg(pressure));
    } else {
        ui->txtRead->append("Failed to set pressure.");
    }

    // 启用按钮
    ui->btnSend->setEnabled(true);

 

改进说明

1. 禁用按钮：

   • 在发送指令之前，禁用发送按钮（ui->btnSend->setEnabled(false);），防止用户在指令发送过程中重复点击按钮。

   • 这确保了每次点击按钮时，writeSingleRegister 方法只被调用一次。

2. 启用按钮：

   • 在发送指令完成后，重新启用按钮（ui->btnSend->setEnabled(true);），以便用户可以进行下一次操作。

为什么现在可以了

通过禁用和启用按钮，确保每次点击按钮时，程序只会调用一次 writeSingleRegister 方法。这样可以避免因用户多次点击按钮而导致的重复发送指令问题。

推荐添加的改进

1. 超时时间调整：

   • 增加 waitForBytesWritten 和 waitForReadyRead 方法的超时时间，确保程序有足够的时间等待数据发送和接收。

   cpp复制

   if (!serialPort->waitForBytesWritten(1000)) { // 增加超时时间到 1000 毫秒
       emit errorOccurred(QString("Timeout writing to port: %1").arg(serialPort->errorString()));
       return false;
   }

2. 日志记录：

   • 在关键步骤添加日志，记录发送和接收的数据，便于调试。

   cpp复制

   qDebug() << "Sending request:" << request.toHex();
   qDebug() << "Received response:" << response.toHex();

通过以上改进，可以确保程序只发送一次指令，并且正确处理串口通信。