RafaelAybar / qa-empleo

En este proyecto se medirá el índice de fiabilidad de una empresa, así como la calidad de las ofertas que publica
GNU Lesser General Public License v3.0
17 stars 9 forks source link

Refactorizacion codigo #12

Closed vmjelicic closed 3 years ago

vmjelicic commented 3 years ago

He agregado el análisis sintáctico de las entradas y ordenado un poco el código del script.

Closes #5

vmjelicic commented 3 years ago

¿Qué ventajas tiene usar una clase? Es decir, teniendo en cuenta que el script es poco complejo, ¿que aporta POO aquí?

Pensaba en que podría ser útil para agregar alguna funcionalidad más adelante, sin embargo, si te molesta puedo dejarlo como funciones simples.

RafaelAybar commented 3 years ago

¿Qué ventajas tiene usar una clase? Es decir, teniendo en cuenta que el script es poco complejo, ¿que aporta POO aquí?

Pensaba en que podría ser útil para agregar alguna funcionalidad más adelante, sin embargo, si te molesta puedo dejarlo como funciones simples.

No es molestia, es que no entiendo qué problemática resolverá usar POO aquí, por que no sysadmin, no programador. ¿De cara a usar Flask, el manejo de clases aportaría algo? ¿Deberían de organizarse un fichero por clase como en Java?

vmjelicic commented 3 years ago

¿Qué ventajas tiene usar una clase? Es decir, teniendo en cuenta que el script es poco complejo, ¿que aporta POO aquí?

Pensaba en que podría ser útil para agregar alguna funcionalidad más adelante, sin embargo, si te molesta puedo dejarlo como funciones simples.

No es molestia, es que no entiendo qué problemática resolverá usar POO aquí, por que no sysadmin, no programador. ¿De cara a usar Flask, el manejo de clases aportaría algo? ¿Deberían de organizarse un fichero por clase como en Java?

Lo hice pensando en que podría conectar bien con la issue 3 de representación de datos, para generar los gráficos a partir de un método. Tienes razón en lo de dividir clases que no guarden directa relación, por lo que el análisis sintáctico lo separé en un fichero diferente.

RafaelAybar commented 3 years ago

@vmyelicich Hay que solucionar el conflicto de ramas, con esto tengo claras algunas cosillas. Gracias!

vmjelicic commented 3 years ago

@vmyelicich Hay que solucionar el conflicto de ramas, con esto tengo claras algunas cosillas. Gracias!

Listo, conflicto solucionado!