глобальная переменная не работает питон

Question

Здравствуйте разработчики,

я пишу код, который принимает пользовательский ввод и инициализирует класс в зависимости от мощности, как в примере кода ниже:

class X: 
    def __init__(self): 
     return 

    def run(self): 
     print("i am X") 

def func1(cls): 
    exec("global " + cls.lower()) 
    exec(cls.lower() + " = " + cls + "()") 

def func2(mode_to_set): 
    exec(mode_to_set.lower() + ".run()") 

но, как я запустить такой код:

я получаю эту ошибку:

Traceback (most recent call last): 
    File "/Users/noahchalifour/Desktop/test.py", line 16, in <module> 
    func2('X') 
    File "/Users/noahchalifour/Desktop/test.py", line 13, in func2 
    exec(mode_to_set.lower() + ".run()") 
    File "<string>", line 1, in <module> 
NameError: name 'x' is not defined 

Может ли кто-нибудь мне помочь?

Answer 1

Похоже, вы бы лучше с func2 Instantiate и запустить метод:

def func2(mode_to_set): 
    globals()[mode_to_set]().run() 

Таким образом, вы не имеете целую кучу хлама плавучего нежелательно о в вашей глобальной пространство имен, и вы не делаете недоверенный exec. Кроме того, exec ing a global Оператор внутри функции не работает (как вы видели) ... exec - это способ выполнения строки, как если бы это был код. Он не является способом отбрасывать динамически созданные заявления в текущую функцию.

Answer 2

Намного лучше создать экземпляр класса на основе входных данных пользователя будет использовать «заводской шаблон»:

http://python-3-patterns-idioms-test.readthedocs.io/en/latest/Factory.html

В основном вы создаете класс, вся цель состоит в том, чтобы создать другие классы, основанные на ценность. Некоторым людям может показаться, что это излишне, поэтому вы также можете использовать функцию, которая создает классы на основе ввода.

Независимо от того, что вы делаете, то, как вы это делаете сейчас, работает с исходными строками ввода-вывода с помощью exec, это плохая идея. Наилучшим вариантом является то, что он вводит новые ошибки, которые почти невозможно отслеживать, поскольку они фактически не записаны нигде. В худшем случае пользователь каким-то образом находит способ отправить строку функции, вы в значительной степени уничтожили всю безопасность, на которую надеялись.

В принципе «exec» обычно должен быть последним средством. Как правило, более элегантные и безопасные способы решения проблемы.

Answer 3

Словари, словари, словари. Ваша программа должна поддерживать контроль над тем, какой код выполняется, вместо того, чтобы позволить пользователю создавать новый код динамически.

classes = {'X': X} 
instances = {} 

def func1(cls): 
    var = cls.lower() 
    instances[var] = classes[cls]() 

def func2(mode_to_set): 
    instances[mode_to_set.lower()].run() 

func1('X') 
func2('X') 

Единственное отличие состоит в том, что вы не имеете глобальную переменную с именем x; у вас есть глобальный словарь с ключом x, который ссылается на ваш экземпляр.