代码 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;
    }

[si]

不知道大佬怎么写的，要是我一般会写成连续的。
if() return;
if() return;
if() return;

[whx20202]

代码整洁之道原则：提前 return

if(不满足)：
return
if （不满足）：
return

如果你的多层 if else 都有代码块，那会复杂一点，但是也可以优化

[yesterdaysun]

试试 guard clause

```java
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) {
return false;
}
Long existId = supplierList.get(BaseConstants.INT_ZERO).getId();
return Objects.equals(existId, param.getId());
}
```

[LxExExl]

返回 true 要么为空 要么成员唯一且和输入相等

if (list is null || list.size() == 1 && list.getFirst() == input) return true

else return false

[danliuwo]

switch

[wutiantong]

return CollectionUtils.isEmpty(supplierList) || supplierList.size() == BaseConstants.INT_ONE && Objects.equals(supplierList.get(BaseConstants.INT_ZERO).getId(), param.getId());

一条语句搞定，假如你搞不清||和&&的优先级关系，就自己加个括号吧。

[taaaang]

可以看看《重构 改善既有代码的设计》

[BingoXuan]

连续 if 可读性更高，更容易理解。可读性和代码行数不是成反比的，有时候提高可读性，你需要更多的代码。虽然看起来很冗余，但实际上会让阅读代码的人理解起来更加清晰。

如果 python 这种缩进严格的语言，而你同事喜欢这样嵌套写。淘宝买把游标卡尺来 review 会比较好。

[felixlong]

你这 code 有 bug 啊。supplierList 为空为什么返回的是 true?

[yesterdaysun]

BaseConstants.INT_ONE 和 BaseConstants.INT_ZERO 应该就是 int 0 和 1 吧, 需要定一个常数吗?

单单为了避免魔数也不需要做到这个程度吧

[MrUser]

exists = false;
if (!exists) {}
if (!exists) {}
if (!exists) {}
return exists;

[kaedea]

scoping function 了解一下

[wleexi]

@felixlong 额 不是的。为空表示新号码，可以通过检查的

[qq976739120]

典型的箭头型代码块....如何解决楼上已经说得很好了

[AngryPanda]

尽早 return

[Raymon111111]

提早 return 是正确答案.

[wleexi]

@wutiantong 想过这种，但是考虑了可读性可能不是特别高

[wleexi]

@qq976739120 不太明白

[billgreen1]

表驱动，代码大全里面提到的

[Mutoo]

BaseConstants.INT_ONE
BaseConstants.INT_ZERO

0 和 1 本身就有自带语议了，不属于魔数，没必要搞个这么长的常量吧