MZFQ2425 / DataAccessMZFQ2425

Repository to upload Data Access exercises - Martha Fernández Quispe
MIT License
0 stars 0 forks source link

First Period Final Activity first assessment #1

Open jrgs opened 5 days ago

jrgs commented 5 days ago

1.- No se usa JDK 17. 2.- No se detecta la ausencia de servidor postgre 3.- Hay llamadas Hibernate en los controladores – no respeta MVC 4.- No se usa MD5 para las contraseñas. Se usa plain_password (incorrecto) 5.- No se valida el formato del email 6.- No se confiman las actualizaciones de datos 7.- No se confirman los cambios de contraseña

MZFQ2425 commented 4 days ago

Hola buenas noches Jose,

Te contesto por puntos: 1 - El JDK 17 lo he utilizado por incompatibilidades con JavaFX, entiendo que no es lo ideal, pero a medida que he ido creando el proyecto me he encontrado con muchos stoppers y he decidido tirar de esa versión, si en PC de clase no lo tiene no tengo problema en adaptarme.

Puntos que no acabo de entender para poder corregirlos: 3 - Desde donde deberian hacerse las llamadas hibernate, ¿desde el mismo modele sellers puede ser? 6, 7 - Sí que se confirman, una vez se modifican se informa en el label de label_msg, ¿cuál seria la forma correcta de hacerlo?

Puntos que creo que pueden ser erróneos porque sí que los he hecho: 4 - Sí que utilizo MD5 para las contraseñas, de hecho recupero el password del login y lo paso a md5 para hacer la consulta sobre la base de datos. Y en la parte de modificacion muestro el plain_password (modificable) y al hacer un update también modifico el password encriptado. En la funcion de actualizar uso hashUtil.hashPassword(password)) para actualizar tambien el password. 5 - Sí que lo valido, uso un regex en la función checkValues() para validar que se trata de un mail valido (arrobas, extensiones .com .net etc).

Gracias, con cualquier cosa empiezo modificar el código para que quede correcto :)

jrgs commented 4 days ago
  1. Sobre el JDK 17, me gustaría saber qué incompatibildades te ha dado. ¿Usas MacOS?
  2. Las llamadas a Hibernate, si se sigue un esquema MVC (Modelo-Vista-Controlador), se deberían hacer desde una clase aparte (Modelo), esta sería llamada por el controlador y mostraría los datos en la vista.
  3. Cuando digo que no se confirman quiero decir que no se pide al usuario que confirme la operación.
  4. Lo compruebo de nuevo.
  5. Lo compruebo de nuevo.
jrgs commented 4 days ago
  1. Ok, comprobado.
  2. Comprobado, el número se valida, el email, no (prueba email@email). La función de comprobación, por cierto, se puede simplificar.
MZFQ2425 commented 4 hours ago

Buenas tardes,

Sobre el JDK, yo estaba usando el JDK 8 ya que en el pdf de Introduction JavaFX lo recomendaban, esa fue la versión que me daba errores, me permitía crear el proyecto, pero en la parte visual del SceneBuilder de los fxml me saltaban muchos errores y no podía editar, al final lo que me funcionó fue cambiar a la última versión (JDK23) e instalarme un plugin. He puesto el JDK17, dime si debería poner otra versión y veo de aplicarla.

El resto de cambios mencionados así como los requerimientos para la segunda parte del proyecto final ya están hechos, saludos !