Данные через порт_сортировка_вывод на монитор

@SergioSV · Регистрация: 07.06.2023

Студворк — интернет-сервис помощи студентам

Здравствуйте. Нужна помощь в проекте.
Комп принимает данные через serial port, примерный вид - 1 0 1 0 0 140 +34 -23 1 0... и т. д.
Программа читает данные строки, разделяет данные по пробелу и записывает в переменные. По данным переменных выводится инфа на экран и меняется label. Данные постоянно обновляются.

При запуске кода вылетает окно Widget и аварийно завершается (ASSERT failure in QList<T>::operator[]: "index out of range", file /usr/include/x86_64-linux-gnu/qt5/Qt..ore/qlist.h, line 579 )

mainwindow.cpp

C++ (Qt)

#include "mainwindow.h"
#include "ui_mainwindow.h"
 
#include <QApplication>
#include <QCoreApplication>
#include <QSerialPort>
#include <QtSerialPort/QSerialPortInfo>
#include <QDebug>
#include <QLabel>
 
 
MainWindow::MainWindow(QWidget *parent)
    : QMainWindow(parent)
    , ui(new Ui::MainWindow)
 
{
    ui->setupUi(this);
 
    serialPort = new QSerialPort(this);
    serialPort->setPortName("/dev/ttyACM0");
    serialPort->setBaudRate(QSerialPort::Baud9600);
    serialPort->setDataBits(QSerialPort::Data8);
    serialPort->setParity(QSerialPort::NoParity);
    serialPort->setStopBits(QSerialPort::OneStop);
    serialPort->setFlowControl(QSerialPort::NoFlowControl);
    serialPort->open(QIODevice::ReadWrite);
 
    connect(serialPort, SIGNAL(readyRead()), this, SLOT(readData()));
 
    foreach (const QSerialPortInfo &info, QSerialPortInfo::availablePorts())
    {
        QSerialPort port;
        port.setPort(info);
        if (port.open(QIODevice::ReadWrite))
        {
            qDebug() << "Название: " + info.portName() + " " + info.description() + info.manufacturer();
        }
    }
}
 
 
 
 
MainWindow::~MainWindow()
{
 
    delete ui;
    serialPort->close();
    delete serialPort;
 
}
 
void MainWindow::readData()
{
 
    QByteArray dataBuffer;                                             // Создаем буфер для чтения данных
 
 
    while (serialPort->waitForReadyRead()) {                           // Читаем данные из порта
        dataBuffer.append(serialPort->readLine());
 
        QList<QByteArray> dataList = dataBuffer.split(' ');            // Разделяем данные по символу " "
        //QList<int> inList;                                               // пишем результат
 
        for (int i=0;i<dataList.size();i++)                              // перебираем все кусочки строки
        {
            int var1 = dataList[0].toInt();                                // переводим их int и записываем в новый массив
            int var2 = dataList[1].toInt();
            int var3 = dataList[2].toInt();
            int var4 = dataList[3].toInt();
            int var5 = dataList[4].toInt();
            int var6 = dataList[5].toInt();
            int var7 = dataList[6].toInt();
            int var8 = dataList[7].toInt();
            int var9 = dataList[8].toInt();
            int var10 = dataList[9].toInt();
            int var11 = dataList[10].toInt();
            int var12 = dataList[11].toInt();
            int var13 = dataList[12].toInt();
            int var14 = dataList[13].toInt();
            int var15 = dataList[14].toInt();
            int var16 = dataList[15].toInt();
            int var17 = dataList[16].toInt();
 
            qDebug() << "Переменная " << i << " : " << dataList[i];       //выводим данные
 
        }
        for (int i=0;i<dataList.size();i++)
        {
        dataList[i].clear();
        }
 
        dataBuffer.clear();                                          // Очищаем буфер данных
    }
}

mainwindow.h

C++

#ifndef MAINWINDOW_H
#define MAINWINDOW_H
 
#include <QMainWindow>
#include <QtSerialPort/QSerialPort>
#include <QtSerialPort/QSerialPortInfo>
#include <QDebug>
#include <QTimer>
#include <QCoreApplication>
#include <QtCore>
#include <QByteArray>
 
 
QT_BEGIN_NAMESPACE
namespace Ui { class MainWindow; }
QT_END_NAMESPACE
 
