Ho un esempio di codice che vedo spesso nei siti web che vorrei migliorare e che apprezzerebbe un po 'di aiuto. Spesso vedo 5-10 istruzioni if annidate in un metodo page_load che mirano ad eliminare l'input dell'utente non valido, ma questo sembra brutto ed è difficile da leggere e mantenere.Come riordinare troppe istruzioni per la leggibilità
Come suggeriresti di pulire il seguente esempio di codice? La cosa principale che sto cercando di eliminare sono le dichiarazioni annidate.
string userid = Request.QueryString["userid"];
if (userid != ""){
user = new user(userid);
if (user != null){
if (user.hasAccess){
//etc.
}
else{
denyAccess(INVALID_ACCESS);
}
}
else{
denyAccess(INVALID_USER);
}
}
else{
denyAccess(INVALID_PARAMETER);
}
Come potete vedere, questo diventa abbastanza disordinato molto rapidamente! Ci sono schemi o pratiche che dovrei seguire in questo caso?
Wont raggiungere quel caso poiché c'è un caso utente == null sopra di esso sir – lemon
Il caso dell'utente == null è già controllato. L'ordine di cui sono scritte le affermazioni è significativo. Devi iniziare controllando gli oggetti per null, quindi valori illegali ecc. Ecc. –
Penso che questo approccio mi piaccia, grazie per il consiglio. – NickGPS