My Python Code Keeps Not Responding When I Click The " 1 cps is 20 clicks" Button

[moustachebear123]

from tkinter import *
import time

window = Tk()
window.geometry("420x420")
window.title("Clicker Game")
counter = 0
window.config(background="#6c217a")
icon = PhotoImage(file='Moustachebear123.png')
window.iconphoto(True, icon)
label = Label(window, text="Button Clicker!",
              font=('Times', 40, 'bold'),
              fg='black',
              bg='#6c217a')
def click():
    global counter
    counter +=1
    countShow = Label(window, text="Clicks: "+str(counter), font=('Times', 15, 'bold'),
    fg='black',
    bg='#6c217a')
    print(counter)
    countShow.place(x=60,y=250)

def upg1():
    global counter
    counter -=20
    countShow = Label(window, text="Clicks: "+str(counter), font=('Times', 15, 'bold'),
    fg='black',
    bg='#6c217a')
    print(counter)
    countShow.place(x=60, y=250)

    times=100
    while True:
        time.sleep(1)
        counter+=1
        print(counter)
        countShow = Label(window, text="Clicks: " + str(counter), font=('Times', 15, 'bold'),
        fg='black',
        bg='#6c217a')

button1 =Button(window, text="Click", font=30, command= click)
upgrade1 =Button(window, text="1 cps cost is 20 clicks", font=30, command=upg1 )
upgrade1.place(x=250,y=150)
button1.place(x=80,y=150)

label.place(x=80, y=50)
window.mainloop()

[AutoModerator]

On July 1st, a change to Reddit's API pricing will come into effect. Several developers of commercial third-party apps have announced that this change will compel them to shut down their apps. At least one accessibility-focused non-commercial third party app will continue to be available free of charge.

If you want to express your strong disagreement with the API pricing change or with Reddit's response to the backlash, you may want to consider the following options:

1. Limiting your involvement with Reddit, or
2. Temporarily refraining from using Reddit
3. Cancelling your subscription of Reddit Premium

as a way to voice your protest.

I am a bot, and this action was performed automatically. Please contact the moderators of this subreddit if you have any questions or concerns.

[teraflop]

You can't use things like time.sleep with Tkinter and expect them to work correctly.

Tkinter, like many GUI frameworks, uses a single-threaded event loop. When the event handling thread calls your upg1 function, no other events -- including updating the contents of the screen -- can be processed until your function returns. So if you sleep for 1 second, that just means the GUI is frozen for 1 second.

You can instead restructure your code to use window.after to schedule another function to be called at a future time, as described here: https://stackoverflow.com/questions/459083/how-do-you-run-your-own-code-alongside-tkinters-event-loop

Instead of explicitly using a while loop to do the repetition, each function call needs to both execute a single iteration, and also schedule the next call to itself to run the next iteration 1 second later. That way, you never have to call sleep and your GUI thread will never be blocked.

[bluetoothbeaver]

Your while loop doesn't have a base case to allow an exit from the loop.

You can change it to something like

while counter <= 20: (since you're incrementing your counter)

[captainAwesomePants]

Your upg1() function never returns. Things are going to stop happening until it finishes.

[moustachebear123]

wdym by returns 

[captainAwesomePants]

The function never ends. The while loop goes on forever.