Ciao a tutti,
sto studiando da qualche settimana il fantastico libro di Steve McConnell intitolato "Ingegneria del Codice". Nell'applicare le sue teorie al mio codice mi sono trovato di fronte ad un problema e cioè riguardante la buona progettazione di classi e routine(metodi).
Nel libro l'autore consiglia di frammentare e ridurre le routine hai minimi termini questo per diminuire la complessità del codice e quindi per una serie di vataggi. Applicando queste teorie al Web ho creato una classe che effettua l'autenticazione tramite una stored procedure del Db.
Prima avevo un metodo che effettuava una chiamata alla stored procedure che verificava l'esistenza di username e password nel db e tramite join sulla tabella "ruoli" restituiva il nome del ruolo come risultato del metodo. Ora questo metodo in realtà effettua due funzioni: la prima verifica la corrispondenza delle credenziali e la seconda trova il corrispondente ruolo. Stando alla teoria sopracitata dovrei fare due metodi: TrovaCorrispondenza(user, pass):bool e l'altro IdentificaRuolo(user, pass):string dove il secondo metodo include il primo. Il problema è che nel metodo consigliato io effettuo due chiamate al database mentre con il metodo vecchio ne effettuavo una sola.
Nel web questo non potrebbe essere uno spreco di risorse?
In un intranet dove l'accesso è limitato a un numero calcolato di utenti potrei non fare questa considerazione?

---

Modificato da lucas80 il 22 febbraio 2006 10.51 -

---

Modificato da lucas80 il 22 febbraio 2006 10.58 -

Questo è il sistema che usavo anche io e sembra buono, comunque collaudato, ma i dubbi me li ha messi in testa la lettura di questo libro che suggerisce che ogni metodo deve fare una cosa mentre in questo caso ne fa già due. Una è quella di verificare l'esistenza dell'utente e l'altra e di verificarne il ruolo. Sembra una pippa mentale , in realtà fare un join sulla tabella utenti e quella ruoli è il massimo: riempio la classe utente e sono apposto.Ma metti caso che hai le informazioni di login su un database e i ruoli in un file xml. In questo caso dovresti cambiare il tuo metodo "validate" quindi rivedere tutto il codice del metodo e della query al db. Nell'altro caso hai un metodo che è validateUser che controlla l'esistenza nel db e un metodo getRuolo che controlla l'esistenza nell'xml. in uno pseudocodice tipo questo:

metodo ValidaUtente(username, password)
controllo che esista nel db
se esiste restituisco il suo id

metodo TrovaRuolo(username, password)
uso ValidateUtente(username, password)
faccio una query sul xml in base all'id che mi è stato restituito

in questo caso la mia prima funzione fa solo una cosa specifica e se un giorno devo cambiare qualche fonte non cambio la logica di un metodo o del database ma modifico solo un metodo. Questo è il vantaggio indiscusso a livello di manutenzione e di "Riduzione della complessità"
ma il problema è che nel caso specifico io cosi faccio due query al db quando potrei farne una.. Che pippa...

---

Modificato da lucas80 il 22 febbraio 2006 15.32 -

Grazie Marco ho capito perfettamente ed è molto simile a ciò che faccio io di solito, ma questo libro mi ha scombussolato. Potremmo andare avanti fino all'infinito ad esempio il metodo ValidateUser che tu implementi fa due cose: verifica e nel caso restiuisce una select giusto?
Ecco secondo lui questo è sbagliato e credo di aver capito anche i motivi vantaggiosi di tale affermazione ma il problema è che diventa troppo meticolo fare un lavoro del genere..Comunque ti consiglio la lettura di questo libro. In pratica per uno smanettone quale sono io è stata una bella scoperta. Da quando seguo i consigli il codice è diventato molto limpido e semplice da gestire anche se ancora ho problemi di questo tipo(pippe come quella che ho postato).

Le regole di best pratiQue sono molto interessanti.
Ma io non le prenderei come oro colato.
Insomma non c'è nessun manuale finale che contiene tutte le regole, costrutti e comportamenti da assumere per programmare al meglio.
Ci sono delle regole di buon senso, di buona "calligrafia" se vogliamo ...ma non delle regole da seguire al 100%.
Ricordo il mio prof di c che spaccava la $%"%$" dicendo che una funzione non doveva avere più di 7 righe.
Che dovevi utilizzare al 50% i puntatori e che i cicli negativi sono più performanti.
Valanga di cavolate :P

Ps: visione mia della cosa.