fenster | display videos in a TextureView | Video Utils library

The tech-lead is wrong: your code is perfectly efficient the way it is written. Making the code shorter does not make it faster or more "elegant".

However, shorter code can be easier to maintain and change. Your code is fine as long as the number of heart containers is always exactly 12. But if you want to change that (to increase/decrease the difficultly of the game, or let the player get new heart containers) then this code won't work. It is hard-coded to work with exactly 12 heart containers only.

To change this, put this repetitive code in a loop. You'll need to look at the pattern of how the numbers change and create a small math formula for it. I've come up with the following. (I've also added constants instead of the integer literals, so that the code is easier to read and change.)

welcome to stack overflow. I modified your code a bit (you had a , after defining start_button and didnt import sys, also i put your code below ##### Window Setting #### into a boilerplate-code).

Your main issue was, that you do not make your Text widget available in your run and furthermore in your test function (executed as a thread). So i handed over your widget as an argument to both functions (probably not the most pythonic way, however). For executing a command bound to a button i used from functools import partial and binded the command including an argument via command=partial(run, textbox). Then i simply handed over the argument in run to the thread with args=[textbox] in the line where you create & start the thread. Finally, i updated the textbox with textbox.insert(tk.END, msg + "\n") in your test function while removing the print(). The insert appends any text at the end to the textbox, the "\n" starts a new line.

Here is the (slightly restructured) complete code (app.py):

First of all, in order to create a class in python, we use the class keyword, whereas in your code, you used the word Class, which will throw an error since it isn't a keyword in python.

So replace this line: Class Menü(Highscore): with this line: class Menü(Highscore):

Now to address your issue, see this answer. Basically, there are two ways in which you can prevent the error that you are getting:

The __init__ method in Button takes 5 arguments self, x, y, image, scale:

You should not access the control to get the data items. Rather access the source collection directly:

MainViewModel.cs

Your code could do with variable names that are more descriptive, like $addto --> $processId and $toadd --> $processName for instance. Now, you are making code where it is very easy to make mistakes..

Also, I do not see the need to scope almost every variable you have as $global:. If anywhere, you should use $script: scope inside scriptblocks that otherwise do not have any knowledge of these variables defined outside the scriptblock..

Anyway, to add values to the two columns in your ListView, you ca do this:

Lets take a closer look at this part of your code:

I just added my method hide_window to my main function, when script starts, you see something moving on to screen for 2ms.. but it disapears very quick... so thats something I can live with

Try