PyQt5 closeEvent reimplementation

user4592357

thanks for a good answer. i've tried many "solutions" (including ones in the mentioned links) but none of works as a expected. now i understand why.

and you're right, since i'm going to deploy the application sooner or later, there is no need to worry about such things

user4592357

btw, is there a way to pass command line arguments to a deployed app (sounds surreal but im asking)?

SGaist

Do you mean like QCommandLineParser ? Or Python's argparse module ?

JonB

@user4592357
What do you mean about deployed app & argument passing? If by any chance you mean can you pass arguments once the app is launched from the desktop via a shortcut, then yes. In general, your PyQt app's __main__ sees sys.argv like normal.

user4592357

@JonB,
yes that's what i meant. i'll try that.

@SGaist,
should i reimplement __del__() for calling __write_settings()? the book which i use as a reference says that, "in all X classes presented in the book, none reimplements it"

SGaist

Can you give the reference ?

user4592357

@SGaist
https://fatima-python.wikispaces.com/file/view/Rapid+GUI+Programming+with+Python+and+Qt.pdf
page 79 (book page, pdf page is 93)

JonB

@user4592357
I wouldn't put anything as complicated as saving settings in a destructor. Your closeEvent() sounded right for this. Why are we suddenly discussing __del__()?

SGaist

Because of this:

@SGaist said in PyQt5 closeEvent reimplementation:

Personally, I usually write down settings in the destructor of a class rather than close event.

That's what I do in C++.

JonB

@SGaist
Ohhh, didn't see that.

Umm, far be it for me to disagree, but personally I wouldn't. I thought destructors were supposed to be cheap and have no side-effects. So many things can happen when saving settings. I wouldn't do it in C# FWIW, and I wouldn't do it in Python. (I also would not load settings in __init__().)

user4592357

@SGaist
what about the book's statement?

SGaist

In the python case, that's indeed something debatable. Most of the time, people don't need to implement __del__.

What I would do is to store the settings once you close the corresponding dialog so that you avoid the trouble you had with the unintended interruption you have (unless it's done while the dialog is open).

In the extreme case, you can even save you settings on modification if you have an "apply immediately" without cancel style of application preferences.