if判断如何写的漂亮,增加可读性

这个问题想了好长时间,自己平常写代码也会出现很多if嵌套问题,也没有很好的处理,感觉代码不好看,在网上查资料总结了下,常见if else判断 出现在逻辑异常判断和 各种不同状态处理

常见异常例子

逻辑异常是一个正常的业务流程,状态管理也是一个正常的业务流程, 举个例子

逻辑异常

// 逻辑异常
function addUser(user: any){
    if(user.name ==null){
     //do what
    }
    if(user.phone == null){
     // do other what
     }
}

状态处理

// 状态处理
funtion applyUser(){
    if (user.status == 1){
     // do what
    }
    if ( user.status == 2){
     // do what;
    }
}

常见就这2种,夹杂在一起,这些代码都是必须的,所以会给人无从下手的感觉

逻辑异常解决办法

1. 合并条件(A:熟知语言常if判断为false的条件,B:常用 或条件);

function addUser(user){
 // 重构前
 if(user.name == null || user == ''){
   return false'
}
if(user.name == null){
return false
}

// 合并后,(js中,null,undefined, '',0 在 if中都表示 false,其他语言类似特征可以参考这个)
if(!user.name){
   return false;
}

// 重构前
if(user.age < 18)
       return false;

   if(user.age > 60)
       return false;

   if(!user.name)
       return false;
// 重构后
if(user.age < 18 || user.age > 60 || !user.name){
return false;
}

总结: 如果有一系列条件测试都得到相同结果，将这些结果测试合并为一个条件表达式。

2 少用 result 临时变量,直接return, 如果if-else没有嵌套关系,,提到第一层

getMoney(user){
   let result;
   if(user.type == 'isChild') {
       result = do1();
   }else{
       if(user.type == 'oldman'){
           result = do2();
       }
       else{
           if(user.type == 'qingnian'){
               result = do3();
           else{
               result = do4();
           }
       }
   }
   return result;
   }

   // 重构后
    getMoney(user){
       if(user.type == 'isChild') {
           return do1();
       };
       if(user.type == 'oldman'){
          return  do2();
       };
       if(user.type == 'qingnian'){
           return  do3();
       };
       return do4();
   }

3. 把正常的业务流程放到最外面

public double getAdjustedCapital(){
   double result = 0.0;
   if(_capital > 0.0 ){
       if(_intRate > 0 && _duration >0){
           resutl = (_income / _duration) *ADJ_FACTOR;
       }
   }
   return result;
}
#### A:，减少嵌套和移除临时变量：
public double getAdjustedCapital(){
   if(_capital <= 0.0 ){
       return 0.0;
   }
   if(_intRate > 0 && _duration >0){
       return (_income / _duration) *ADJ_FACTOR;
   }
   return 0.0;
}
#### B:在优化一把,把主要逻辑放到外面
public double getAdjustedCapital(){
   if(_capital <= 0.0 ){
       return 0.0;
   }
   if(_intRate <= 0 || _duration <= 0){
       return 0.0;
   }

   return (_income / _duration) *ADJ_FACTOR;
}

总结: 将条件反转使异常情况先退出，让正常流程维持在主干流程。

在举个例子,看个比较复杂的,多层嵌套

public ArrayList<Student> getStudents(int uid){
        ArrayList<Student> result = new ArrayList<Student>();
        Student stu = getStudentByUid(uid);
        if (stu != null) {
            Teacher teacher = stu.getTeacher();
            if(teacher != null){
                ArrayList<Student> students = teacher.getStudents();
                if(students != null){
                    for(Student student : students){
                        if(student.getAge() > = 18 && student.getGender() == MALE){
                            result.add(student);
                        }
                    }
                }else {
                    logger.error("获取学生列表失败");
                }
            }else {
                logger.error("获取老师信息失败");
            }
        } else {
            logger.error("获取学生信息失败");
        }
        return result;
    }

// 优化一把
public ArrayList<Student> getStudents(int uid){
        ArrayList<Student> result = new ArrayList<Student>();
        Student stu = getStudentByUid(uid);
        if (stu == null) {
            logger.error("获取学生信息失败");
            return result;
        }

        Teacher teacher = stu.getTeacher();
        if(teacher == null){
            logger.error("获取老师信息失败");
            return result;
        }

        ArrayList<Student> students = teacher.getStudents();
        if(students == null){
            logger.error("获取学生列表失败");
            return result;
        }

        for(Student student : students){
            if(student.getAge() > 18 && student.getGender() == MALE){
                result.add(student);
            }
        }
        return result;
    }

状态管理

1.代码中另外一种常见的if else是根据 状态返回不同业务, 如何状态处理具有公共性,这样的话我们可以把状态处理抽成一个公共的方法,

    function addUser(user){
    if(user.status == "isChild"){
       let flg = user.getMony;
       user.money = flg* user.xxxx
    } else if(user.status == "old"){
       let flg = user.getMony;
       user.money = flg* user.xxxx
    }

    // 把处理公共的抽成一个方法

    function getMoney(user){ return money};
    function allMoney (user){
    if(user.status == "isChild"){
       return getMoney(user);
    } else if(user.status == "old"){
       ureturn getMoney(user);
    }
    }

2.要是很复杂的就可以用面向对象的玩法,写个抽象类,每个具体的实现这个抽象类,就不存在if else(其实面向对象的打法,基本可以做到不用if else )

 public getSpeed(){
    switch(_type){
        case EUROPEAN:
            return getBaseSpeed();
        case AFRICAN:
            return getBaseSpeed()-getLoadFactor()*_numberOfCoconuts;
        case NORWEGIAN_BLUE:
            return (_isNailed)?0:getBaseSpeed(_voltage);
    }
}
 // 重构一把
 class Bird{
    abstract double getSpeed();
}

class European extends Bird{
    double getSpeed(){
        return getBaseSpeed();
    }
}

class African extends Bird{
    double getSpeed(){
        return getBaseSpeed()-getLoadFactor()*_numberOfCoconuts;
    }
}

class NorwegianBlue extends Bird{
    double getSpeed(){
        return (_isNailed)?0:getBaseSpeed(_voltage);
    }

这种打法伤筋动骨,一般用的少

以上就是自己体会,和从网上查找资料的一些总结,自己能学习学习,也希望能帮助到大家