[z.....y]

Robię program w QT odbierający pakiety wysyłane po UDP, które ma pokazywać na ekranie (konkretnie w QLabel). Problem w tym, że program mi się wywala po odebraniu pierwszego pakietu. (Program nieoczekiwanie zakończył pracę). Po odpaleniu debuggera wskazało mi jedną z linii, odpowiadającą za wyświetlenie tekstu w oknie. Pytanie: co robię źle?

Kod

#programowanie #qt #kiciochpyta

[lechium06]

@zacietrzewiony: Nie powinno być

QString::number(bbb)

?

[z.....y]

@lechium06: Błąd w kodzie, który tutaj wstawiłem. Oryginalnie u siebie mam

QString::number(bbb);

i wywala.

[lechium06]

@zacietrzewiony: ok. To co to znaczy "wywala"? Leci jakiś wyjątek?

[z.....y]

@lechium06: Konkretnie robi się tak: http://i.imgur.com/VknvI6f.png

A debugger pokazuje to: http://i.imgur.com/1Nepf1k.png

[lechium06]

@zacietrzewiony: Problem jest prawdopodobnie gdzieś wcześniej w kodzie. Krzyż na drogę i powodzenia. Ostatnie co mi przychodzi do głowy - readDatagram coś zwraca? Może ilość bajtów, które rzeczywiście się odebrały? Może ona nie jest równa pendingDatagramSize() ? Może przy okazji któraś z tych wartości nie jest równa 3?

[z.....y]

@lechium06:

readDatagram

nic nie zwraca. Reszta wygląda tak.

[pp555]

@zacietrzewiony: skąd wziąłeś informację, że readDatagram nic nie zwraca? http://qt-project.org/doc/qt-5/qudpsocket.html#readDatagram - "Returns the size of the datagram on success; otherwise returns -1."

[z.....y]

@pp555: Nic mi się nie pokazało po prawej stronie podczas debuggowania. Mea culpa.

[Vetinari]

@zacietrzewiony: W Qt zmieniać GUI można tylko z głównej pętli zdarzeń aplikacji (a dokładniej z tej, w której okno zostało utworzone), spełniasz ten warunek?

[Vetinari]

@zacietrzewiony: Dodatkowo nie sprawdzasz czy pendingDatagramSize>0. Potem nie sprawdzasz czy datagram.size()>=2.

[WolverinePL]

@zacietrzewiony: ui->label->setText

Tu dwa pointery są po drodze, zobacz czy są poprawne.

@Vetinari: Chyba w głównym wątku :)

[z.....y]

@WolverinePL: Są poprawne, podpowiada mi je.

@Vetinari: Najlepsze jest to, że jak rysowałem tekst w QPaintEvent to mi się pokazywał normalnie i nie wywalało błędu. A tu coś takiego...

[Vetinari]

@WolverinePL: No, w głównym wątku ale sygnały itp są wykonywane w QEventLoop, tam jest trochę pomieszane. ui-> wskazuje na okno robione w QtCreator, raczej powinien być poprawny wskaźnik.

[Vetinari]

@zacietrzewiony: Możesz pokazać więcej kodu? Bo tak trudno ocenić. Ewentualnie może nie być miejsca na stosie na resize(), nie wiem jak to działa w QByteArray, mogę jutro w pracy sprawdzić, bo tam mam całe SDK na linuksie zbudowane z debug.

[z.....y]

@Vetinari: Proszę, to ostatnia wersja, która u mnie działała: https://gist.github.com/anonymous/5ac88c163e7ee0528c0a

[Vetinari]

@zacietrzewiony: Wydaje mi się, że to:

connect(odbieranie, SIGNAL(readyRead()),this, SLOT(odbieranyciagznakow()));

może powodować, że setText() zostanie wywołany w innym wątku niż działa pętla zdarzeń okna, spróbuj w connect użyć: Qt::QueuedConnection.

[z.....y]

@Vetinari: Masz chyba rację, jednak jak mam teraz:

connect(odbieranie, SIGNAL(readyRead()),this, SLOT(odbieranyciagznakow()), Qt::QueuedConnection);

to nadal nie działa.

[pp555]

@Vetinari : Qt::QueuedConnection daje chyba tylko to, że jak obiekt jest przeniesiony do innego wątku to slot wywoła się w tym wątku, przy Qt::DirectConnection slot wywołałby się w wątku z którego jest emitowany sygnał.

@zacietrzewiony: tworzyłeś jakieś wątki w tym projekcie (instancje klasy QThread lub pochodnych)?

[z.....y]

@pp555: Nie tworzyłem.

[Vetinari]

@zacietrzewiony:

datagram.resize(odbieranie->pendingDatagramSize()+1);

    odbieranie->readDatagram(datagram.data(),