Skip to main content
Ampliación de la respuesta
Origen Enlace
abulafia
  • 58.7k
  • 3
  • 54
  • 89

Ampliación (respondiendo a la ampliación de la pregunta)

El usuario publica su código, en el que intenta usar de nuevo el decorador como un método de una clase, con esta sintaxis:

def database_managment(self, *args):
    def connect_database(*args):
        self.connection = sqlite3.connect('database/Bank-Data')
        self.cursor = connection.cursor()
        function(*args)
        self.connection.commit()
        self.connection.close()

    return connect_database

y dice que le "funciona".

Lo cierto es que ese código no puede funcionar por dos razones:

  1. El decorador tiene más parámetros de los que debería. Un decorador sólo puede recibir un parámetro, y aquí tiene self más *args. Funciona aparentemente porque *args podría estar vacío. Pero entonces self no representará al objeto de la clase, sino a la función que va a ser decorada.
  2. Dentro del decorador hay una llamada a function(*args), la cual dará un error porque function no está declarada, ni es un parámetro ni nada.

No obstante estos errores no se descubrirán hasta que no intente ejecutarse una función decorada con ese decorador. Ya que en este caso el código es parte de un programa que usa Tk, y que las funciones están asociadas a botones, el error no se descubre hasta que se pulse uno de esos botones. Por eso el programa "arranca sin problemas" y puede pensarse que está correcto, pero fallará al pulsar un botón.

La sintaxis correcta para el decorador sería:

def database_managment(function):
    def connect_database(obj, *args):
        self.connection = sqlite3.connect('database/Bank-Data')
        self.cursor = connection.cursor()
        function(obj, *args)
        self.connection.commit()
        self.connection.close()

    return connect_database

Como vemos en esta sintaxis no hay referencias a self, por lo que este decorador puede escribirse dentro o fuera de la clase, de forma indistinta. Yo aconsejo fuera, porque tener dentro de una clase un método que no use self sería confuso.

Ampliación (respondiendo a la ampliación de la pregunta)

El usuario publica su código, en el que intenta usar de nuevo el decorador como un método de una clase, con esta sintaxis:

def database_managment(self, *args):
    def connect_database(*args):
        self.connection = sqlite3.connect('database/Bank-Data')
        self.cursor = connection.cursor()
        function(*args)
        self.connection.commit()
        self.connection.close()

    return connect_database

y dice que le "funciona".

Lo cierto es que ese código no puede funcionar por dos razones:

  1. El decorador tiene más parámetros de los que debería. Un decorador sólo puede recibir un parámetro, y aquí tiene self más *args. Funciona aparentemente porque *args podría estar vacío. Pero entonces self no representará al objeto de la clase, sino a la función que va a ser decorada.
  2. Dentro del decorador hay una llamada a function(*args), la cual dará un error porque function no está declarada, ni es un parámetro ni nada.

No obstante estos errores no se descubrirán hasta que no intente ejecutarse una función decorada con ese decorador. Ya que en este caso el código es parte de un programa que usa Tk, y que las funciones están asociadas a botones, el error no se descubre hasta que se pulse uno de esos botones. Por eso el programa "arranca sin problemas" y puede pensarse que está correcto, pero fallará al pulsar un botón.

La sintaxis correcta para el decorador sería:

def database_managment(function):
    def connect_database(obj, *args):
        self.connection = sqlite3.connect('database/Bank-Data')
        self.cursor = connection.cursor()
        function(obj, *args)
        self.connection.commit()
        self.connection.close()

    return connect_database

Como vemos en esta sintaxis no hay referencias a self, por lo que este decorador puede escribirse dentro o fuera de la clase, de forma indistinta. Yo aconsejo fuera, porque tener dentro de una clase un método que no use self sería confuso.

se añadieron 74 caracteres en el cuerpo
Origen Enlace
abulafia
  • 58.7k
  • 3
  • 54
  • 89

Esta forma es algo más limpia, pues no tienes dentro de una clase un método que no usarecibe self como primer parámetro (lo que puede resultar chocante a quien lo lea), e incluso generar errores en linters y ademáseditores). Además de ese modo el decorador puedes usarlo en otras clases.

