En el momento de la definición del diccionario estás invocando las funciones f1
y f2
, y son los valores devueltos los que se insertan en el diccionario.
Si lo que quieres es un switch, un modo sería:
funcs = { '1': f1, '2':f2 }
while True:
selection = getOption()
f = funcs.get(selection, None)
if not f is None:
f(x,y)
else:
print "No existe la función"
Problemas:
- Todas las funciones tienen que usar el mismo número de argumentos, aunque no usen todos los argumentos
- Todas las funciones retornan algún valor
- El control de errores crece a medida que aumenta el número de casos del switch, por lo que es mejor realizarlo dentro de cada función.
Por supuesto, cada uno de estos problemas se pueden evitar, pero mi recomendación es que uses estructuras if..elif..else
y evitar propagar errores a otras partes si se puede (eg: getOption
debería controlar sus propios errores)
seleccion = getOption()
if seleccion == 1:
f1(x,y)
elif seleccion == 2:
f2(x)
else:
print "Selección errónea"
getOption
?