代码逻辑吐槽2.

不合理的代码

1. if else 根据结果返回值是否为空来判断，而不是一开始就区分好类型再处理。

先来看下正常的逻辑

if ("type1".equals(obj.type)) {
    doSth1();
} else if ("type2".equals(obj.type)) {
    doSth2();
}

下面是不正常的逻辑

result = doSth1();
if (result == null) {
    doSth2();
}

代码重构的时候对 doSth1() 方法进行了包装，结果返回结果一直不为null。造成了doSth2() 方法永远都不会调用，这种问题相对较隐蔽，需要debug才能比较清晰的发现问题。

2. if条件

if (null != obj || null != obj.getReturn()) {}

if (null != obj && null != obj.getReturn()) {}

3. JS的class和id的使用

列表页上下各有上一页，下一页，导出等按钮。结果每个按钮都定义一个id，然后相同的方法写两遍。

修改方法：

1. 把 id 改为 class，方法绑定在 class上即可。

2. 使用选择器，$("#id1, #id2").function()。