Esta forma es algo más limpia, pues no tienes dentro de una clase un método que no usa self (lo que puede resultar chocante a quien lo lea), y además de ese modo el decorador puedes usarlo en otras clases.

Esta forma es algo más limpia, pues no tienes dentro de una clase un método que no recibe self como primer parámetro (lo que puede resultar chocante a quien lo lea, e incluso generar errores en linters y editores). Además de ese modo el decorador puedes usarlo en otras clases.

Origen Enlace
abulafia
  • 58.7k
  • 3
  • 54
  • 89

Aunque en el título de la pregunta mencionas "dentro de una clase", el código que pones como parte de la pregunta no muestra ninguna clase. Por otro lado ese código es en general incorrecto porque el decorador que has escrito no retorna la función interna (y hay un error de sintaxis en la definición de esa función porque faltan los paréntesis).

Supongamos que lo que querías hacer era algo como esto:

class A:
  def __init__(self, name):
    self.name = name
    
  def decorador(self, function):
      def funcion_interna():
          print("Decorada empieza")
          function()
          print("Decorada termina")
      return funcion_interna

  @decorador
  def funcion(self):
       print("saludo", self.name)

La intención es por tanto definir un decorador como método de una clase, y usar ese decorador para decorar otros métodos de la misma clase.

Lo anterior no funciona, como has descubierto. Ya que el método decorador() espera dos parámetros (self, y function), pero luego lo usas poniendo @decorador, asumirá que la función que aparece detrás será el primer argumento a pasarle al decorador (tomará el lugar de self) y le faltará entonces un segundo parámetro function.

Si quitamos sin más el self de la declaración del decorador y lo dejamos así:

class A:
  def __init__(self, name):
    self.name = name

  def decorador(function):
      def funcion_interna():
          print("Decorada empieza")
          function()
          print("Decorada termina")
      return funcion_interna

  @decorador
  def funcion(self):
       print("saludo", self.name)

Tampoco estaría bien. Aunque esta vez el código anterior no produce errores al ser ejecutado, ha definido una clase que no se comportará correctamente. Si probamos a instanciar esta clase y llamar a su método funcion lo vemos:

>>> a = A("Pepe")
>>> a.funcion()
TypeError
----> 1 a.funcion()
TypeError: funcion_interna() takes 0 positional arguments but 1 was given

¿Qué pasó? Que al hacer a.funcion() en realidad se invoca el método pasándole como primer parámetro (en self) la referencia al objeto. Es decir, se está ejecutando A.funcion(a). Esto a su vez, ya que la función ha sido decorada, se traduce en que lo que realmente se ejecutará será funcion_interna(a), es decir, a la función interna se le está pasando como parámetro el objeto sobre el que actúa. Pero la declaración de la función interna no espera tal parámetro, y de ahí el error.

Hay que añadir a funcion_interna() un primer parámetro que será el objeto sobre el que actúa. Esta función llama internamente a funcion(), y deberá pasarle también ese objeto pues funcion() lo espera (lo recibirá en self). Es decir el código final que sí funcionará es:

class A:
  def __init__(self, name):
    self.name = name
    
  def decorador(function):
      def funcion_interna(obj):
          print("Decorada empieza")
          function(obj)
          print("Decorada termina")
      return funcion_interna

  @decorador
  def funcion(self):
       print("saludo", self.name)

>>> a = A("Pepe")
>>> a.funcion()
Decorada empieza
saludo Pepe
Decorada termina

Nota final

¿Por qué quieres el decorador dentro de la clase? Funcionará exactamente igual si está fuera:

def decorador(function):
    def funcion_interna(obj):
          print("Decorada empieza")
          function(obj)
          print("Decorada termina")
    return funcion_interna

class A:
  def __init__(self, name):
    self.name = name
    
  @decorador
  def funcion(self):
       print("saludo", self.name)

Esta forma es algo más limpia, pues no tienes dentro de una clase un método que no usa self (lo que puede resultar chocante a quien lo lea), y además de ese modo el decorador puedes usarlo en otras clases.