Выполнение функций в словаре переключателей

Все ваши функции выполняются при создании словаря, а не при доступе к ключу.

Вам нужно использовать lambda (без каких-либо параметров, они уже известны), чтобы убедиться, что функция вызывается только при необходимости:

 switcher = { 0: lambda : vertical.Vertical_Tank(level, area), 1: lambda : horiz.Horiz_Cylinder_Dished_Ends(dish, radius, level, length), 2: lambda : strapping.Calc_Strapped_Volume(Strapping_Table, level), 3: lambda : poly.Fifth_Poly_Calcs(Tank_Number) } 

затем вызовите, когда вы вернетесь, с сообщением об ошибке как лямбда, который возвращает его:

 return switcher.get(Tank_Shape, lambda : "ERROR: Tank type not valid")() 

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

• Переопределяя switcher при каждом вызове функции Tank_Shape_Calcs , это, как правило, не очень хорошая идея.
• Требование всех аргументов, которые должны быть переданы (из-за их определения как позиционеров), когда может потребоваться только несколько из них, вот почему мы имеем *args 🙂

Если мое понимание того, что вы делаете, правильное, я бы переключил переключатель вне функции, как Tank_Shape для function сопоставления объектов:

 switcher = { 0: vertical.Vertical_Tank, 1: horiz.Horiz_Cylinder_Dished_Ends, 2: strapping.Calc_Strapped_Volume, 3: poly.Fifth_Poly_Calcs } 

Затем определите Tank_Shape_Calcs чтобы взять лишние аргументы как кортеж с *args :

 def Tank_Shape_Calcs(Tank_Shape, *args): return switcher.get(Tank_Shape, lambda *_: "ERROR: Tank type not valid")(*args) 

и вызывать свою функцию после .get ting.

Это также .get трюк Джин, чтобы определить lambda в .get но делает это с помощью *_ , чтобы позволить ему получать .get со многими аргументами (которые, следовательно, игнорируются).

Что вы делаете в своем коде, создается словарь с целыми ключами (0-3), а результаты функции – как значения. Следовательно, вы сначала вызываете все функции, а затем получаете доступ к возвращаемым значениям этих функций. Я бы изменил ваш код следующим образом:

 def Tank_Shape_Calcs(Tank_Shape, level, area, dish, radius, length, Strapping_Table, Tank_Number): switcher = { 0: (vertical.Vertical_Tank, (level, area)), 1: (horiz.Horiz_Cylinder_Dished_Ends, (dish, radius, level, length)), 2: (strapping.Calc_Strapped_Volume, (Strapping_Table, level)), 3: (poly.Fifth_Poly_Calcs, (Tank_Number,)) } func, args = switcher.get(Tank_Shape, (None, None)) if func is not None: return func(*args) 

Здесь вы сначала получаете функцию, которую хотите вызвать, с соответствующими аргументами и вызывать ее.