venganzaalchocolate / backendlistaprecios Goto Github PK

Hay varías cosas que se pueden hacer para mejorar el código

1. En primer, deberían de comentar qué hace cada función. Aunque tienen un nombre de funciona autodescriptivos, siempre viene bien que se indique qué es lo que hace cada función. Es casi más recomendable comentar lo que hace la función, que lo que hace una línea en concreto. El comentario de la función indica que hace, para qué sirven los datos de entrada, y los datos de salida, y si es un método, indica como cambia el sistema.
2. Teneís varios ficheros donde insertáis funciones solamente. y las utilizáis en otros ficheros. Para que quede más límpio el código deberíais de utilizar clases y definir roles para cada clase. Mirando el código diría que lo mejor es crear una clase: ApiService.py que es la clase que realiza toda la acción relativa a las acciones con la API. Yo quitaría lo que tenéis de renovador.py y lo incluiría dentro de ApiService.py, ya que realmente todo está dentro del mismo concepto. Lo que llamas parsearDatos.py sería claramente una Entidad. En ese caso, divide el código en dos, uno para la entidad de Campañas y otro para la entidad de Productos. Crea un método que sea map o fill para pasarle un json y hacer el mapeo entre lo que llega el json y lo que se espera la entidad.
3. En App.py puedes dejar la interacción con la API, pero utilizando la clase ApiService y las entidades Campanias y Productos, así el código estaría más limpio y no te volverías loco buscando donde están definidas las funciones.