简化条件表达式之分解条件表达式（Decompose Conditional）

你有一个复杂的条件语句。从if、then、else三个段落中分别提炼出独立函数。

          动机：

       程序之中，复杂的条件逻辑是最常导致复杂度上升的地点之一。你必须编写代码来检查不同的条件分支、根据不同的分支做不同的事，然后，你很快就会得到一个相当长的函数。大型函数自身就会使代码的可读性下降，而条件逻辑则会使代码更难阅读。在带有复杂条件逻辑的函数中，代码（包括检查条件分支的代码和真正实现功能的代码）会告诉你发生的事，当常常让你弄不清为什么会发生这样的事，这就说明代码的可读性的确大大降低了。

       和任何大块头代码一样，你可以将它分解为多个独立函数，根据每个小块代码的用途，为分解的新函数命名，并将原函数中对应的代码改为调用新建函数，从而更清楚的表达自己的意图。对于条件逻辑，将每个分支条件分解为新函数还可以给你带来更多好处：可以突出条件逻辑，更清楚地表明每个分支的作用，并且突出每个分支的原因。

          做法：

       1、将if段落提炼出来，构成一个独立函数。

       2、将then和else段落提炼出来，各自构成一个独立函数。

       如果发现嵌套的条件逻辑，先观察是否可以使用Replace Nested Conditional with Guard Clauses （以卫语句取代嵌套条件表达式）。如果不行，才开始分解其中的每个条件

if (username == null || username.equals("") || password == null || password.equals("")) {
        //todo
 }

这段条件代码要做的事情就是判断用户名和密码是否正确，但是它用了三个“||”运算符，整个条件判断显得十分冗长。那么，怎么简化呢？我们可以分别把它们提出来，就像下面这样

if (userNameIsValid(username) && passwordIsValid(password)) return true;
    //todo
}
 
 private boolean userNameIsValid(Object userName) {
    if (userName == null || userName.equals("")) return false;
    return true;
}
 
private boolean passwordIsValid(Object password) {
    if (password == null || password.equals("")) return true;
    return true;
}

 虽然条件语句很短 但是不要觉得短就没有提炼的必要，我们要关注的是代码意图和它自身的差距。提炼之后可读性会更高，它看起来就像一条注释一样简单明了。

---

 

参考 https://blog.csdn.net/CountryWisdom/article/details/41286837