我有一些我经常在网站上看到的代码示例,我想改进它,并希望得到一些帮助。通常,我会在page_load方法中看到5-10个嵌套的if语句,旨在消除无效的用户输入,但这看起来很难看,很难读取和维护。如何整理太多的if语句以提高可读性
您会如何建议清理以下代码示例?我试图消除的主要事情是嵌套的if语句。
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);
}
正如你所看到的,这很快就会变得非常混乱!在这种情况下,我应该遵循哪些模式或做法?
惯于到达那种情况下,因为有一个用户==空的情况下,先生 – lemon 2009-10-30 11:35:00
案件用户== null已被选中。报表写入的顺序非常重要。你必须从检查对象为空,然后非法值等开始。 – 2009-10-30 11:35:54
我想我喜欢这种方法,感谢您的建议。 – NickGPS 2009-10-30 11:35:56