A UDP socket send/receive.

[jenya7]

@jsulm said in A UDP socket send/receive.:

@jenya7 Please read documentation: there is no signal ReadyRead in QUdpSocket, but https://doc.qt.io/qt-5/qiodevice.html#readyRead
And static variables should be avoided as much as possible. Usually they are a sign for bad design.

OK. So this way - connect(socket, SIGNAL(readyRead()), this, SLOT(ReadyRead())); on readyRead my ReadyRead is invoked?

[jsulm]

@jenya7 Yes.
But as @JonB already suggested you should switch to new Qt5 syntax to detect such issues already at compile time.
See https://wiki.qt.io/New_Signal_Slot_Syntax/de

[J.Hilk]

@jenya7

what you wrote:

connect(socket, SIGNAL(ReadyRead()), this, SLOT(ReadyRead()));

what it should be:
Qt5 and Qt6

connect(socket, & QUdpSocket::readyRead, this,  &MyUDP::ReadyRead);

Qt4 and earlier:

bool connectWasSuccessfull = connect(socket, SIGNAL(readyRead()), this, SLOT(ReadyRead()));
Q_ASSERT(connectWasSuccessfull);

you should read up on signal and slots and on static. Because static does not "limit anything to only 1 file"

[jenya7]

@J-Hilk
I have Qt 4.8. Indeed this SIGNAL(readyRead() is quite unclear. This way QUdpSocket::readyRead it's clear and I can see all events on smart autocomplete.
One more thing - how can I display a received message to a user? Say in textEdit. qDebug() is not usefull in a real application. How can I pass the data outside the class? Outside the void MyUDP::ReadyRead().

On signal event I get to MyUDP::ReadyRead() but who is signaling to UI to display message?

[jsulm]

@jenya7 said in A UDP socket send/receive.:

How can I pass the data outside the class?

Using signals and slots. In your MyUDP class add a signal with a QString parameter. Then emit this signal when you want to show a message with the text as signal parameter. Connect this signal with a slot in your GUI class which is responsible for showing the message. In this slot you can show the text then.

[jenya7]

@jsulm said in A UDP socket send/receive.:

@jenya7 said in A UDP socket send/receive.:

How can I pass the data outside the class?

Using signals and slots. In your MyUDP class add a signal with a QString parameter. Then emit this signal when you want to show a message with the text as signal parameter. Connect this signal with a slot in your GUI class which is responsible for showing the message. In this slot you can show the text then.

Sorry I fail to understand. Right now the receiving is working and I got the message in a global string
msg = QString(udp_buffer);
Now what signal do I emit?

[jsulm]

@jenya7 said in A UDP socket send/receive.:

Now what signal do I emit?

One you need to define in your MyUDP class as I wrote before:

class MyUDP...
{
signals:
    void showMessage(const QString& msg);
...
void MyUDP::ReadyRead()
{
   ...
   showMessage(QString(buffer));

Please read https://doc.qt.io/qt-5/signalsandslots.html

[jenya7]

@jsulm
Wow. That's cool. And I can implement showMessage(const QString& msg); in any file?
Say ui->textEditNetRX->append(msg); is available in "mainwindow.cpp.

[jsulm]

@jenya7 said in A UDP socket send/receive.:

in any file?

Yes, just add a slot where you need it and connect it to the signal.
Signals/slots is a very powerful concept and it also helps to decouple classes from each other. For example your MyUDP class does not need to know who connects to its signal, it just emits the signal. You can connect 0..n slots to a signal, you can connect a signal to 0..n other signals.

[jenya7]

@jsulm
Thank you very much.

[jenya7]

In mainwindow.h I created a slot

class MainWindow : public QMainWindow
{
    Q_OBJECT

public:
    explicit MainWindow(QWidget *parent = nullptr);
    ~MainWindow();

    MyUDP udp;

public slots:
    void ShowUdpMessage(const QString& msg, const QByteArray data);
//and so on...

in mainwindow.cpp

void MainWindow::ShowUdpMessage(const QString& msg, const QByteArray data)
{
    QByteArray arr = data;
    ui->textEditNetRX->append(msg);
}

now in main.cpp

MainWindow w;
 w.show();

QObject::connect(&???, &???, &w, &MainWindow::ShowUdpMessage);

How do I tie the signal source?
The signal generates by MyUDP udp that resides in the MainWindow class.

[jsulm]

@jenya7

QObject::connect(&udp, &MyUDP::showMessage, &w, &MainWindow::ShowUdpMessage);

[jenya7]

@jsulm
QObject::connect(&udp, &MyUDP::showMessage, &w, &MainWindow::ShowUdpMessage);
&udp - is not visible in the scope. also MainWindow::udp <- udp not recognized although it declared public:MyUDP udp; in MainWindow.

[jsulm]

@jenya7 You should do the connection inside MainWindow (for example in its constructor), because MainWindow needs the signal and it also has the MyUDP instance...

"although it declared public:MyUDP udp" - don't declare it public, just do the connection inside MainWindow.

[jenya7]

I try in the constructor

MainWindow::MainWindow(QWidget *parent) : QMainWindow(parent), ui(new Ui::MainWindow)
{
    ui->setupUi(this);
    ui->tabWidget->setCurrentIndex(0);
    ui->textEditTerminalTx->installEventFilter(this);
    
     QObject::connect(&udp, &MyUDP::ShowUdpMessage, &MainWindow, &MainWindow::ShowUdpMessage);
}

third argument - &MainWindow <- 'MainWindow' does not refer to a value.

[JonB]

@jenya7
&MainWindow should be this. And you ought understand why (it's an instance of MainWindow).

[jenya7]

OMG! It's working! Thank you so much!