2

No es una tarea, estoy aprendiendo programación con python con un vídeo tutorial de Youtube y el ejercicio una vez transcrito en código no me da los mismos resultados que en el vídeo y no se por que.

Simplemente no me hace la división y me da como resultado "None", no puedo pasar a Decoradores por este tipo de código. Agradecería su ayuda.

    def revisar(func):
      def validar(x, z):
        if z == 0:
          print("No se puede dividir entre cero.")
          return
        func(x, z)
      return validar
    
    def division(a, b):
      return a / b
    
    division = revisar(division)
    
    print(division(4, 2))
3
  • 2
    Debes colocar return func(x, z) si colocas el llamado a la función abajo del return simplemente no se ejecuta ya que todo acaba en el return
    – Christian
    Commented el 2 jun. 2022 a las 11:40
  • 1
    Gracias, esta noche l volver analizo mejor el porque ya que necesito entenderlo para meterme con los Decoradores. Gracias Commented el 2 jun. 2022 a las 12:32
  • Perdona, me he confundido, al estar en celular no lo he visto bien. El return pertenece al if y está bien, lo que debes de colocar es return 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 defecto None)
    – Christian
    Commented el 2 jun. 2022 a las 13:25

3 respuestas 3

3

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'
2
  • 1
    Tu explicación va mas allá de por donde voy en el vídeo curso y me cuesta seguirte. Aun así gracias. Commented el 5 jun. 2022 a las 12:19
  • 1
    Siempre hay que tener presente que estas respuestas deben servir a otras gentes que se pasen por aquí el futuro. Seguramente, tu pregunta esté ya respondida con la primera parte de mi respuesta. Siempre puedes pedir que te aclaremos lo que no veas claro. Commented el 6 jun. 2022 a las 16:08
2

Dentro de la función validar, en caso de que no se cumpla la condición z == 0, debes retornar la función func()

def validar(x, z):
    if z == 0:
      print("No se puede dividir entre cero.")
      return
    # Aquí corriges
    return func(x, z) # antes no había return
0

tengo una respuesta simple para esta pregunta, que seria esto:

Código

def revisar(func):
  def validar(x, z):
    if z == 0:
      print("No se puede dividir entre cero.")
      return
    return func(x, z) # Lo único que hay que arreglar
  return validar

def division(a, b):
  return a / b

division = revisar(division)
print(division(2, 1))

El código es el mismo, lo unico que habria que arreglar de ese código es ponerle return a la función que admite los parámetros: func(x, z) y así no mostraría None al final y listo.

Ojo: quiero aclarar que en los comentarios de arriba ya sugirieron esto, pero quise ponerlo para responder a la pregunta, además que las respuestas que ya están son un poco confusas.

1
  • Las de arriba no son comentarios, son respuestas, igual que la tuya. Esto no es un foro. Es un sitio con una modalidad diferente. Esto que estás leyendo, sí es un comentario. Te sugiero hacer el recorrido de bienvenida y consultar el centro de ayuda para aprender como funciona el sitio. Por cierto, puedes dejar acotaciones en los comentarios o sugerir ediciones (mejorando y sin alejarse de la intención del autor) a las respuestas si crees conveniente.
    – Mateo
    Commented el 5 ago. a las 4:50

Tu Respuesta

By clicking “Publica tu respuesta”, you agree to our terms of service and acknowledge you have read our privacy policy.

¿No es la respuesta que buscas? Examina otras preguntas con la etiqueta o formula tu propia pregunta.