Cita:
Iniciado por Xerelo 
Está bien, sólo te pondría dos pegas
El método isUsernameOrPasswordValid.
Deberías haber usado un regex, no ese tochazo de condición. Fíjate en como compruebas el mail, ahí se está usando un regex.
DaoUser2
Efectivamente es más profesional, pero no como tú la has hecho. La forma "profesional" sería haber creado un DAO con sus getters y setters (un bean), y un DTO que es el que se encarga de ejecutar las consultas. Tú has mezclado los dos en uno.
Totalmente de acuerdo, pero cabe clarar que DAO son las operaciones, Y El DTO es la que trae métodos get y set