Question

我想知道下面的哪些代码通常被接受或者甚至更好：

function doStuff(check) {
    if(!check) {
        return "blablabla";
    } else if (check >= 5) {
        return "blablabla";
    } else {
        return "blablabla";
    }
}

或

function doStuff(check) {
    if(!check) {
        return "blablabla";
    } else if (check >= 5) {
        return "blablabla";
    }
    return "blablabla";
}

由于return语句使JavaScript解释器离开当前函数，因此我没有看到在第二个示例的末尾放置else语句的原因，因为到达那里的唯一方法是提到的比较结果是假的。

这只是一个选择你喜欢的地方的情况，但是要保持一致，还是有一个实际的规则呢？

Answer 1

我甚至更进一步：

function doStuff(check) {
    if(!check)
        return "blablabla";
    if (check >= 5)
        return "blablabla";

    return "blablabla";
}

但这是一个品味问题。虽然我发现最好省略不必要的else语句，但我也有心同事说你应该总是把这个else放在那里以使控制结构更清晰...但在这么简单的情况下我不会打扰他们。

但是，对于数字值，我希望check == 0改为!check。

哪一段代码通常被接受？

1 个答案: