Unsolved Resizing mainwindow using resizeEvent works very slow
-
In drawPath() you always call readPosition(inFile);
I don't know what readPosition() really does, but if it always read the file then you read the file each time you call drawPath(). -
@jsulm said:
So, you should not draw anything inside resizeEvent()
resizeEvent() does not directly call paintEvent() but I have paintEvent() in my widget to draw a line ....
When the window is resize, the point (x, y) will be recalculated to match the window size so they get to the right coordinate. If I can not use paintEvent(), how can I draw a line when resize?
-
I have tried to call calculatedXY(someVector, width, height) in resizeEvent() without calling drawPath() but it doesn't seem to work.... it seems like the vector data from readPosition(inFile) doesn't pass to someVector although I have declared it as global variable. Any suggestions?
-
You should debug your application to find out where it is hanging for 4 seconds.
-
Well, I don't know how readPosition() is implemented. But in general if the data in the file is static you should read that file only once (for example when your application is starting). To me it looks like you're doing it each time drawPath() is called.
-
@jsulm said:
Well, I don't know how readPosition() is implemented. But in general if the data in the file is static you should read that file only once (for example when your application is starting). To me it looks like you're doing it each time drawPath() is called.
so here is my readPosition()
void DrawPath::readPosition(QString inFile_) { //read from text file QFile file(inFile_); if(!file.open(QIODevice::ReadOnly | QIODevice::Text)){ qDebug() << "error: unable to open..."; } QTextStream in(&file); while(!in.atEnd()){ QString lineData = in.readLine(); QStringList listData = lineData.split("\t"); someX << listData[4].toFloat(); someY << listData[5].toFloat(); } for(int i = 0; i < someX.size()-1; i++){ //collect vector data into coordinate(x,y) xyPos.setX(someX[i]); xyPos.setY(someY[i]); //create vector of coordinate(x, y) xyPosVector.append(xyPos); } qDebug() << "Successfully loaded!"; file.close(); }
At the moment, the data is static but it will be updated maybe daily for the complete version of the program. I would like to know if it's gonna be lagged.
-
Edit: the program will be used for a realtime (x, y) calculation so I need a fast working resize window.
If you want a "realtime" behavior, then thread the IO! It's not the
QWidget::resizeEvent
's job to read files and do (complicated) calculations. In your worker thread or main thread (by means of scheduling, but it's better to actually thread the operation) do the calculations and update a couple of fields. Then in the resize or paint events do the bare minimum of setting the geometry or painting the data. Events are queued and no event will be processed if the last one hasn't finished, so a slow resize event processing will cause your application to hang, not update widgets and not respond to user input, not process sockets or timers!DrawPath::readPosition
is heavy (it reads a whole file), as @jsulm already suspected, and you do that in the event function(s), so everything is waiting for the IO to complete.Kind regards.
-
@soloespresso readPosition() does not set someVector!
If the change frequency of the file is so low then you should not read it all the time - it's just wasted time. You can use QFileSystemWatcher class to detect changes in this file and then reread it. -
@kshegunov said:
If you want a "realtime" behavior, then thread the IO!
And ow can I do that?
I have tried calling only the function to calculate (x, y) without calling drawPath() in resizeEvent() something like this
path->calculatedXY(someVector, width, height);
but it gives me Runtime Error and "index out of range" for the vector used in that function.
-
@soloespresso said:
And ow can I do that?
Look here for an introduction.
but it gives me Runtime Error and "index out of range" for the vector used in that function.
See @jsulm's explanation as to why this is happening.
Additionally yourfor
loop is wrong:for (int i = 0; i < someX.size()-1; i++) {
this skips the last element in the vector.
Kind regards.
-
@jsulm said:
@soloespresso readPosition() does not set someVector!
I have some other functions in between. The readPosition() I posted is to give an idea what readPosition() does which is extract the data and keep them in a vector.
If the change frequency of the file is so low then you should not read it all the time - it's just wasted time. You can use QFileSystemWatcher class to detect changes in this file and then reread it.
OK, I will take a look on that.
-
@kshegunov said:
@soloespresso said:
And ow can I do that?
Look here for an introduction.
Ok, I will take a look.
but it gives me Runtime Error and "index out of range" for the vector used in that function.
See @jsulm's explanation as to why this is happening.
Additionally yourfor
loop is wrong:for (int i = 0; i < someX.size()-1; i++) {
this skips the last element in the vector.
Kind regards.
Right!!! I didn't see it. Thank you.
-
for (int i = 0; i < someX.size()-1; i++) {
fix this but still Runtime Error and "index out of range".
I will check QFileSystemWatcher and thread after.
-
fix this but still Runtime Error and "index out of range".
This hasn't anything to do with your error, it was just an observation. As to the assertion (which I'm assuming your code is failing), you should track that in the debugger.
-
@kshegunov said:
fix this but still Runtime Error and "index out of range".
This hasn't anything to do with your error, it was just an observation. As to the assertion (which I'm assuming your code is failing), you should track that in the debugger.
So let me make sure if what I am doing is ok for Qt.
I have drawPath() in QWidget.void DrawPath::drawPath(QString inFile, QString outFile, float width, float height) { readPosition(inFile); calculatedXY(someVector, width, height); writePosition(outFile, latLongPointVector); }
Can I call calculatedXY(someVector, width, height); outside drawPath() in resizeEvent() like this?
void MainWindow::resizeEvent(QResizeEvent *event) { //scaled window size scaledWidth = double(event->size().width())/360.0; scaledHeight = double(event->size().height())/180.0; //QString inFile = ":/input.txt"; //QString outFile = "output.txt"; //set label window size ui->label->setGeometry(0, 0, 360*scaledWidth, 180*scaledHeight); //set draworbit window size //path->drawPath(inFile, outFile, scaledWidth, scaledHeight); path->calculatedXY(someVector, scaledWidth, scaledHeight); path->setGeometry(0, 0, 360*scaledWidth, 180*scaledHeight); }
-
@soloespresso
No one is preventing you from callingcalculatedXY()
, but if it's a good design choice really depends what that function actually does. -
@kshegunov said:
@soloespresso
No one is preventing you from callingcalculatedXY()
, but if it's a good design choice really depends what that function actually does.Thank you for the confirmation.
Because calling drawPath() will call readPosition(), calculatedXY() and writePosition(), calling calculatedXY() directly will prevent resizeEvent() to reread the text file.But this give me that Runtime Error and "index out of range" which I will debug later.
Now the problem is how to draw the line? I use paintEvent() to draw a line and the function is called when drawPath() is called.
-
"Now the problem is how to draw the line?" - what is the problem with that? When paintEvent() is called you draw the line.
-
@jsulm said:
"Now the problem is how to draw the line?" - what is the problem with that? When paintEvent() is called you draw the line.
But then it means paintEvent() is called directly in resizeEvent()?
Is something like the code below possible?void MainWindow::resizeEvent(QResizeEvent *event) { //scaled window size scaledWidth = double(event->size().width())/360.0; scaledHeight = double(event->size().height())/180.0; //QString inFile = ":/input.txt"; //QString outFile = "output.txt"; //set label window size ui->label->setGeometry(0, 0, 360*scaledWidth, 180*scaledHeight); //set draworbit window size //path->drawPath(inFile, outFile, scaledWidth, scaledHeight); path->calculatedXY(someVector, scaledWidth, scaledHeight); path->paintEvent(); path->setGeometry(0, 0, 360*scaledWidth, 180*scaledHeight); }
-
No, paintEvent() is called by Qt whenever a repaint is needed.
For example it is called after resizing a widget (as stated in documentation).