class MainWindow : public QMainWindow
{
    Q_OBJECT
 
public:
    MainWindow(QWidget *parent = nullptr);
    ~MainWindow();
 
private slots:
 
    void readData();
 
 
 
private:
    Ui::MainWindow *ui;
    QSerialPort *serialPort;
};
#endif // MAINWINDOW_H

@sdf45 · 28.09.2023, 20:45

строки 67-83: если элементов в списке меньше 16 или он пустой = UB

Алексей1153 · 28.09.2023, 21:15

Сообщение от SergioSV

for (int i=0;i<dataList.size();i++) // перебираем все кусочки строки
{
int var1 = dataList[0].toInt(); // переводим их int и записываем в новый массив
int var2 = dataList[1].toInt();
int var3 = dataList[2].toInt();
int var4 = dataList[3].toInt();
int var5 = dataList[4].toInt();
int var6 = dataList[5].toInt();
int var7 = dataList[6].toInt();
int var8 = dataList[7].toInt();
int var9 = dataList[8].toInt();
int var10 = dataList[9].toInt();
int var11 = dataList[10].toInt();
int var12 = dataList[11].toInt();
int var13 = dataList[12].toInt();
int var14 = dataList[13].toInt();
int var15 = dataList[14].toInt();
int var16 = dataList[15].toInt();
int var17 = dataList[16].toInt();
qDebug() << "Переменная " << i << " : " << dataList[i]; //выводим данные
}

странный цикл. В чём смысл этих строк int varN = dataList[N-1].toInt(); ? Просто убери их

Добавлено через 1 минуту

Сообщение от SergioSV

for (int i=0;i<dataList.size();i++)
{
dataList[i].clear();
}
dataBuffer.clear();

это всё тоже не нужные действия, можно удалить эти строки

@SergioSV · 29.09.2023, 10:40 **[ТС]**

Всего 17 переменных, не больше и не меньше.
Сделал так:

C++ (Qt)

        for (int i=0;i<dataList.size();i++)                              // перебираем все кусочки строки
        {
            inList.append(dataList[i].toInt());
}

Но неправильно раскладывает данные и переменные не от 1 до 17, а - 0, 1, 2, 2, 1 , 2.....

Алексей1153 · 29.09.2023, 10:54

SergioSV,

C++

#include <iostream>
#include <string>
#include <sstream>
 
int main()
{
    QByteArray ba="1 0 1 0 0 140 +34 -23 1 0";
    
    std::string s{ba.data(),ba.size()};
    std::istringstream iss{s};
    int i{};
    while(iss>>i)
    {
        std::cout<<i<<'\n';
    }
}

@SergioSV · 29.09.2023, 17:27 **[ТС]**

Сообщение от Алексей1153

Сообщение от SergioSV
for (int i=0;i<dataList.size();i++)
{
dataList[i].clear();
}
dataBuffer.clear();
это всё тоже не нужные действия, можно удалить эти строки

Без этого не работает, ошибку выдает

Сейчас сделал так, в порт выдает данные, но когда только запускаю, появляется главный widget и label и начинают идти данные, то появляется сообщение что программа не отвечает, но данные в порт идут!

C++ (Qt)

void MainWindow::readData()
{
 
    QByteArray ba;                                          // Создаем буфер для чтения данных
 
 
    while (serialPort->waitForReadyRead()) {                      // Читаем данные из порта
        ba.append(serialPort->readLine());
 
 
        QList<QByteArray> dataList = ba.split(' ');         // Разделяем данные по символу " "
        QList<int> inList;                                          // пишем результат
 
        for (int i=0;i<dataList.size();i++)                          // перебираем все кусочки строки
        {
 
          inList.append(dataList[i].toInt());                        // переводим их int и записываем в новый массив
 
 
 
          //qDebug() << "Переменная " << i << " : " << dataList[i];       //выводим данные
 
 
          if ( i == 0 and inList[i] == 1)
          {
              ui->label->setText(dataList[0]);
             QPixmap pixmap("/home/project qt/untitled01/start.png");
             ui->label->setPixmap(pixmap);
 
              qDebug() << "вкл! ";
             }
             if ( i == 1 and inList[i] == 1)
              {
                  qDebug() << "Режим ! ";
              }
              if ( i == 2 and inList[i] == 1)
              {
                  qDebug() << "Режим! ";
              }
.........
 
          ba.clear();                                          // Очищаем буфер данных
        }
 
 
        dataList.clear();
        QThread::msleep(50);
    }
 
}

