Intuyo que estás creando un decorador para validar los argumentos de la función division
.
El funcionamiento buscado sería:
>>> division(4, 2)
2
>>> division(4, 0)
No se puede dividir entre cero.
El decorador se puede usar como has puesto en el código, pero resulta más claro y pythónico usarlo con el prefijo @
:
@revisar
def division(a, b):
return a / b
Veamos cómo definir el decorador revisar
:
Como argumento entra una función de dos argumentos para validar que no sea 0 el segundo argumento. Para ello creamos una función interna que valide los argumentos antes de invocar a la función original:
def revisar(f):
def wrapper(x, y):
if y == 0:
print("No se puede dividir entre cero.")
return None
return f(x, y)
return wrapper
revisar(division)
nos devuelve una función wrapper de modo que wrapper(x, y)
chequea que y != 0
antes de ejecutar division(x, y)
.
Sólo existe un problema que suele pasar desapercibido: la función division
ha perdido su nombre e, incluso, sus docstrings (si las tuviera):
>>> division.__name__
'wrapper'
Sin entrar en detalles, la solución pasa por usar functools.wraps
:
from functools import wraps
def revisar(f):
@wraps(f)
def wrapper(x, y):
if y == 0:
print("No se puede dividir entre cero.")
return None
return f(x, y)
return wrapper
@revisar
def division(a, b):
"""División de dos números reales"""
return a / b
>>> division.__name__
'division'
>>> division.__doc__
'División de dos números reales'
return func(x, z)
si colocas el llamado a la función abajo delreturn
simplemente no se ejecuta ya que todo acaba en elreturn
return
pertenece alif
y está bien, lo que debes de colocar esreturn func(x, z)
pues debes de retornar el resultado del llamado a la función. Lo que haces ahí es solo llamarla y no retornas nada (por defectoNone
)