search

SoftwareSynthesis / SoftwareEngineeringProject

Progetto per lo sviluppo del software di comunzione MyTalk relativo al corso di Ingegneria del Software (Laurea Triennale Universita' degli Studi di Padova)
0 stars 0 forks source link

[abook.controller.*] parametri obbligatori #262

Closed diegoberaldin closed 11 years ago

diegoberaldin commented 11 years ago

Non viene fatto nessun controllo nemmeno in questi controller, un simile problema era stato segnalato in #255 , per il fatto che eventuali parametri non siano presenti nella richiesta.

È mia opinione che un controllo del genere vada fatto, vuoi per evitare di fare operazioni senza senso o dannose nel caso di richieste malevole, vuoi per mettersi al riparo da eventuali errori del client. Le richieste nel client sono infatti state rimaneggiate più volte seguendo tutte le iterazioni (prima per togliere le operazioni e inoltrarle agli indirizzi delle servlet, poi per rimetterci le operazioni, ecc. ecc.) quindi gli errori possono benissimo esserci!

P.S. Mi sembra che le vecchie servlet fossero fatte meglio di questi control-cosi... :skull:

diegoberaldin commented 11 years ago

Dimenticavo: getUserMail() ovunque anche in questo package...

EDIT: Errore mio, sistemato!

marcoskivo commented 11 years ago

io non ho fatto altro che prendere le vecchie servlet e sistemare il codice.. non ho aggiunto ne tolto niente.. spero domani ci sia mene cosi sistemiamo una volta per tutte questi casini..