Алексей1153 · 29.09.2023, 17:50

Сообщение от SergioSV

программа не отвечает

пока из порта идут данные, программа будет превращаться в тыкву

Сообщение от SergioSV

while (serialPort->waitForReadyRead())

----------

Сообщение от SergioSV

dataList.clear();

опять лишнее действие (хотя, тут оно ничего не делает всё равно)

----------

код жуткий, на быстродействие положен огромный болт )))

@SergioSV · 30.09.2023, 01:21 **[ТС]**

Сообщение от Алексей1153

код жуткий, на быстродействие положен огромный болт )))

Я понимаю что до идеала далеко и здесь бы не было бы вопросов если бы все хорошо работало!
Можете помочь?

Добавлено через 6 минут

Сообщение от sdf45

строки 67-83: если элементов в списке меньше 16 или он пустой = UB

Уже переписал код , данные есть, но как сказано выше

Сообщение от Алексей1153

пока из порта идут данные, программа будет превращаться в тыкву

и тут конечно ступор немного....

Алексей1153 · 30.09.2023, 08:25

SergioSV, я бы сделал работу с портом в отдельном потоке. Ну а тут я бы перевёл чтение в слот от сигнала [signal] void QIODevice::readyRead() - так хотя бы не заткнётся очередь обработки сообщений

Сообщение от SergioSV

QList<QByteArray> dataList = ba.split(' '); // Разделяем данные по символу " "
QList<int> inList; // пишем результат
for (int i=0;i<dataList.size();i++) // перебираем все кусочки строки
{
inList.append(dataList[i].toInt()); // переводим их int и записываем в новый массив

насчёт этого я привёл обработку выше (пост №5). Кстати, можно даже упростить чуток

C++

#include <iostream>
#include <string>
#include <sstream>
 
int main()
{
    QByteArray ba="1 0 1 0 0 140 +34 -23 1 0";
    
    auto s=ba.toStdString();
    std::istringstream iss{s};
    int i{};
    while(iss>>i)
    {
        std::cout<<i<<'\n';
    }
}

@SergioSV · 30.09.2023, 09:52 **[ТС]**

C++ (Qt)

void MainWindow::readData()
{
 
    QByteArray ba;                                          // Создаем буфер для чтения данных
 
 
    while (serialPort->waitForReadyRead()) {                      // Читаем данные из порта
        ba.append(serialPort->readLine());
 
 
        QByteArray ba;
 
        while (serialPort->waitForReadyRead()) {                      // Читаем данные из порта
        ba.append(serialPort->readLine());
 
            auto s=ba.toStdString();
            std::istringstream iss{s};
            int i{};
            while(iss>>i)
            {
                qDebug() << i;
            }
    }
}
}

Данные в порт идут и нет сообщения что программа не отвечает! Но выдает ошибку если хочу взять данные и вывести в label

Алексей1153 · 30.09.2023, 10:36

SergioSV, если MainWindow::readData() - это слот, который ты сейчас подключил к сигналу, то зачем тут наличие строк № 7, 8 и 13 ?

Сообщение от SergioSV

Но выдает ошибку если хочу взять данные и вывести в label

что за ошибка?
Как именно, когда и какие выводишь данные?

Добавлено через 59 секунд
опять же, зачем readline, если можно readAll

@SergioSV · 30.09.2023, 10:50 **[ТС]**

Данные через порт_сортировка_вывод на монитор

В зависимости от данных нужно поменять картинку, а где-то просто выводить температуру

Сообщение от Алексей1153

опять же, зачем readline, если можно readAll

Если readALL то данные вразброс

@SergioSV · 30.09.2023, 10:57 **[ТС]**

а сейчас так

@SergioSV · 30.09.2023, 11:14 **[ТС]**

Сообщение от Алексей1153

что за ошибка?

Ошибки нет и данные идут, но вылетает сообщение

@insite2012 · 30.09.2023, 21:18

Сообщение от SergioSV

Ошибки нет и данные идут, но вылетает сообщение

Выносите работу с портом в отдельный поток, пусть читает и через сигнал-слот отправляет данные в интерфейс.

@sdf45 944 / 566 / 165 Регистрация: 30.03.2021 Сообщений: 1,899
	28.09.2023, 20:45	2
	строки 67-83: если элементов в списке меньше 16 или он пустой = UB 0

Алексей1153 фрилансер 5221 / 4757 / 1000 Регистрация: 11.10.2019 Сообщений: 12,475
	28.09.2023, 21:15	3
	Сообщение от SergioSV for (int i=0;i<dataList.size();i++) // перебираем все кусочки строки { int var1 = dataList[0].toInt(); // переводим их int и записываем в новый массив int var2 = dataList[1].toInt(); int var3 = dataList[2].toInt(); int var4 = dataList[3].toInt(); int var5 = dataList[4].toInt(); int var6 = dataList[5].toInt(); int var7 = dataList[6].toInt(); int var8 = dataList[7].toInt(); int var9 = dataList[8].toInt(); int var10 = dataList[9].toInt(); int var11 = dataList[10].toInt(); int var12 = dataList[11].toInt(); int var13 = dataList[12].toInt(); int var14 = dataList[13].toInt(); int var15 = dataList[14].toInt(); int var16 = dataList[15].toInt(); int var17 = dataList[16].toInt(); qDebug() << "Переменная " << i << " : " << dataList[i]; //выводим данные } странный цикл. В чём смысл этих строк `int varN = dataList[N-1].toInt();` ? Просто убери их Добавлено через 1 минуту Сообщение от SergioSV for (int i=0;i<dataList.size();i++) { dataList[i].clear(); } dataBuffer.clear(); это всё тоже не нужные действия, можно удалить эти строки 0

Алексей1153 фрилансер 5221 / 4757 / 1000 Регистрация: 11.10.2019 Сообщений: 12,475
	29.09.2023, 17:50	7
	Сообщение от SergioSV программа не отвечает пока из порта идут данные, программа будет превращаться в тыкву Сообщение от SergioSV while (serialPort->waitForReadyRead()) ---------- Сообщение от SergioSV dataList.clear(); опять лишнее действие (хотя, тут оно ничего не делает всё равно) ---------- код жуткий, на быстродействие положен огромный болт ))) 0

