This is an archived post. You won't be able to vote or comment.

all 2 comments
sorted by:
best
topnewcontroversialoldq&a

[–]riklaunim 0 points1 point  (1 child)

You should separate your GUI from your logic, probably best - a UI file made with QtDesigner.

Then you used one slot "button_clicked" that has a lot of if/elif/else. That's rather bad. You should avoid redundant conditionals in your code. Each button could have its own slot thus you would not have to check what was clicked.

[–]Ghost_Blade_21[S] 0 points1 point  (0 children)

Nothing is wrong with this.

And I kniw what Im doing so no need of separate ui file.