Tengo un ejemplo de código que veo a menudo en sitios web que me gustaría mejorar y agradecería algo de ayuda. A menudo veo 5-10 declaraciones if anidadas en un método page_load que apuntan a eliminar la entrada de usuario no válida, pero esto se ve feo y es difícil de leer y mantener.Cómo ordenar demasiadas declaraciones if para la legibilidad
¿Cómo recomendaría limpiar el siguiente ejemplo de código? Lo principal que intento eliminar son las sentencias if anidadas.
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);
}
Como puede ver, esto se pone muy sucio muy rápidamente! ¿Hay algún patrón o práctica que debería seguir en este caso?
Wont llegar ese caso ya que hay un caso de usuario == nulo en la parte superior señor – lemon
El caso de usuario == null ya está marcado. El orden en que se escriben las declaraciones es significativo. Tienes que comenzar revisando los objetos para valores nulos, luego ilegales, etc. –
Creo que me gusta este enfoque, gracias por el consejo. – NickGPS