@SergioSV 0 / 0 / 0 Регистрация: 07.06.2023 Сообщений: 8
	30.09.2023, 01:21 [ТС]	8
	Сообщение от Алексей1153 код жуткий, на быстродействие положен огромный болт ))) Я понимаю что до идеала далеко и здесь бы не было бы вопросов если бы все хорошо работало! Можете помочь? Добавлено через 6 минут Сообщение от sdf45 строки 67-83: если элементов в списке меньше 16 или он пустой = UB Уже переписал код , данные есть, но как сказано выше Сообщение от Алексей1153 пока из порта идут данные, программа будет превращаться в тыкву и тут конечно ступор немного.... 0

Алексей1153 фрилансер 5221 / 4757 / 1000 Регистрация: 11.10.2019 Сообщений: 12,475
	30.09.2023, 10:36	11
	SergioSV, если `MainWindow::readData()` - это слот, который ты сейчас подключил к сигналу, то зачем тут наличие строк № 7, 8 и 13 ? Сообщение от SergioSV Но выдает ошибку если хочу взять данные и вывести в label что за ошибка? Как именно, когда и какие выводишь данные? Добавлено через 59 секунд опять же, зачем readline, если можно readAll 0

@SergioSV 0 / 0 / 0 Регистрация: 07.06.2023 Сообщений: 8
	30.09.2023, 10:50 [ТС]	12
	В зависимости от данных нужно поменять картинку, а где-то просто выводить температуру Сообщение от Алексей1153 опять же, зачем readline, если можно readAll Если readALL то данные вразброс 0

@SergioSV 0 / 0 / 0 Регистрация: 07.06.2023 Сообщений: 8
	30.09.2023, 10:57 [ТС]	13
	а сейчас так 0

@SergioSV 0 / 0 / 0 Регистрация: 07.06.2023 Сообщений: 8
	30.09.2023, 11:14 [ТС]	14
	Сообщение от Алексей1153 что за ошибка? Ошибки нет и данные идут, но вылетает сообщение 0

@insite2012 5501 / 4269 / 1213 Регистрация: 12.10.2013 Сообщений: 12,280 Записей в блоге: 2
	30.09.2023, 21:18	15
	Сообщение от SergioSV Ошибки нет и данные идут, но вылетает сообщение Выносите работу с портом в отдельный поток, пусть читает и через сигнал-слот отправляет данные в интерфейс. 0