hxtao001 2013-09-03 12:05
浏览 1454
已采纳

像这样多层if嵌套的代码如何优化

感觉自己看起来都费劲( ⊙ o ⊙ ),

[code="java"]
public String login() {
boolean loginpass = false;
Pckey pckey = null;
int validate = 0; // 密码验证标识
Integer keyright = 1;
HttpServletRequest request = ServletActionContext.getRequest();
String remoteIp = request.getRemoteAddr();
String localIp = request.getLocalAddr();
String passFlag = request.getParameter("validate");
if (StringUtils.isNotEmpty(passFlag)) {
validate = Integer.parseInt(passFlag);
}
if (StringUtils.isNotEmpty(username)) {
keyright = Integer.parseInt(userright);
pckey = pckeyService.findKey(username, keyright);
}
if (pckey != null) {
boolean locked = pckeyService.isLocked(pckey);
if (!locked) { // 没锁定

            if (pckeyService.reachMaxCount(pckey)) {
                pckeyService.clearErrorCount(pckey);
            }
            // 验证密码标识
            if (validate == 100) {
                pckeyService.clearErrorCount(pckey);
                HttpSession session = ServletActionContext.getRequest()
                        .getSession();
                session.setAttribute("keyname", username); // 登录名
                session.setAttribute("keyright", userright); // 登录权限
                Calendar c = Calendar.getInstance();
                c.add(Calendar.HOUR_OF_DAY, 10);
                session.setAttribute("logintime", c.getTime().toString()); // 登录时间
                loginpass = true;
            } else {

                int errorCount = pckeyService.addErrorCount(pckey);
                if (errorCount > 0) {
                    msg = "用户名或密码错误,剩余" + errorCount + "次key将被锁定";
                } else {
                    msg = "用户名或密码错误,此key已被限制登陆!";
                    pckeyService.setLockDate(pckey);
                }
            }
        } else {
            // 锁定
            msg = "此key已经被锁定,您无法登陆系统";
        }
    } else {
        // key 不合法
        msg = "请插入合法key";
    }

    // 添加审计日志
    auditlogService.addAuditlog(remoteIp, username, localIp, 1, 1,
            loginpass == true ? "登录系统成功!" : "登陆系统失败", 1);
    return loginpass == true ? SUCCESS : "error";
}

[/code]
[code="java"]
private List queryFromlist(List list) {
List sublist = new ArrayList();
if (list == null || list.size() == 0) {
return null;
}
for (AuditlogVO vo : list) {
if (StringUtils.isNotEmpty(startTime)) {
long starttime = DateUtils.date2LongNumber(startTime);
long createdate = DateUtils.date2LongNumber(vo.getCreatedate());
if (starttime > createdate)
continue;
}
if (StringUtils.isNotEmpty(endTime)) {
long endtime = DateUtils.date2LongNumber(endTime);
long createdate = DateUtils.date2LongNumber(vo.getCreatedate());
if (createdate > endtime)
continue;
}

        if (StringUtils.isEmpty(logType) || logType.equals("全部")
                || logType.equals(vo.getTypename())) {
            if (StringUtils.isEmpty(logaction) || logaction.equals("全部")
                    || logaction.equals(vo.getActionname())) {
                if (StringUtils.isEmpty(logLevel) || logLevel.equals("全部")
                        || logLevel.equals(vo.getLevelname())) {
                    if (StringUtils.isEmpty(logResult)
                            || logResult.equals("全部")
                            || logResult.equals(vo.getResult())) {
                        if (StringUtils.isEmpty(subname)
                                || subname.equals(vo.getSubname())) {
                            if (StringUtils.isEmpty(from)
                                    || from.equals(vo.getAddress())) {
                                if (StringUtils.isEmpty(targetIp)
                                        || targetIp.equals(vo.getObjname())) {
                                    sublist.add(vo);

                                }
                            }
                        }

                    }
                }
            }
        }

    }

    return sublist;
}

[/code]

  • 写回答

3条回答 默认 最新

  • chenqisdfx 2013-09-03 15:24
    关注

    第一个就不说了,如果不拆分的话,也是可以接受的。至于第二个,也就是private List queryFromlist(List list)方法中的for循环体,我倒是觉得有必要处理一下。
    比如,for中的第一第二个if语句可以提取为一个返回值为boolean的方法。这要for循环中就好看一点,然后第三个if语句不用提取函数,直接定义一个boolean变量,如下格式:
    boolean b = (StringUtils.isEmpty(logType) || logType.equals("全部") || logType.equals(vo.getTypename())) ;
    b &= (StringUtils.isEmpty(logaction) || logaction.equals("全部") || logaction.equals(vo.getActionname()));
    b &= (StringUtils.isEmpty(logLevel) || logLevel.equals("全部") || logLevel.equals(vo.getLevelname())) ;
    ....

    本回答被题主选为最佳回答 , 对您是否有帮助呢?
    评论
查看更多回答(2条)

报告相同问题?

悬赏问题

  • ¥15 求差集那个函数有问题,有无佬可以解决
  • ¥15 MATLAB动图问题
  • ¥15 【提问】基于Invest的水源涵养
  • ¥20 微信网友居然可以通过vx号找到我绑的手机号
  • ¥15 寻一个支付宝扫码远程授权登录的软件助手app
  • ¥15 解riccati方程组
  • ¥15 display:none;样式在嵌套结构中的已设置了display样式的元素上不起作用?
  • ¥15 使用rabbitMQ 消息队列作为url源进行多线程爬取时,总有几个url没有处理的问题。
  • ¥15 Ubuntu在安装序列比对软件STAR时出现报错如何解决
  • ¥50 树莓派安卓APK系统签名