Прием данных

[Eugene1911]

Здравствуйте
У меня возникла следующая проблема при приеме данных: сначала приходил не весь пакет байтов (необходимо 5, а приходило по 2). Я соединил сигнал readyRead со слотом wait_serialReceive, а в нем вызвал singleShot:

Код:

void MainWindow::wait_serialReceive()//ожидание приема полного пакета данных
{
    QTimer::singleShot(5, this, SLOT(serialReceive()));
}

В слоте serialReceive() читаю данные и вывожу их на текстовую метку и через qDebug:

Код:

void MainWindow::serialReceive()//прием данных
{
    QByteArray temp_data;
    int a;

    temp_data = serial->readAll();

    a = temp_data[0];
    qDebug()<<"first byte"<<a;
    qDebug()<<"all array"<<temp_data.toHex();
    ui->labelRead->setText(temp_data.toHex());
}

При запуске программы текстовая метка остается пустой, а qDebug пишет:
first byte 7
all array "07a46c450a"
first byte 0
all array ""

Т.е. программа читает данные, а потом очищает массив? Или как?

[Eugene1911]

Проблема решилась так:
temp_data объявил в хедере
В слот wait_serialReceive добавил строчку

Код:

temp_data.clear();

Теперь все отображается. Есть еще один вопрос: сообщение которое выводится в метке, неправильное, должно быть так:
"a46c450a07"
а выводится так:
"07a46c450a".
Может, все-таки я накосячил при приеме?

[Bepec]

Всё у тебя правильно. Ты массив не правишь. Ты просто копируешь его первый элемент. Чтобы его убрать, надо сделать remove

[Eugene1911]

Ясно, спасибо

[8Observer8]

Почему у вас 5 написано?

Код

C++ (Qt)
QTimer::singleShot(5, this, SLOT(serialReceive()));

Это чтобы за 5 милисекунд успеть забрать данные? Или я что-то не понял?

[Eugene1911]

Мне нужно установить время, за которое придет весь пакет данных. Я пробовал ставить разное время, в итоге остановился на 5 мс, т.к. за это время все байты успевают придти. А большее время решил не устанавливать))

[Old]

Мне нужно установить время, за которое придет весь пакет данных. Я пробовал ставить разное время, в итоге остановился на 5 мс, т.к. за это время все байты успевают придти. А большее время решил не устанавливать))

А вы уверены, что и на других компьютерах данные будут успевать приходить за 5 мс?
Думаю лучше сразу делать нормально и универсально.

[Eugene1911]

Да, вы правы,я об этом не подумал)) А как тогда лучше сделать? Просто установить значение таймера побольше или сделать как-то по другому? Всего нужно принять 5 байтов

[Old]

Да, вы правы,я об этом не подумал)) А как тогда лучше сделать? Просто установить значение таймера побольше или сделать как-то по другому? Всего нужно принять 5 байтов

Так вы и ждите в слоте wait_serialReceive() что бы были доступны нужные 5 байт, а потом вычитывайте их и обрабатывайте.

Код

C++ (Qt)
void MainWindow::wait_serialReceive()
{
    if( serial->bytesAvailable() < 5 )
        return;
 
    QByteArray data = serial->read( 5 );
 
    // Обрабатываем....
}
 

Ну и 5 лучше константой заменить.

[OKTA]

Old - скоропечтная машинка! Не даешь слова сказать 

[Eugene1911]

Спасибо большое за совет, переделаю