Почему def не видет переменную, которая была объявлена глобальной?

Question

[3zd1]

Программа выдает ошибку:

NameError: name 'priceshop' is not defined

на 33 строчке.
Саму переменную я создавал ниже и в функции def она была объявлена глобальной. При этом переменную bar в функции def auto_bar() он не считает ошибкой. Предполагаю, что проблема в том, что сама переменная создается в def shop() и считается "локальной".

#библиотеки
from tkinter import *
from tkinter.ttk import Progressbar

#переменные
cnt = 0
cnt = int(cnt)
bar_cnt = 0
bar_cnt = int(bar_cnt)
pricePB_int = 10
pricePB_int = int(pricePB_int)
pricePB_str = pricePB_int
pricePB_str = str(pricePB_str)

#вызов функции клика
def clicked():
	global cnt
	cnt += 1
	wtext.configure(text = "Печенье: " + str(cnt))

def auto_price():
	global pricePB_int
	global pricePB_str
	global fgPS
	global priceshop

	if cnt < pricePB_int:
		fgPS = '#FF0000'
	else:
		fgPS = "#000000"
	priceshop.configure(text = "Цена: " + str(pricePB_str))

# вызов функции прогресс бара
def auto_bar():
	global bar_cnt
	global cnt
	print('Progressbar')
	if bar_cnt >= 100:
		bar_cnt = 0
		bar_cnt += 5
		cnt += 1
		print(">100")
	else:
		bar_cnt += 5
		print("<100")
	bar.configure(value = bar_cnt)
	wtext.configure(text = "Печенье: " + str(cnt))
	auto_price()
	window.after(1000, auto_bar)

def buying():
	print('buy')

#вызов магазина
def shop():
	global pricePB_int
	global pricePB_str
	wshop = Toplevel()
	wshop.resizable(False, False)
	wshop.title("Магазин")
	shopx = window.winfo_x() + ((window.winfo_width() / 2) - 320)
	shopy = window.winfo_y() + ((window.winfo_height() / 2) - 240)
	wshop.geometry('%dx%d+%d+%d' % (240, 144, shopx, shopy))
	wshop.iconbitmap(r'shop.ico')

	btnshop = Button(wshop, text = "Купить", bg = "#FF9900", activebackground="#FF6600", command = buying)
	btnshop.place(anchor = "center", x = 120, y = 72)

	priceshop = Label(wshop, text = "Цена: " + str(pricePB_str))
	priceshop.place(anchor = "center", x = 120, y = 95)
	priceshop['fg'] = fgPS

	PBupgrage = Label(wshop, text = "Скорость работы: 1с" + " -> " + "1.2c")
	PBupgrage.place(anchor = "center", x = 120, y = 45)

#параметры интерфейса
window = Tk() # создание окна
window.title("Кликер")
window.geometry('640x480')
window.resizable(False, False)
window.config(bg = "#35f29d")
window.iconbitmap(r'cookie.ico')

#счетчик
wtext = Label(window, text= "Печенье: " + str(cnt), bg = "#35f29d")
wtext.place(anchor = "n", x = 320, y = 180)

#кнопка
btn = Button(window, text="Клик", command = clicked, bg = '#FF9900', activebackground="#FF6600")
btn.place(anchor = 'center', x = 320, y = 220)

#прогресс бар
bar = Progressbar(window, length=200)
bar.place(anchor = "center", x = 320, y = 10, relwidth = .25)
bar['value'] = bar_cnt

#меню
menu = Menu(window)
menu.add_cascade(label='Магазин', command = shop)
window.config(menu=menu)

#запуск функции
auto_bar()

window.mainloop() # программа в ожидании действий

Comments

[Everything_is_bad]

для новичков глобальные переменные зло, а ты еще и не понимаешь как это работает, просто перестань их использовать.

[3zd1]

Everything_is_bad, а как тогда использовать функцию def, если она не видит локальные переменные? Или чем заменить можно def?

[Vindicar]

3zd1, привыкай использовать классы.

[Everything_is_bad]

3zd1, а откуда вообще подобный трэш появился?

cnt = 0
cnt = int(cnt)
bar_cnt = 0
bar_cnt = int(bar_cnt)
pricePB_int = 10
pricePB_int = int(pricePB_int)

[3zd1]

Everything_is_bad, без написания этих переменных в начале, ниже код не находит их и выдает ошибку. Сначала для каждой переменной выдаю значение, после определяю тип переменных. Как сократить - не знаю и не вижу в этом смысл на данный момент работы.

