Почему я не могу изменить переменную в другом модуле?

Question

[Никита Каргин]

У меня есть несколько модулей python
classes.py:

class screen:
    def __init__(self,elements):
        self.elements=elements
    def show(self):
        for element in self.elements:
            self.elements[element].show()
    def execute(self):
        for element in self.elements:
            if self.elements[element].executable:
                self.elements[element].execute()
        command = input()
        self.elements[command].action()
class button:
    def __init__(self,text,action):
        self.executable=False
        self.text=text
        self.action=action
    def show(self):
        print(self.text)
class mission:
    def __init__(self,structure,actions):
        self.structure=structure
        self.actions=actions

funcs.py

from game import setScreen
from classes import *
import sys
def StartGame():
    setScreen(StartGameScreen)
def Settings():
    setScreen(SettingsScreen)
def Quit():
    sys.exit()
def NewGame():
    pass
def LoadGame():
    pass
def DeleteGame():
    pass

StartGameButton=button('Начать игру',StartGame)
SettingsButton=button('Настройки',Settings)
QuitButton=button('Выйти из игры',Quit)
screen1=screen({'1':StartGameButton,'2':SettingsButton,'3':QuitButton})
NewGameButton=button('Новая игра',NewGame)
LoadGameButton=button('Загрузить',LoadGame)
DeleteGameButton=button('Удалить запись',DeleteGame)
StartGameScreen=screen({'1':NewGameButton,
                        '2':LoadGameButton,
                        '3':DeleteGameButton})

game.py

from funcs import *
import sys
curscreen=None
def setScreen(value):
    global curscreen
    curscreen=value
    curscreen.show()
try:
    curscreen=screen1
    curmission=None
except NameError:
    pass
try:
    while 1:
        curscreen.show()
        curscreen.execute()
except (NameError,AttributeError):
    pass

С помощью функции setScreen я хочу изменять значение переменной curscreen в модуле game из модуля funcs. Если вызвать setScreen(StartGameScreen) в интерактивном режиме из модуля game.py, то все идет как надо.
Однако если вызвать функцию StartGame, содержащую все тот же setScreen(StartGameScreen), то ничего не поменяется. В чем проблема?

Comments

[rPman]

вставь отладочные print, может прояснится что

[Никита Каргин]

rPman, там стоит curscreen.show(). Это отладочная команда. В случае, если менять через setScreen и в случае с StartGame() выводится все как будто curscreen действительно изменился, но только в случае с setScreen он реально меняется.
Может где-то еще вставить print?

[rPman]

пльцем в небо, может вызываешь другой setScreen

[Никита Каргин]

rPman, Проверил еще раз, только один setScreen

[rPman]

добавь в каждую строчку функции print 1..2.3..4..5...
либо отладчик

ибо происходит что то не то

[Никита Каргин]

rPman, Все равно ничего не понятно. Если можешь, посмотри сам. Для меня все выглядит как будто в обоих случаях все работает одинаково, но во втором почему то результат не тот

[MinTnt]

Такая странная структура кода в файлах. Как я понял главный файл game.py, который
импортирует функции "from funcs import *". При этом в самом файле funcs.py импортируется функция самой game.py "from game import setScreen".

По итогу в предоставлении самого game.py это выглядит как наличие в одном файле двух точно идентичных функций setScreen. Хоть и не очень помню специфику Python, относительно импорта себя из себя самой. Но здесь или ошибка, или неправильная работа скрипта гарантирована

[MinTnt]

Крч, я не очень понял какой файл главный, но приблизительную проблему тебе описал.

[Andy_U]

Однако если вызвать функцию StartGame, содержащую все тот же setScreen(StartGameScreen), то ничего не поменяется.

Нигде не вижу вызова функции StartGame. И проблемы, соответственно.

[Никита Каргин]

MinTnt, есть варианты как это правильнее сделать?

[Никита Каргин]

Andy_U, в коде она не вызывается напрямую. StartGame это атрибут action у StartGameButton. Если после запуска программы ввести 1, то будет вызвана эта функция

[Andy_U]

Никита Каргин, Ага. И что при этом происходит неправильно с вашей точки зрения?

[Никита Каргин]

Переменная curscreen должна стать равной StartGameScreen, но она не меняется

[Andy_U]

Никита Каргин, С чего вы это взяли?

[Никита Каргин]

Andy_U, что при запуске приложения,что в интерактивном режиме при вызове функции ничего не происходит

[Andy_U]

Никита Каргин, Как это не происходит? А кто вам текст после ввода единички печатает в консоль?

"C:\Program Files\Python38\python.exe" "D:/My Documents/PycharmProjects/toster/game.py"
Начать игру
Настройки
Выйти из игры
1
Новая игра
Загрузить
Удалить запись
Начать игру
Настройки
Выйти из игры

[Никита Каргин]

Это отладочная команды. Как видите, последние 3 строчки показывают, что curscreen все же остаётся равным screen1

[Andy_U]

Никита Каргин, Какой screen1 - он у вас неопределен, и вы вылетаете из первого try. А нет, подождите.

[Никита Каргин]

Он определен в funcs

[Andy_U]

Никита Каргин, В общем, не пишите так больше таких головоломок.

[Никита Каргин]

MinTnt, game - главный
funcs - функции
classes- классы

[MinTnt]

Никита Каргин, ну в принципе предполагаю что проблема решена.

Answer 1

[Никита Каргин]

Была неправильная структура файла, получался перекрестный импорт
Исправленный код:
game.py

from funcs import *
from classes import *
import sys
try:
    curscreen=screen1
except:
    pass
try:
    while 1:
        curscreen.show()
        curscreen.execute()
        if curscreen.newScreen!=None:
            curscreen=curscreen.newScreen
except (NameError,AttributeError) as e:
    print(e)

funcs.py

from classes import *
import sys

def StartGame():
    return StartGameScreen
def Settings():
    return SettingsScreen
def Quit():
    sys.exit()
def NewGame():
    pass
def LoadGame():
    pass
def DeleteGame():
    pass
NewGameButton=button('Новая игра',NewGame)
LoadGameButton=button('Загрузить',LoadGame)
DeleteGameButton=button('Удалить запись',DeleteGame)
StartGameScreen=screen({'1':NewGameButton,
                        '2':LoadGameButton,
                        '3':DeleteGameButton})
StartGameButton=button('Начать игру',StartGame)
SettingsButton=button('Настройки',Settings)
QuitButton=button('Выйти из игры',Quit)
screen1=screen({'1':StartGameButton,'2':SettingsButton,'3':QuitButton})

classes.py

class screen:
    def __init__(self,elements):
        self.elements=elements
    def show(self):
        for element in self.elements:
            self.elements[element].show()
    def execute(self):
        for element in self.elements:
            if self.elements[element].executable:
                self.elements[element].execute()
        command = input()
        newScreen=self.elements[command].action()
        if newScreen!=None:
            self.newScreen=newScreen
class button:
    def __init__(self,text,action):
        self.executable=False
        self.text=text
        self.action=action
    def show(self):
        print(self.text)
class mission:
    def __init__(self,structure,actions):
        self.structure=structure
        self.actions=actions