How to exit void QIODevice::readyRead() function ? Is this function also interrupt?

Mucahit

Because after the readyread function works, it doesn't shut down unless I move the mouse.Readyread function gets stuck. I want to leave this function when the last data import is finished. The atEnd() function should work for this but it didn't work.

jsulm

@Mucahit Can you show your code? Especially slot connected to readyRead().

Mucahit

Of course,

QByteArray data;
QByteArray totaldata;

void QIODevice::readyRead()
{
data=serial.readAll();
totaldata=totaldata+data;
}

void MainWindow::paintEvent(QPaintEvent *)
{
qDebug()<<totaldata; // I want to print the data I receive here because I will draw pies based on the data I received
}

jsulm

@Mucahit said in How to exit void QIODevice::readyRead() function ? Is this function also interrupt?:

I want to print the data I receive here because I will draw pies based on the data I received

And what is the problem?

Mucahit

Does not print this data without moving the mouse. So I can't get out of this readyread function and switch to paintevent.

J.Hilk

@Mucahit said in How to exit void QIODevice::readyRead() function ? Is this function also interrupt?:

paintEvent is not being called, just because totalData changed ...

try calling update() it will schedule a paint event call at the next possible time.

void QIODevice::readyRead()
{
data=serial.readAll();
totaldata=totaldata+data;
update();
}

Mucahit

Yes that's exactly what i want but the update function didn't work.

I had to use the update function like this:

void QIODevice::readyRead()
{
data=serial.readAll();
totaldata=totaldata+data;
QMainWindow main;
main.update();
}

JonB

@Mucahit said in How to exit void QIODevice::readyRead() function ? Is this function also interrupt?:

QMainWindow main;
main.update();

You really can't do this. That is a local variable, it has nothing to do with your main window. This is basic C++ stuff.

J.Hilk

@Mucahit said in How to exit void QIODevice::readyRead() function ? Is this function also interrupt?:

void QIODevice::readyRead()
{
data=serial.readAll();
totaldata=totaldata+data;
QMainWindow main;
main.update();
}

this should do nothing, that creates a new main window and destroys it immediately, you don't event show() it.

You have to signal your mainwinow instance to update, do that via a custom signal, emitted from your iodevice

jsulm

@Mucahit said in How to exit void QIODevice::readyRead() function ? Is this function also interrupt?:

QMainWindow main;

?
You create a NEW QMainWindow instance here!
Why don't you simply try to actually draw anything in your paintEvent?

Mucahit

I dont know how to do i am new in qt and c ++. I do not know how to renew my previously found chart pie in my main window with any data I read from the serial port. When the application is first opened, the chart pie is painted with 50 percent, but it will be 70 percent when any value comes from the serial port. Sorry for bothering you.

jsulm

@Mucahit You can redraw completely based on updated data. If paintEvent() is called the previously drawn content is deleted.

Mucahit

But I want to do this outside the readyread function when all the data reading is finished, so that it does not hinder my data reading speed. How can I do that ?

jsulm

@Mucahit First: you do it already outside of readyRead()! update() does not call paintEvent() directly, it only puts the event in the event queue. Second: do you have a protocol or how do you know when the reading is done? If you have a protocall when you can use a bool variable which is set to true when all data was read (and call update()) and use this variable inside paintEvent() to check whether you need to paint or not.

Mucahit

I understood very well what you said. Yes i have a protocol but i can't trigger paintevent.

jsulm

@Mucahit said in How to exit void QIODevice::readyRead() function ? Is this function also interrupt?:

but i can't trigger paintevent

Why not? You just need to store received data, set a bool flag and call update.

Mucahit

Now my code is below:

QByteArray data;
QByteArray totaldata;

void QIODevice::readyRead()
{
data=serial.readAll();
totaldata=totaldata+data;
if(totaldata=="xFe/xD3/xFD")
{
MainWindow main;
main.update();
// or main.repaint();
}
}
void MainWindow::paintEvent(QPaintEvent *)
{
qDebug()<<totaldata; // I want to print the data I receive here because I will draw pies based on the data I received
}

I see the data i read from the serial port in the output of this code, right? But i don't see anything. As i said, i can only get my qdebug output when I hover my mouse over the window. It never prints that value on the screen if I don't move it.

jsulm

@Mucahit Sorry, but this code does not make any sense! You already were told that creating a temporary QMainWindow instance and calling update() on it is simply wrong! You need to call update on your widget where you want to paint.
I suggest you take time to check some paintEvent examples, like https://doc.qt.io/qt-5/qtwidgets-widgets-analogclock-example.html

Mucahit

Thank you sir, i know how i can make a watch. All i want is to call paintevent in readyread function and i couldn't solve this problem. All i want is a line of code not an article, i can't. Please can you share how to call paintevent in function code to me ?

jsulm

@Mucahit said in How to exit void QIODevice::readyRead() function ? Is this function also interrupt?:

Please can you share how to call paintevent in function code to me ?

This already was explained: you do NOT call paintEvent directly, Qt framework does it for you when the widget needs to be repainted. You call update() to trigger a paint event when needed.
"All i want is a line of code not an article" - so, you want a magical line of code without spending time to learn how to paint? You already spent one day in forum. You could have read the example in this time to understand how it works...