优化if else这种嵌套过多的泥石流代码

优化if else这种嵌套过多的泥石流代码

原始泥石流代码

const showReplyTimePerson = () => {
    let showReplyInfo;
    // 判断状态是否是 已回复 或 已解决 或 已关闭
    const replyInfo = ['reply', 'resolved','closed'].includes(currentStatusNameUs);
    // content是返回数据里面一个字段，我们判断是否是返回为空
    if(content){
        if(replyInfo){
            showReplyInfo = true;
        } else {
            showReplyInfo = false;
        }
    } else {
        showReplyInfo = false;
    }
    return showReplyInfo;
}

三步改造

• 给变量设置初始值showReplyInfo = false，代替else部分

const showReplyTimePerson = () => {
    let showReplyInfo = false;
    // 判断状态是否是 已回复 或 已解决 或 已关闭
    const replyInfo = ['reply', 'resolved','closed'].includes(currentStatusNameUs);
    // content是返回数据里面一个字段，我们判断是否是返回为空
    if(content){
        if(replyInfo){
            showReplyInfo = true;
        }
    }
    return showReplyInfo;
}

• 适当使用return来阻断程序执行

const showReplyTimePerson = () => {
    if(!content) return false;
    // 判断状态是否是 已回复 或 已解决
    const replyInfo = ['reply', 'resolved','closed'].includes(currentStatusNameUs);
    return replyInfo;
};

• 移除多余变量

const showReplyTimePerson = () => {
    if(!content) return false;
    return ['reply', 'resolved','closed'].includes(currentStatusNameUs);
};

经过三个步骤，代码从14行变成了4行，之前大量的if else嵌套，到最后就一个if语句，这样代码看起来不但高大上，逻辑不混乱，而且可读性也比较强！所以有时候适当的代码重构也是有必要的哦。