javascript - 代码优化相关问题，想请教一下大家的做法

Question

先贴出伪代码

        if(没有过期) {
            if(有次数) {
                // 开始抽奖操作
                向后台申请本次抽奖数据
                    doLottery(function() {
                        拆解本次中奖信息
                        根据中奖信息进行转动*操作
                        打开按钮限制
                    });
            }
            else if(没有次数) {
                alert("没有次数");
                打开按钮限制
            }
        }
        else if(已经过期) {
            alert("已经过期");
            打开按钮限制
        }

这里面的 "打开按钮限制" 的意思是让页面上的抽奖按钮无法连续点击(在本次抽奖行为未完成前)。

大家可以看到这里同样操作的伪代码被重复了3次。

我觉得这样很不优雅，想请教一下大家，应该如何优化这样的代码?

Answer 1

if(没有过期&&有次数){
     // 开始抽奖操作
                向后台申请本次抽奖数据
                    doLottery(function() {
                        拆解本次中奖信息
                        根据中奖信息进行转动*操作
                        打开按钮限制
                    });
}else if(没有过期&&没有次数||已经过期){
    alert("没有次数||已经过期");
    打开按钮限制
}

Answer 2

这样可好

是否能抽奖 = true;
msg = '';
if(已经过期){
    是否能抽奖 = false;
    msg = '已经过期';
}
//是否能抽奖还为真时才判断有没有次数
if(是否能抽奖&&没有次数){
    是否能抽奖 = false;
    msg = '没有次数';
}
if(!是否能抽奖){
     alert(msg);
     打开按钮限制
}else{
    // 开始抽奖操作
    向后台申请本次抽奖数据
    doLottery(function() {
        拆解本次中奖信息
        根据中奖信息进行转动*操作
        打开按钮限制
    });
}

Answer 3

个人观点：

if(!没有过期) 过期... return；
if(!有次数) 没有次数... return;
lottery();

抽奖操作可以单独拉在外面方便复用
var lottery = function(){...}

Answer 4

var s = 0;//已过期
var s = 10;//没过期无次数
var s = 11;//没过期有次数
switch(s){
    case 0:
        alert("已经过期");
        break;
    case 10:
        alert("没过期");
        break;
    case 11:
        // 开始抽奖操作
        向后台申请本次抽奖数据
        doLottery(function() {
            拆解本次中奖信息
            根据中奖信息进行转动*操作
        });
        break;
    default:
}
打开按钮限制