javascript switch() 或 if()

Question

如果我这样做会更好：

if(message == 'redirect')
{
    is_valid.accepted = true;
}
else if(message == 'invalid id')
{
    is_valid.accepted = false;
}
else
{
    is_valid.accepted = false;
}

或者我这样做

switch (message)
{
case 'invalid id':
default:
    is_valid.accepted = false;
    break;
case 'redirect':
    is_valid.accepted = true;
    break;
}

Answer 1

如果您预见到需要添加大量新案例，您可以使用 switch。

如果您不会添加许多新案例，为了清楚起见，我可能会这样做：（

is_valid.accepted = message=='redirect';

另请注意，您对“无效 ID”的检查不会执行任何操作）

不过，如果您必须添加新内容，请注意您没有添加新内容是件好事重复自己不必重复自己不必重复自己，还有性感的格式：

switch (message)
{
    case 'invalid id':
    case 'penguin invasion':
    case 'the internet is down':
    case 'error not enough caffeine':
        is_valid.accepted = false;
        break;

    case 'redirect':
    case 'upvote me':
    case 'vip':
    case 'flamewar':
        is_valid.accepted = true;
        break;

    default:
        is_valid.accepted = false;
        // perhaps log or something
}

想象一下否则你会遇到的所有那些丑陋的else和else-if。

---

旁注：
如果您的规则非常复杂，但仍然是单标志上的白名单-黑名单范例，那么：

var blacklist = ['invalid id', 'penguin invasion', 'the internet is down' 'error not enough caffeine'];
var whitelist = ['redirect', 'upvote me', 'vip', 'flamewar'];

is_valid.accepted = whitelist.indexOf(message)!=-1;

如果您想动态构建白名单，您也可以这样做。

Answer 2

这取决于你对更好的定义。你想要更好的阅读体验还是更好的性能？

我总是 jsPerf 的东西。如果可读性使我的代码更快/更正确，我并不太关心它。

这是一堆不同 switch 与 if/else if/if ==/if === 语句的 jsPerf。

http://jsperf.com/switch-if-else/16

这是修订版 16测试的。因此，如果您在 10 周后查看此测试，请确保滚动到底部并获取最新的测试。

Answer 3

在某些情况下，switch 语句比 if/else 更高效/更具表现力。虽然以下 if/else 语句

let x = 123;

if (x) {/*...*/} // implicit type casting (to boolean)
else {/*...*/}

可以轻松转换为：

switch (!!x) { // explicit type casting (to boolean)
  case true: /*...*/ break;
  default: /*...*/
}

另一方面，此 switch 语句

function algo(x) {/*...performing a complex algorithm...*/}

switch (algo(123)) { // executed once
  case "result 1": /*...*/ break;
  case "result 2": /*...*/ break;
  case "result 3": /*...*/ break;
  default: /*...*/
}

会导致令人难以置信的低效 if/else 语句（switch 更高效）：

if (algo(123) === "result 1") {/*...*/}
else if (algo(123) === "result 2") {/*...*/}
else if (algo(123) === "result 3") {/*...*/}
else {/*...*/}

或者需要一个带有附加变量的 if/else，该变量专门为此目的而声明：

let y = algo(x); // additional variable

if (y === "result 1") {/*...*/}
else if (y === "result 2") {/*...*/}
else if (y === "result 3") {/*...*/}
else {/*...*/}

请注意，附加元素（如变量）会导致更多复杂性和复杂性使得程序更容易出错。 switch 语句不需要这样的变量，因为它更具表现力。

Answer 4

如果您在同一变量上处理一长串可能的条件，则 Switch 会更好。在这种情况下，我认为没有太多理由使用 switch() ，除非您更喜欢这种语法。

Answer 5

如果您使用 if 语句，我个人更喜欢在 if 之上设置默认值，如下所示：

is_valid.accepted = false;
if(message == 'redirect')
{
    is_valid.accepted = true;
}

这样，您始终默认为安全行为，如果稍后添加更多选项，则该行为不太可能被破坏。此外，您可以一目了然地看到默认行为，而无需仔细阅读 if-then-else 逻辑。而且代码要短得多。

Answer 6

三元？
is_valid.accepted = (message !== '无效 ID') ?真：假；