Qextserialport und QTextEdit

[Stefan R (Gast)]

Original funktion:

void Dialog::onReadyRead()
{
    if (port->bytesAvailable()) {
        ui->recvEdit->moveCursor(QTextCursor::End);
        ui->recvEdit->insertPlainText(QString::fromLatin1(port->readAll()));
    }
}

Fehlerfreie Ausgabe in QTextEdit (Projektspezifische Variable mit 
Wert..):

azimut_145
azimut_145
azimut_145
<usw ..>

Geänderte funktion:

void Dialog::onReadyRead()
{
    if (port->bytesAvailable()) {
        ui->recvEdit->moveCursor(QTextCursor::End);
        ui->recvEdit->insertPlainText(QByteArray("some text:"));
        ui->recvEdit->insertPlainText(QString::fromLatin1(port->readAll()));
    }
}

Fehlerbehaftete Ausgabe in QTextEdit:

some text:azimut_145
some text:azimut_145
some text:azimut_145
some text:azimut_145
some text:azimut_145some text:
some text:azimut_some text:145
some text:azimusome text:t_145
some text:azsome text:imut_145
some text:azimut_145
some text:azimut_145
<usw ..>

Die Ausgabe erfolgt also verschachtelt und zufällig! Meine Vermutung: 
Irgendwelche Threads werden parallel abgearbeitet.

Lösungsvorschläge?

Programminfos:
Qextserialport 1.2 beta
qt 5.0.1 unter qt creator 2.6.2

Gruß Stefan

[Jörg B. (jbernau)]

Stefan R schrieb:
> ui->recvEdit->insertPlainText(QByteArray("some text:"));

ui->recvEdit->insertPlainText("some text:");

Weshalb nicht gleich QString verwenden? Sonst wird aus QBytrAtrray 
später QString ...


Grüßle

[Rolf M. (rmagnus)]

Hast du mal versucht, zwischen den beiden Ausgaben den Cursor nochmal 
explizit ans Ende zu setzen?

[Stefan R. (lenox)]

Hallo Jörg, hallo Rolf,

beides habe ich expiziet probiert, ohne Erfolg. Gleiches Fehlerbild.

Gruß Stefan

[hans (Gast)]

versuch mal

void Dialog::onReadyRead()
{
    if (port->bytesAvailable())
    {
        QString buffer;
        buffer.append("some Text:");
        buffer.append(QString::fromLatin1(port->readAll());
        ui->recvEdit->append(buffer);
    }
}

insertPlainText

[Rolf M. (rmagnus)]

hans schrieb:
>         QString buffer;
>         buffer.append("some Text:");
>         buffer.append(QString::fromLatin1(port->readAll());
>         ui->recvEdit->append(buffer);

oder auch gleich:

        ui->recvEdit->append("some Text:" + QString::fromLatin1(port->readAll());

[M. K. (avr-frickler)]

Stefan R schrieb:
> Die Ausgabe erfolgt also verschachtelt und zufällig! Meine Vermutung:
> Irgendwelche Threads werden parallel abgearbeitet.

Nö das Programm macht schon genau das was es soll, nur ist es nicht das 
was du möchtest!

> Lösungsvorschläge?
1. Mach dir mal Gedanken was die beiden Methoden bytesAvailable() und 
readAll() machen könnten.

2. Informiere dich mal über Zeilenabchlusszeichen.

[Rolf M. (rmagnus)]

M. K. schrieb:
> Stefan R schrieb:
>> Die Ausgabe erfolgt also verschachtelt und zufällig! Meine Vermutung:
>> Irgendwelche Threads werden parallel abgearbeitet.
>
> Nö das Programm macht schon genau das was es soll, nur ist es nicht das
> was du möchtest!

Also mir würde da jetzt kein Grund einfallen, warum sich das Programm so 
verhalten sollte.

[M. K. (avr-frickler)]

Das Problem was Stefan R hat ist, dass die PC-Software und der Sender 
nicht synchronisiert sind.

Er fragt einfach nach ob Daten im Puffer sind:
 ja   : "some text:" ausgeben und eingelesenes anhängen
 nein : na dann versuche ich es so schnell wie möglich noch einmal

Jetzt passiert folgendes:
Die ersten male passt die Zeitspanne von bytesAvailable() bis readAll() 
und die Daten werden sauber ausgegeben.

Beim nächsten mal war der PC aber mit readAll() schneller fertig als der 
Sender den Puffer füllen konnte und schon haben wir den Salat.

Dank Präemptiven Multitasking [1] kann man kaum vorhersagen wie viel 
Zeit zwischen den Aufrufen von bytesAvailable() und readAll() 
verstreichen.

Außerdem ist überhaupt nicht sicher zu welchem Zeitpunkt die Methode 
bytesAvailable() aufgerufen wird, dass kann schon nach dem ersten Byte 
sein oder auch wenn der Sender bereits alles losgeworden ist. Die 
Methode bytesAvailable() sagt zwar wie viele Bytes sich im Puffer 
befinden, dies wird hier aber überhaupt nicht ausgewertet.

[1] 
http://de.wikipedia.org/wiki/Multitasking#Pr.C3.A4emptives_Multitasking

[Rolf M. (rmagnus)]

Ja, du hast natürlich recht. Daß Daten da sind bedeutet nicht 
automatisch, daß es sich dabei um genau eine komplette Zeile handelt.
Wenn er das will, sollte er sich evtl. mal die Funktionen canReadLine() 
und readLine() statt bytesAvailable() und readAll() anschauen.

M. K. schrieb:
> Dank Präemptiven Multitasking [1] kann man kaum vorhersagen wie viel
> Zeit zwischen den Aufrufen von bytesAvailable() und readAll()
> verstreichen.

Das ist aber auch relativ egal. Wenn Bytes verfügbar sind, sind sie das 
auch später noch, sofern sie zwischendurch niemand anders ausliest.

M. K. schrieb:
> Er fragt einfach nach ob Daten im Puffer sind:
>  ja   : "some text:" ausgeben und eingelesenes anhängen
>  nein : na dann versuche ich es so schnell wie möglich noch einmal

Ich vermute, daß "onReadyRead()" ein Slot ist, der automatisch 
aufgerufen wird, wenn sich was an der Schnittstelle getan hat. Damit 
wird es nicht so schnell wie möglich versucht, sondern erst wenn das 
nächste Mal Daten zur Abholung bereit stehen.

[Michael J. (jogibaer)]

Hallo,

ich benutze auch Qextserialport.

Allerdings mache ich es per Signal/Slot.

Daten hole so ab:
QByteArray data = Port->readAll();

Dabei sind die Daten in einem Protokoll eingebettet.


Jogibär

[Stefan R (Gast)]

Vielen Dank für eure Hilfe, dass Problem ist nun gelöst.

Hier die Zusammenfassung der Änderungen zum original Code:

port = new QextSerialPort(ui->portBox->currentText(), settings, QextSerialPort::EventDriven);

connect(port, SIGNAL(readyRead()), SLOT(onReadyRead()));

void Dialog::onReadyRead()
{
    if (port->canReadLine())
    {
        ui->recvEdit->appendPlainText("uC> " + QString::fromLatin1(port->readLine()));
    }
]