QUdpSocket does not trigger ReadyRead signal

[onurcevik]

I am reading and sending QImage with QUdpSocket in a for loop but I can't get the QImages because readyRead() signal doesn't get emmited. Here is my code:

udpsocket.h

#ifndef UDPSOCKET_H
#define UDPSOCKET_H

#include <QObject>
#include <QUdpSocket>

#include "camera.h"

class UDPSocket : public QObject
{
    Q_OBJECT
public:
    explicit UDPSocket(QObject *parent = nullptr);
    void sendFrame(QImage frame);

signals:


public slots:
    void readyRead();

private:

    QUdpSocket *socket;


};

#endif // UDPSOCKET_H

udpsocket.cpp

 #include "udpsocket.h"
#include <QDebug>
#include <iostream>
#include <fstream>
#include <string>
#include <sstream>
#include <unistd.h>
#include <udpsocket.h>
#include <QBuffer>
#include <QDebug>
#include <QImage>
#include <QImageWriter>

#define XRES 640
#define YRES 480
#define SIZE 640*480*3



UDPSocket::UDPSocket(QObject *parent) : QObject(parent)
{

    socket = new QUdpSocket(this);
    bool deneme= socket->bind(QHostAddress::LocalHost,1234);
    if(deneme)
        qDebug()<<"binded";
    connect(socket,SIGNAL(readyRead()),this,SLOT(readyRead()));

}


void UDPSocket::sendFrame(QImage image)
{
    qDebug()<<"will send";

    QBuffer buffer;
    QImageWriter writer(&buffer,"bmp");
    writer.write(image);
    QByteArray data;
    data.append(buffer.data());
    socket->writeDatagram(data,QHostAddress::Broadcast,1234);


    qDebug()<<"sent";


}

void UDPSocket::readyRead()
{
    qDebug()<<"readyread";
    QByteArray buffer;
    buffer.resize(socket->pendingDatagramSize());

    QHostAddress sender;
    quint16 senderPort;

    socket->readDatagram(buffer.data(),buffer.size(),&sender,&senderPort);


    std::ostringstream stream;
    stream << "frames/" << qrand() << ".bmp";
    std::string new_string = stream.str();
    std::ofstream image;
    image.open(new_string);
    image << "P6\n" << XRES << " " << YRES << " 255\n";

    image.write(buffer.data(), SIZE);
    image.close();
    qDebug()<<"image arrived";
}

main.cpp

#include <QCoreApplication>
#include <camera.h>
#include <QThread>
#include <QDebug>
#include <iostream>
#include <fstream>
#include <string>
#include <sstream>
#include <unistd.h>
#include <udpsocket.h>

#include <QPixmap>


#define XRES 640
#define YRES 480

int main(int argc, char *argv[])
{
    QCoreApplication a(argc, argv);
    UDPSocket udpObject;

    QImage *image;

     Camera camera("/dev/video0", XRES, YRES);

        for(int i=0; i<10; i++)
        {
           auto frame = camera.frame();
           QThread::sleep(0.0416);
           image = new QImage(frame.data,XRES,YRES,QImage::Format_RGB888);
           udpObject.sendFrame(*image);



}


    return a.exec();
}

When I run this code I see

binded
will send
sent

10 times (because of for loop in main.cpp) and when I debug readyread() doesn't even run.

How can I fix this code so I can recieve the images I sent ? Should I use connectToHost instead of bind() ?

Note:camera.h is my class that I use to read images from camera and probably has nothing to do with error.

[SGaist]

Hi,

Are you trying to send and receive the image using the exact same socket ?

[onurcevik]

@SGaist In main.cpp I create the UDPSocket object outside the for loop then use the same object to access the sendFrame(QImage frame); function to send the sockets inside the loop . So I guess I use the same socket.

[SGaist]

Then show what you are doing.

From what you wrote, you don't even have an event loop running hence not signals/slots working.

Also, why are you trying to use UPD to send data from your application back to the same application ?

[onurcevik]

@SGaist I already posted almost all of my code? What do you mean "show what you are doing?"
I am sending/recieving on same PC to test if I can so later I can implement it for different PC's. Since Im newbie I dont know much about event loops can you show some example?

[jsulm]

@onurcevik Can you show your main.cpp?

