代码 if 嵌套过多，怎么优化比较好

[wleexi]

场景 保存信息时，发现手机号重复返回 false 编辑时，手机号与自身之外的手机号重复，返回 false, 没有重复，或者编辑时号码不变返回 true

    private boolean checkMobileExists(Long shopId, CreateSupplierParam param) {
        List<Supplier> supplierList = this.get(shopId, param.getContactMobile());
        if (!CollectionUtils.isEmpty(supplierList)) {
            if (supplierList.size() == BaseConstants.INT_ONE) {
                Long existId = supplierList.get(BaseConstants.INT_ZERO).getId();
                if (Objects.equals(existId, param.getId())) {
                    return true;
                }
            }
            return false;
        }
        return true;
    }

[jmk92]

private boolean checkMobileExists(Long shopId, CreateSupplierParam param) {
List<Supplier> supplierList = this.get(shopId, param.getContactMobile());
if (CollectionUtils.isEmpty(supplierList)) return false;
if (supplierList.size() != BaseConstants.INT_ONE) return false;
Long existId = supplierList.get(BaseConstants.INT_ZERO).getId();
if (Objects.equals(existId, param.getId())==false) return false;
return true;
}

[biossun]

只是展开嵌套的话，可以写成：

private boolean checkMobileExists(Long shopId, CreateSupplierParam param) {
List<Supplier> supplierList = this.get(shopId, param.getContactMobile());

if (CollectionUtils.isEmpty(supplierList)) {
return true;
}

if (supplierList.size() == BaseConstants.INT_ONE &&
Objects.equals(supplierList.get(BaseConstants.INT_ZERO).getId(), param.getId())) {
return true;
}

return false;
}

[codingKingKong]

是不是大概这样?
```java
private boolean checkMobileExists(Long shopId, String mobile, Long id) {
List<Long> supplierList = this.get(shopId, mobile);

if (!CollectionUtils.isEmpty(supplierList))
return true;
if (supplierList.size() != 1)
return false;
Long existId = supplierList.get(0);
return Objects.equals(existId, id);

}

private List<Long> get(long shopId, String mobile){
return Collections.emptyList();
}
```

[hv3s1]

if 里面疯狂套三元。 会不会被接手的人打

[ArcherD]

用 java 12 的 switch expression， pattern match 还要再过几个版本才能支持

[ysc3839]

@fuxiuyin 可以改成 do {} while (0);
这种写法在 C 语言里面挺常见的吧？算是把 while 当 goto 用。

[kaneg]

把比较饶人的判断语句写成方法，比如那个 INTONE，不去跟设计的人确认是很难理解为什么那么判断，可以用一个有意义的方法名，比如 isFirstNumber （）

[ITACHIJAMES]

do{
}while(false)
结合 break 将多级嵌套转化成同级并列

[msg7086]

我们十年前用的 Resharper 辅助重构，可以反转 if 来简化代码，只要点几下就行了。

[lxfxf]

@ArcherD 那不就是 scala 嘛，逃

[guanhui07]

提前 return