Refactor gestione password

sommovir / cineteca-k

0 stars 0 forks source link

Open sommovir opened 2 years ago

sommovir commented 2 years ago

La gestione per il controllo password dovrebbe essere refactorata. La classe ManagementPassword ha vari problemi che verranno elencati tra le task

[x] La classe dovrebbe esporre semplicemente un metodo pubblico per il controllo della password
[x] La classe si può fare singleton , non c'è bisogno di costruirla.
[x] La signature del metodo di validazione deve dichiarare di lanciare eccezioni ( quelle del task #24)
[x] La classe non ha bisogno di salvare negli attributi le password, deve solo verificarle

GhChancellor commented 2 years ago

Non capisco cosa vuoi che faccia. Per la singletone ok ma il resto non capisco

sommovir commented 2 years ago

allora sul primo punto, avevo sbagliato, non volevo dire metodo "statico" ma pubblico . ( ho corretto ).

sommovir commented 2 years ago

ti faccio un esempio di uso finale di una classe come questa:

sulla GUI:

String username = this.jTextField.. .getText(); char [] password = ...

try{

ManagementPassword.getInstance.login(username,password);

se non ci sono eccezioni arriva qua e può chiudere il login e aprire la vera interfaccia } catch(...){

se va nel catch il metodo login ha lanciato eccezioni stampare in GUI il messaggio delle eccezioni. }

GhChancellor commented 2 years ago

questa card è la fusione delle card #28 #30 non sarebbe il caso di fonderle in master? se si come?

sommovir commented 2 years ago

si hai ragione, in effetti facciamo così, te la segno sul branch delle password e poi mergiamo direttamente