Skip to content

Alvaro Ordóñez Romero / DAE-aha00026-aor00039

Go to a project
Tags give the ability to mark specific points in history as being important
  • practica3-security   Hemos intentado hacerla funcionar pero seguimos con el error de NO AUTORIZADO.
    457fc1d9 · Modificaciones Security ·

    Nota grupal: 6.5

  • practica3-tests   Tag con todos los tests funcionando y el correspondiente PDF adjunto.
    21c72f28 · Funcionamiento de todos los tests ·

    PR3DAE.pdf

  • practica2   Finalizacion de practica 2
    dd3f1821 · Changes by Álvaro Ordoñez colabora en el anterior commit ·

    Nota grupal: 6.2

  • Practica1   Práctica1-aha00026-aor00039
    aa68f991 · Adición del resumen de la práctica 1 ·

    En general, código bien estructurado. No obstante, tener en cuenta los siguientes comentarios:

    División de carpetas entre entidades y Servicio (tenéis solamente una donde alojáis todo, tanto entidades como el servicio)

    ----Entidades----

    Reserva: No tiene sentido crear un atributo de SistemaCarPooling en la entidad reserva, hace que el código no esté claro. Si en algún momento necesitáis añadir un elmento al vector de reservas, sería mejor que le paséis el vector por parámetro.

    En el método esViajero(), debéis usar equals() para comparar cadenas en lugar de ==.

    SistemaCarPooling: Los métodos setViajes y setUsuarios podrían no ser necesarios si no planeas cambiar estas listas después de su creación. Considerar eliminarlos si no se utilizan.

    Usuario: Idem con el atributo sistemaCarPooling, no tiene lógica que dentro de un usuario tengais toda la información del sistema.

    *----Test----

    ReservaTest: Redundancia del método setUp(): Este método al estar etiquetado como @Before, se ejecuta antes de cada prueba. Luego no sería necesario llamarlo en cada test. Debéis eliminar la llamada setUp() al inicio de cada método de prueba.

    Cambiar el uso de Date por LocalDate.

    Intentar asignar nombres más específicos a las funciones o incluir comentarios que escarifiquen el código.

    UsuarioTest: En el método ofrecerViaje() sería beneficioso incluir comentarios o mensajes de aserción más descriptivos en las pruebas para proporcionar información adicional cuando una prueba falla. Por ejemplo, en el método testOfrecerViaje, sería útil saber por qué el pasajero no puede ofrecer un viaje (qué aserción falla).

    ViajeTest: El método testConstructor() es un buen ejemplo de un método @Before utilizado para inicializar el objeto Viaje antes de cada prueba. Sin embargo, el nombre podría ser más descriptivo, por ejemplo, setUp().

    SistemaCarPoolingTest: En TestLoginSuccess, en lugar de agregar usuarios directamente al vector, podríais implementar métodos en el sistema que sea agregarusuario.

    Por ejemplo:

    assertTrue("El viaje no se agregó correctamente", sistema.getViajes().contains(viaje));

    Conclusión: Debéis tener en cuenta estos comentarios para futuras prácticas.

    Nota de práctica 1: 8.0