[Everything_is_bad]

3zd1, а ты точно понимаешь что ты делаешь? bar_cnt = 0 ты уже как int определил, bar_cnt = int(bar_cnt) бессмыслено

[3zd1]

Everything_is_bad, до твоего объяснения, честно, нет. считал, что переменным надо назначать тип (int, str) для корректной работы. Я бы хотел уточнить, как использовать одну переменную и для текста и для подсчетов? Просто 2 переменные я использую с одинаковыми значениями, но для разных целей:

pricePB_str = pricePB_int
priceshop = Label(wshop, text = "Цена: " + str(pricePB_str))
if cnt < pricePB_int:

[Everything_is_bad]

3zd1, pricePB_str тут лишняя

[3zd1]

Everything_is_bad, почему? Если использовать только pricePB_int, он выдаст ошибку: TypeError: can only concatenate str (not "int") to str

[Everything_is_bad]

3zd1, если не делать str(pricePB_int), то конечно будет, у тебя проблемы с базой, порешай задачи попроще и без tkinter

[3zd1]

Everything_is_bad, я так пытался делать, но он потом жалуется на то, что нельзя применять в

if cnt < pricePB_int:

str. А каждый раз менять тип переменной ради одного действия - бессмыслица. Проще назначить две переменные с одинаковым значением.

[Everything_is_bad]

3zd1, а зачем менять? про f-строки я даже не заикаюсь

priceshop = Label(wshop, text = "Цена: " + str(pricePB_int))
if cnt < pricePB_int:

[3zd1]

Everything_is_bad, если я написал данную строку, значит в моменте он из за этого выдал ошибку, связанную с типом. Сейчас я не смогу повторить этот момент, чтобы показать для чего поменял. Можешь, пожалуйста объяснить, чем мой вариант плох? Он в будущем мне доставит много хлопот или в большом коде будет, как палка в колесе?

[Everything_is_bad]

3zd1, я не знаю как объяснить такие базовые вещи, твой вариант избыточен, ты зачем-то делаешь совершенно не нужные действия.

Answer 1

[Сергей Горностаев]

global - это инструкция не о том, где создать переменную, а где искать.

Comments

[Максим Припадчев]

Я бы даже назвал, global подсказкой для python, python - не генерирует байт код, для global. И по сути это всего лишь подсказка для python, на этапе определения области видимости переменных.

[3zd1]

Максим Припадчев, то есть, я правильно понимаю, что def ищет переменные внутри себя, а global указывает на переменные, которые были созданы вне функции def? Тогда почему он не видит переменную priceshop, если та была объявлена глобальной в функции def auto_price()?

[Everything_is_bad]

3zd1, перестань придумывать как работает global и почитай учебник

[Максим Припадчев]

3zd1,

если та была объявлена глобальной в функции def auto_price()?

global не объявляет переменные. global подсказывает в какой области видимости рассматривать уже объявленную переменную.

пример

a = 1

def f():
    # global a
    for i in range(5):
        a= a + 1
    print(a)
f()

если не осуществить подсказку, то python решит, что 'a' находится в локальной области видимости, и не найдет ее. Если раскоментировать 'global' a то будет работать, как ожидается. Что бы это понимать надо почитать про области видимости, про мутабельность объектов, ну и про дандер методы тоже, потому что есть нюансы, как определены оперции присваивания, арифметические и т.д. Может быть множество нюансов. Например вот так уже global влиять не будет.

a = 1

def f():
    # global a
    print(a + 1)
f()

есть он или нету, python ,будет искать a в локальной, потом не локальной, потом в глобальной, потом во встроенных, ну и при таком сценарии он найдет ее в глобальной области видимости.

[3zd1]

Everything_is_bad, хорошо. Мне выше посоветовали начать пользоваться классами, вместо классами. Я сейчас изучаю классы и может у меня мало знаний, но как классы могут помочь? Добавить priceshop в класс, который не находится в функции def? Так ему все равно надо дать какой то параметр: значение или тип. А

priceshop = Label(wshop, text = "Цена: " + str(pricePB_int))

не принимает никакие значения или типы (int, str, None). Подскажи, пожалуйста, в каком направлении мне думать над решением ошибки, или какие учебники почитать, на какую тему?

[3zd1]

Максим Припадчев, Спасибо большое! Обязательно возьму на заметку предложенные темы