[onurcevik]

@jsulm I edited and added main.cpp to my question.

[jsulm]

@onurcevik A loop with sleep() inside (before starting event loop) and you expect this to work? It will not. See what @SGaist wrote.

[onurcevik]

@jsulm @SGaist Sorry for my noob mistakes. I searched event loop in QT but I couldn't find how to implement it. Can you help me with that ? Should I use Threads ?

[jsulm]

@onurcevik

return a.exec();

this starts the event loop. No need for threads. Use asynchronous nature of QUdpSocket instead. You should take a look at one of the example applications for networking.

[onurcevik]

@jsulm I deleted sleep and for loop it still did not work. I did take a look couple of examples even tried to run it. A pinpoint to what wrongs with my code would be really helpful. Since removing sleep and for did not work either.

Edit: I will switch to QWidget application and try to send through a button click.

[aha_1980]

Hi @onurcevik,

• On which platform are you?
• Which Qt version do you use?
• Do you have multiple network adapters in your computer?

Regards

[onurcevik]

@aha_1980

• I am on Kubuntu
• I use Qt Creator 4.6.2 Based on Qt 5.11.1 (GCC 8.1.0, 64 bit)
• I only have 1 modem.

[aha_1980]

@onurcevik

Can you please post the output of ifconfig -a?

It is likely that your packet is not even send out. You can check that with Wireshark.

Edit:

I use Qt Creator 4.6.2 Based on Qt 5.11.1 (GCC 8.1.0, 64 bit)

That is the Creator version, not the Qt version I asked for.

[onurcevik]

@aha_1980

That is the Creator version, not the Qt version I asked for.

I use Qt Creator 4.6.2 Based on Qt 5.11.1 (GCC 8.1.0, 64 bit)

QT version is next to the QT Creator Version which is 5.11.1

the output of ifconfig -a :

enp27s0: flags=4163<UP,BROADCAST,RUNNING,MULTICAST>  mtu 1500
        inet 192.168.2.44  netmask 255.255.255.0  broadcast 192.168.2.255
        inet6 fe80::a98d:26da:2afd:3299  prefixlen 64  scopeid 0x20<link>
        ether 00:d8:61:15:7d:b8  txqueuelen 1000  (Ethernet)
        RX packets 37659  bytes 41459688 (41.4 MB)
        RX errors 0  dropped 0  overruns 0  frame 0
        TX packets 27301  bytes 2684010 (2.6 MB)
        TX errors 0  dropped 0 overruns 0  carrier 0  collisions 0

lo: flags=73<UP,LOOPBACK,RUNNING>  mtu 65536
        inet 127.0.0.1  netmask 255.0.0.0
        inet6 ::1  prefixlen 128  scopeid 0x10<host>
        loop  txqueuelen 1000  (Local Loopback)
        RX packets 1963  bytes 180145 (180.1 KB)
        RX errors 0  dropped 0  overruns 0  frame 0
        TX packets 1963  bytes 180145 (180.1 KB)
        TX errors 0  dropped 0 overruns 0  carrier 0  collisions 0

@aha_1980 said in QUdpSocket does not trigger ReadyRead signal:

[aha_1980]

@onurcevik said in QUdpSocket does not trigger ReadyRead signal:

QT version is next to the QT Creator Version which is 5.11.1

That is the Qt version Creator is built with, which is not necessarily the version you are using to compile your program.

From you ifconfig I see, that there is only one physical interface., so bind should select the correct one.

Please check with wireshark that the packet is sent out.

[SGaist]

Just for the sake of simplification, did you take a look at the examples linked in the QUdpSocket documentation ?

Thinking about:

Broadcast Sender
Broadcast Receiver
Multicast Sender
Multicast Receiver

[onurcevik]

@aha_1980 I now realized that I can't even send the socket due to large size which is 1 mb. Do you know how can I divide it into chunks of bytes ? Or should I create new topic?

[SGaist]

What exactly is the goal of your application ?

[onurcevik]

@SGaist I want to take pictures from camera then send and recieve them continuously in order to implement a real time video streaming application. The reason I am sending images instead of video is because I will later work on image processing. I realized that image size is too big for this which is 6404803 BYTES. But I also can't switch to TCP because I need the speed of UDP.