readyread() signal doesn't emit after sendmessage() signal on pushbutton

mrjj

Hi
Did you put a break point in each sendmessage and gotNewMesssage functions to make sure you did not
create a ping-pong situation where signals are kept firing?

fari35

@mrjj as soon as the signal is emitted it executes a slot and then returns and continue its execution. There is some problem in threading. Like I also checked after putting a sleep statement in readyread() signal in fthread.cpp file and it worked fine for like some time and after that it crashed and then didin't work

mrjj

@fari35

well put breakpoints on the slots and check they are called just once when you press the send button.

fari35

@mrjj Yes, I have checked right nnow the slots are only called once

JonB

@fari35
I have told you before to put qDebug() statements in. You are supposed to do more investigation yourself and report the exact problem rather than just keep throwing your whole code at the community for answer....

fari35

@JonB I didn't attch the whole code initially but after they asked for it , I've attached my whole code, but now I have cut it short. the problem I'm getting is that readyread() siggnal is not emitting after send button is pressed. Before pressing the send button it emits even if the client sends 10 messages it emits 10 times but as soon as the send button is pressed it stops emitting

Christian Ehrlicher

The code still does not show where the QTcpSocket is created - and when it's like I guess you implemented it, it's still created in the wrong thread. Please provide a minimal, compilable example!

fari35

@Christian-Ehrlicher here is the code for run() where QTcpSocket is created:

void fthread::run()
{


    tcpSocket = new QTcpSocket();

    if (!tcpSocket->setSocketDescriptor(socketDescriptor)) {
        emit error(tcpSocket->error());
        return;
    }

    connect(tcpSocket, SIGNAL(readyRead()), this, SLOT(readyRead()));

    connect(tcpSocket, SIGNAL(disconnected()), this, SLOT(disconnected()));

    exec();

}

Christian Ehrlicher

@fari35 said in readyread() signal doesn't emit after sendmessage() signal on pushbutton:

connect( this, SIGNAL(sendmessage(QString)),thread_obj, SLOT(sendmessage(QString)));

As I said - the function sendmessage(QString) is executed in the main thread. Please read the docs about QThread or don't use QThread at all when you don't know the basics. There is no reason to use threads at all...

fari35

@Christian-Ehrlicher Actually I need to make a multi threaded server for my project and I'm using Qt for that. I want to send the message to all the clients who have been assigned a separate thread, Can you please tell me how can I do that because I'm not getting it.

JonB

@fari35
What is it you need beyond the Threaded Fortune Server Example ? How much are you based on that?

fari35

@JonB In this example it's not shown how to send the message to the client if the signal is emitted from the main thread like here I'm making a UI for the server so I want to send the message to the client as soon as the send button is clicked, but I don't know how to do it. I'm not even getting help from anywhere.

Christian Ehrlicher

Again: if you want to use thread (for whatever reason, maybe just to show that you're a famous programmer...), make sure you understand what you're doing and read the documentation and examples:

"It is important to remember that a QThread instance lives in the old thread that instantiated it, not in the new thread that calls run(). This means that all of QThread's queued slots and invoked methods will execute in the old thread. Thus, a developer who wishes to invoke slots in the new thread must use the worker-object approach; new slots should not be implemented directly into a subclassed QThread."

JKSH

@fari35: To clarify the documentation quoted by @Christian-Ehrlicher's: Do not implement any slots in fthread. Instead, implement your slot in a separate QObject. Create that object together with tcpSocket.

fari35

@JKSH @Christian-Ehrlicher If I implement the slot in a separate Qobject then how can I send the data from that class as my TcpSocket is created in fthread class?
My issue is same as this one :
https://stackoverflow.com/questions/35927165/qt-multi-threaded-multi-client-tcp-server-socket-write-issue/65750692#65750692

JKSH

@fari35 said in readyread() signal doesn't emit after sendmessage() signal on pushbutton:

If I implement the slot in a separate Qobject then how can I send the data from that class as my TcpSocket is created in fthread class?

Let's take a step back.

Imagine that your server only needs to communicate with one client. Imagine that you have no threads; your QTcpSocket is in the main thread. How can you send data from a QObject to the socket?

fari35

@JKSH I'll simply create a QTcpSocket and then after initializing it I'll send the data to the client

Christian Ehrlicher

@fari35 said in readyread() signal doesn't emit after sendmessage() signal on pushbutton:

If I implement the slot in a separate Qobject then how can I send the data from that class as my TcpSocket is created in fthread class?

Did you actually ook at least once in the documentation. Hint: there is an example which describes how to properly create an object living in another thread and how to connect signals from/to them.

Anonymous_Banned275

@Christian-Ehrlicher said in readyread() signal doesn't emit after sendmessage() signal on pushbutton:

Did you actually ook at least once in the documentation.

Perhaps if you throttle DOWN your nagging attitude a little ?
It seldom helps resolving TECHNICAL issue.
There seems to be few people perpetuating similar format recently , I am not particularly picking you.
Just saying.

Christian Ehrlicher

@AnneRanch said in readyread() signal doesn't emit after sendmessage() signal on pushbutton:

Perhaps if you throttle DOWN your nagging attitude a little ?

I simply don't understand how someone can ignore our hints the whole time and asking the same stuff over and over again. I already said in my first post here what's wrong and that threading is not that easy. So if it's not easy and I (ask someone who wants to learn) would read the documentation and study the examples to see how all is going.