Closed AlexGonRo closed 7 years ago
Original comment by Alejandro González Rogel (Bitbucket: agr00095, GitHub: Unknown):
Sobre el commit final 3259d62
Se ha realizado una gran y última refactorización sobre el código. Muchos de los paquetes han sido renombrados, por lo que se ve un gran número de lineas borradas y añadidas.
El flujo del programa no se ha cambiado en absoluto.
Así mismo, esta es una tarea en la que llevaba ya un tiempo trabajando, por lo que los cambios que se han hecho (cambio de nombre de paquetes, aparición de alguna nueva clase o método, etc) ya estába reflejado en los anexos que he ido subiendo y no necesitan ser cambiados.
Se han eliminado absolutamente todos los "warnings" de Scalastyle.
Esta refactorización también ha incluido modificar la parte de ayuda de la interfaz, a la que se le han añadido algunas nuevas secciones y modificado ligeramente otras.
Original comment by Alejandro González Rogel (Bitbucket: agr00095, GitHub: Unknown):
Sobre el issue ee8def5
Se ha realizado una primera iteración revisando el código.
Aunque se reflejan muchas lineas añadidas/cambiadas/modificadas, la lógica de los algoritmos no ha sido cambiada.
Se han realizado cambios de cara a:
Completar la documentación del Scaladoc.
Completar la documentación de la ayuda de la interfaz.
Corregir el código de acuerdo a las medidas estáticas de calidad comentadas en el issue #22. Además, se añade al repositorio el fichero .xml que contiene las reglas.
Renombrado de algunas clases/métodos, aunque no descarto más cambios futuros.
Reestructuración de algunos métodos o variables (variables locales que pasan a ser de clase, métodos cuya función ha sido dividida en varios métodos más pequeños, etc.). En ningún caso se modifica la funcionalidad que ya tenían estos métodos.
Se ha eliminado la carpeta “Pruebas”, que se encontraba en el proyecto. Contenía un par de clases usadas durante la primera iteración a modo de prueba, no tenían que ver con el proyecto, y la relacionada con Java ni siquiera fue implementada completamente.
Esta tarea fue abierta para continuar trabajando con el proyecto hasta que se debatiese como enfocar el problema de rendimiento de los algoritmos. Ahora que tengo trabajo sobre LSHIS y DemoIS, esta tarea queda con la más baja prioridad, con perspectivas de seguir trabajando en el futuro si hay tiempo.
Originally reported by: Alejandro González Rogel (Bitbucket: agr00095, GitHub: Unknown)
Se va a realizar un análisis de todo el código generado durante el TFG con los siguientes fines:
Correcta documentación de todas las clases, métodos y variables. Esto conlleva la correcta generación del Scaladoc
Correcto cumplimiento de las medidas estáticas de calidad propuestas en #22
Renombramiento de clases o métodos si se viese necesario
La motivación viene dada por las labores de implementación llevadas a cabo sobre el código durante los últimos hitos han afectado bastante al código (en particular el hito referente a la interfaz #28, que obligó a modificar muchas otras clases)