Laura García Calderón / DAELaura2526

• En el servicio de incidencias se pasa un parámetro autorId varias veces. Primero se declara de tipo Long en lugar de long. Long es una clase wrapper del tipo básico long. Usar Long es mucho más ineficiente que su tipo básico correspondiente. Pero aparte, es preferible evitar el uso de ids en la interfaz. Si tenemos las clases Usuario, Incidencia etc. no tiene sentido hacer las llamadas extrayendo su ids y obligando a buscar los objetos correspondientes dentro de las operaciones (findById). Además requerir un objeto de la clase Usuario garantiza que se ha hecho un login previo con éxito.

• adminCambiarEstado() debería requerir un objeto usuario y comprobar que es el administrador, si no no hay ninguna manera de evitar que la llame un usuario arbitrario.

• Se usan excepciones genéricas de Java en lugar de excepciones específicas del dominio que aporten información más concretas y faciliten la legibilidad del código. Por ejemplo, lanzar IllegalStateException porque el estado de la incidencia no sea PENDIENTE.

• Los repositorios no se han implementado.

• JAMAS se sube la carpeta target al repositorio. Eso es algo básico del uso de un SCV. Haz un borrado duro de esa carpeta, desde el principio (busca en la documentación cómo hacerlo).

• No hay tests. Eso era un requerimiento básico de la entrega. Tampoco se ha creado un tag para la entrega.

Valoración: 4