laimejesus / aterrizar.com Goto Github PK

Tienen clases que no se considerarían de "dominio" en el package de dominio.

El BuscadorDeVuelos tiene toda la pinta de un servicio.

Por ejemplo con este orden:

La forma que lo usan en el test es: ordenPorDuracion.porMenorOrden
Pero que pasa si se hace dos veces esa linea:
ordenPorDuracion.porMenorOrden
ordenPorDuracion.porMenorOrden

La query que se genera quedaria: vuelos.duracionDeVuelo asp asc
Y eso no esta bueno.

Lo que se podría hacer es tener una tipo de orden, ASC y DESC. Entonces cuando vos le decis ordenPorDuracion.porMenorOrden se setea el tipo ASC y cuando le mandan el mensaje getOrdenadoPor ahi devuelve la query + el tipo

Se entiende?

Estaria bueno que el usuario tenga esa responsabilidad.

Tienen tests donde esperan una excepción y en varios casos manejan las excepciones sin hacer nada "importante". por ejemplo:

JUnit tiene una forma de hacer un assert esperando una excepción sin manejarla.

Ahora tenes eso:

Fijate que esta muy atado a la cantidad de visibilidades que tenes... sumado a la cantidad de código repetido...

se podría simplificar asi:

Hay varios métodos muuuy parecidos:

getSession, persistir, actualizar, borrar, campos, etc.

Porque tienen cosas deprecadas?

Ahora tenes esto:

Y podría quedar asi:

Y se usaria asi:

En OrdenPorEscalas, OrdenPorDuracion y en OrdenPorCostoDeVuelo tienen el mismo código:

Eso lo habíamos hablado en clase. La idea es patearle la responsabilidad de asignar los ids al motor de la base de datos. La solución que hicieron tiene un problema gravísimo, por ejemplo, el contador de ids es volátil y vive en memoria, cada vez que instancias el servicio el contador empieza en 1. Si los datos fuesen persistentes a lo largo del tiempo (no como en sus test que se borran) tendrían un conflicto zarpado con los ids.

Usan el repositorio teniendo un método en el servicio que busca un usuario por nickname.

Tienen este código:

Las validaciones están bien, pero no lo que no esta bien es la forma en la que se realizan.
Si tenes que validar que un vuelo pertenezca a una aerolínea, no podría ser una responsabilidad de la aerolínea?

Lo mismo va para los vuelos-tramos, tramos-asientos.

Podrias tener una validación en cascada.. por ejemplo:

(Aunque no compila, pero podría ser una buena forma de delegar las validaciones a los objeto de dominio )

Por mapeos que están mal, hibernate no levanta y ningún test corre.

Tienen que crear un script con la creación del schema de la base de datos. Porque? Porque si quiero correr sus test, necesito tener las tablas creadas y al no tener el script tengo que ver que tablas usan en su proyecto y con que columnas, etc. Entonces no esta bueno eso.

Creen el script y lo adjuntan en el proyecto. Un buen lugar para el archivo es dentro del directorio "src/main/resources", que es el lugar por convención de maven para adjuntar recursos externos al código. También dentro de el pueden categorizar los archivos, por ejemplo "src/main/resources/db/createSchema.sql"

Todos los tests del RecorderServiceTest fallan.

En la consulta de viajeA te traes todos los vuelos en los que reservaste algún asiento y recorres los tramos a mano, eso se podría resolver con una sola query... y que te devuelva un true o false o que te devuelva el tramo y si esta... es que viajó a ese lugar y si no... no!

Esto es muy grave. No esta bueno que entreguen algo que no compila.