Para gestionar unas consultas de datos estoy creando una clase genérica del tipo:
class MisDatos(object):
_instance = None
@staticmethod
def get_instance(func):
if MisDatos._instance is None:
MisDatos._instance = conexion()
return MisDatos._instance
@staticmethod
def anyade_valores(var):
mi_instancia = MisDatos.get_instance()
return mi_instancia.get(var)
@staticmethod
def elimina_valores(var):
mi_instancia = MisDatos.get_instance()
return mi_instancia.delete(var)
Se observa la estructura con un singleton, para estar siempre utilizando la misma conexión.
Dado que cada método llama siempre en primer lugar a get_instance()
(es decir, la línea mi_instancia = MisDatos.get_instance()
), pensé que lo mejor sería hacer un decorator para automatizar esta parte:
class MisDatos(object):
_instance = None
@staticmethod
def get_instance(func):
def check_instance():
if MisDatos._instance is None:
MisDatos._instance = conexion()
return func()
return check_instance
@staticmethod
@get_instance
def anyade_valores(var):
return MisDatos._instance.get(var)
Sin embargo, actualmente me está diciendo:
TypeError: 'staticmethod' object is not callable
¿Cómo podría corregir esta estructura? ¿Es esta una manera correcta de realizar lo que quiero o, por contra, estoy complicándome más de la cuenta?
get_instance
) es declarado como estático.staticmethod
retorna un descriptor y no una función, lo que causa que al decorar con@get_instance
falle al no ser "callable". Hay varias opciones si quieres usar un decorador, entre ellas declararlo fuera de la clase, crear una clase que agrupe estos métodos auxiliares, etc Por ciertocheck_instance()
necesita recibir el parámetrovar
.get_instance()
debería estar dentro de la clase, pues si no no puedo accedir aMisDatos._instance
desde fuera. Deberé explorar la clase que agrupe los auxiliares, aunque parece que se complicaría demasiado.MisDatos._instance
al ser un atributo de clase y no de instancia es accesible desde fuera de la clase siempre que puedas hacer referencia aMisDatos
, por ejemplo si sacas el decorador de la clase pero lo dejas en el mismo módulo no debe haber ningún problema. Si no usas@staticmethod
con el decorador también funcionaría tal y como lo tienes ahora. Incluso se puede hacer que el propio decorador acepteMisDatos
como argumento. En cualquier caso no olvides hacer que el wrapper recibavar
para que puede retornarfunc(var)
como comentaba antes.