search

AJO2DAM-2122 / DataAccess-AJO2122

0 stars 0 forks source link

Final activity assesment #11

Open jrgs opened 2 years ago

jrgs commented 2 years ago
  1. Existe un comportamiento incorrecto en la aplicación. Cuando se devuelve un libro que está reservado por otra persona, se informa -correctamente- de que existe una reserva, pero se borra la reserva -incorrecto-. La reserva se debe borrar en el momento en que viene el usuario a realizar el alquiler.
  2. Intento devolver un libro y no me deja. Esto pasa cuando un usuario tiene varios libros. Comparas el isbn con el de todos los libros que tiene, como siempre va a haber uno diferente, te dice que no puedes devolverlo. (LibraryController.onAcceptClicked, línea 552).
  3. En LibraryModel.isBookReservedBy, pasas como parámetro el libro, pero en vez de ir a mirar su propiedad reservations, haces una nueva Query: es totalmente innecesaria (repetitiva).
AJO2DAM-2122 commented 2 years ago

Muy buenas @jrgs!!

Muchas gracias por echarle un vistazo. Respecto al fallo número 1, lo siento mucho ya que ha sido un error mío al diseñarlo (yo entendí que se avisaba al devolverlo para que el bibliotecario lo tuviese en cuenta pero me equivoqué).

Respecto al fallo número 2, si que me parece bastante más grave y lo lamento mucho, no haber testeado más en profundidad para haberlo localizado y atajado correctamente. ¿De qué manera se podría solucionar ese error? Aunque ahora supongo que ya no puedo hacer nada, pero al menos para tener en cuenta como podría hacerlo bien.

Un saludo.

jrgs commented 2 years ago

Pues para el fallo número 2, tienes que corregir el bucle que hay en la línea 550. En ese bucle iteras con todos los alquileres, de manera que si uno de los alquileres no coincide con el isbn, terminas dando error. Debería ser lo contrario, iteras por todos los alquileres, de manera que si uno coincide, terminas dando OK, y si ninguno coincide, terminas dando error.

Un saludo

AJO2DAM-2122 commented 2 years ago

Creo que ya he pensado como se podría solucionar. Efectivamente, como tú me indicas, he planteado al revés la lógica de la condición Una duda: ¿en el examen tendría la opción de modificarlo, o ya tengo que dejarlo así a partir de ahora?

Gracias por la aclaración, un saludo!

jrgs commented 2 years ago

Hola Aaron, perdón por tardar tanto en la respuesta...Puedes modificar lo que quieras, y arreglarlo, claro, pero el proyecto ya está calificado, eso sí.

Un saludo