多个“如果”语句？

Question

假设我有一个类似下面的函数checkTime，我必须同时检查多个条件。

var result=0;
function checkTime(time1, time2) {

    if (time1 >= 0 && time2 <= 0) {
        result = 1;
    }
    else if (time1 >= 0 && time2 <= 1) {
        result = 4;
    }
    else if (time1 >= 2 && time2 <= 3) {
        result = 5;
    }
    else if (time1 >= 4 && time2 <= 6) {
        result = 6;
    }
    else if (time1 >= 7 && time2 <= 9) {
        result = 7;
    }
    else if (time1 >= 11 && time2 <= 12) {
        result = 8;
    }
    else if (time1 >= 13 && time2 <= 15) {
        result = 9;
    }
    else if (time1 >= 16 && time2 <= 17) {
        result = 10;
    }
    else if (time1 >= 19 && time2 <= 20) {
        result = 11;
    }
    return result;
}

（上述示例是假设的）

我使用的函数完全有效，但是：

• 有没有更好的方法、程序或公式来取代这一点？（不必如此冗长或丑陋）

Thanx！

Answer 1

首先，你有可能得到一个未定义的结果，这使得事情变得模棱两可。结果应该从0开始吗？这是一个重要的细节。其次，您似乎在处理边界，因此这将有助于更改<代码>

var result = 0;
var diff = time2 - time1;
var bounds = [21, 19, 16, 13, 11, 7, 4, 2, 0];
if (diff <= 0) result = 0; // unexpected outcome
else
    for (position = 1; position < bounds.length; ++position) {
        if (time1 >= bounds[position]) {
            if (time2 < bounds[position - 1]) {
                result = 3 + (bounds.size - position);
            }
            break;
        }
    }
return result;

其他实现是可能的，但是很难根据你的问题来判断你到底在解决什么问题。

后续行动

这段代码有一个空白：

else if (time1 >= 7 && time2 <= 9) {
    result = 7;
}
else if (time1 >= 11 && time2 <= 12) {
    result = 8;
}

如果时间=10和时间2=10，则不存在匹配。当你重复自己的时候，很容易错过这种错误。为每个条件指定下限和上限是不必要的重复。因为我看不到边界的模式（可以委托给函数），所以我只是将下限放入一个数组中，并确保它按降序排序，以便循环可以在第一次匹配后停止。

Answer 2

如果代码相同，并且只有值更改，则可以执行以下操作：

function checkTime(time1, time2) {
  [
    [0, 0, 0], 
    [0, 1, 0]
  ].forEach(function (it) {
    if (time1 >= it[0] && time2 <= it[1]) {
       return it[2];
    }
  });
}

Answer 3

可以使用数组来表示所有组合：

tests = [
    { time1: 0, time2: 0, result: 1 },
    { time1: 0: time2: 1, result: 4 },
    ...
];

for (var i = 0; i < tests.length; i++) {
    if (time1 >= tests[i].time1 && time2 <= tests[i].time2) {
        return tests[i].result;
    }
}