mayteEspi/sanitasRefactor

Pasos para ejecutar proyecto

1. Si se va abrir con un ide (sts...) Instalar lombok en el ide https://projectlombok.org/download

2. El método principal usa un CompletionService para lanzar asincronamente ciertos calculos (simulaciones de poliza frequencia). Todo el código que gestiona esos Callables habria que extraerlo a parte, teniendo una clase que se encarge de esa gestión de Callables, llamarlos y devolver el resultado. De esta manera, aparte de dejar el código de la clase RealizarSimulacion más limpio, seria facilmente testeable (ahora mismo con las llamadas asincronas no es posible implementar un tests que no falle una vez si una vez no...).

3. Ha sido necesario añadir la dependencia de la commons-lang porque las clases del jar entregado sino no funcionaba:

commons-lang commons-lang 2.6

4. Se puede ver que he sobrecargado el método princial de realizar simulacion, esto sería un refactor que haría, pero como no conozco los posibles proyectos que llamarían a realizarSimulacion por ello lo sobrecargo, con la idea de ir haciendo un refactor poco a poco.

5. También en realizar simulación y si hubiera tenido tiempo, habría sacado a un service la propia simulacion, es decir el método simular.

6. Quitaría todos los break, estoy en contra del uso del break para interrumpir un bucle...

7. Usaría java 8, no java 7, así podría usar Streams, CompletableFuture que vendría estupendamente para este caso.

8. Crearía una clase de constantes y pondría en ella todas esas constantes que hay 'a pelo'

9. Yo he metido lombok, es increiblemente bueno para crear builder y para no tener que crear los getter y setter, además podriamos meter los logs usando la anotación de lombok @Slf4j.

10. He actualizado la version de jacobo ya que daba problemas la incluida en el pom.

11. Añadiría mas test de los service realizados como veís no hay cobertura 100%, no he tenido tiempo suficiente.

12. Realizaría paquetería por funcionalidad y siguiendo el patrón de Single Responsability las clases creadas tendrían solo una exclusiva funcionalidad, es decir que creaía una clase en cada paquetería por funcionalidad (beneficiarios, contratacion. ..) para que todo este mas desacoplado.

13. Siguiendo lo anteriormente descrito, refactorizaría si o si soporte(paquetería) por funcionalidad y tipo de clase desacoplaría las clases incluidas en esta paquetería.