萌新求教！关于多个if else如何优化代码质量，效率更高，有什么好的建议？

Question

一段代码中，有很多的ifelse怎么去优化这个代码varsayHello=function(personal){if(personalinstanceofLi){console.log('嘿，你好');}elseif(personalinstanceofDan){console.log('吃早饭了么');}elseif(personalinstanceofArthur){console.log('你今天看起来真漂亮')}};varLi=function(){};varDan=function(){};varArthur=function(){};sayHello(newLi());//嘿，你好sayHello(newDan());//吃早饭了么sayHello(newArthur());//你今天看起来真漂亮

Answer 1

是否有好的方法优化？如何重构？

方法肯定是有的。重构if-else时，心中无时无刻把握一个原则：

尽可能地维持正常流程代码在最外层。

意思是说，可以写if-else语句时一定要尽量保持主干代码是正常流程，避免嵌套过深。

实现的手段有：减少嵌套、移除临时变量、条件取反判断、合并条件表达式等。

下面举几个实例来讲解这些重构方法：

异常逻辑处理型重构方法实例一：

重构前：

double disablityAmount(){

    if(_seniority < 2) 

        return 0;

 

    if(_monthsDisabled > 12)

        return 0;

 

    if(_isPartTime)

        return 0;

 

    //do somethig

 

}

重构后：

double disablityAmount(){

    if(_seniority < 2 || _monthsDisabled > 12 || _isPartTime) 

        return 0;

 

    //do somethig

}

这里的重构手法叫合并条件表达式：如果有一系列条件测试都得到相同结果，将这些结果测试合并为一个条件表达式。

这个重构手法简单易懂，带来的效果也非常明显，能有效地较少if语句，减少代码量逻辑上也更加易懂。

异常逻辑处理型重构方法实例二：

重构前：

double getPayAmount(){

    double result;

    if(_isDead) {

        result = deadAmount();

    }else{

        if(_isSeparated){

            result = separatedAmount();

        }

        else{

            if(_isRetired){

                result = retiredAmount();

            else{

                result = normalPayAmount();

            }

        }

    }

    return result;

重构后：

double getPayAmount(){

    if(_isDead) 

        return deadAmount();

 

    if(_isSeparated)

        return separatedAmount();

 

    if(_isRetired)

        return retiredAmount();

 

    return normalPayAmount();

}