写好 JS 条件语句的 5 条守则

192 阅读5分钟

在用 JavaScript 工作时,我们经常和条件语句打交道,这里有5条让你写出更好/干净的条件语句的建议。


1.多重判断时使用 Array.includes

2.更少的嵌套,尽早 return

3.使用默认参数和解构

4.倾向于遍历对象而不是 Switch 语句

5.对 所有/部分 判断使用 Array.every & Array.some

6.总结

1.多重判断时使用 Array.includes

让我们看一下下面这个例子:


0ad9ed17445d492aaa5deaa9a08e2653.png

第一眼,上面这个例子看起来没问题。如果我们有更多名字叫 cherry 和 cranberries 的红色水果呢?我们准备用更多的 || 来拓展条件语句吗?


我们可以用 Array.includes (Array.includes)重写条件语句。


bfccc36e54bb49e8b45397186bb0ab2d.png

我们把红色的水果(red fruits)这一判断条件提取到一个数组。这样一来,代码看起来更整洁。

2.更少的嵌套,尽早 Return

让我们拓展上一个例子让它包含两个条件。



  • 如果没有传入参数 fruit,抛出错误

  • 接受 quantity 参数,并且在 quantity 大于 10 时打印出来



e701935f05f64525a43d74394d0d3c46.png

在上面的代码, 我们有:



  • 1个 if/else 语句筛选出无效的语句

  • 3层if嵌套语句 (条件 1, 2 & 3)



我个人遵循的规则一般是在发现无效条件时,尽早Return。


d5769b0dc49e4320b23715d1b7df1c74.png

这样一来,我们少了一层嵌套语句。这种编码风格非常好,尤其是当你有很长的if语句的时候(想象你需要滚动到最底层才知道还有else语句,这并不酷)


我们可以通过 倒置判断条件 & 尽早return 进一步减少if嵌套。看下面我们是怎么处理判断 条件2 的:


6e673641071c425c80350422ba198c9c.png

通过倒置判断条件2,我们的代码避免了嵌套语句。这个技巧在我们需要进行很长的逻辑判断时是非常有用的,特别是我们希望能够在条件不满足时能够停止下来进行处理。


而且这么做并不困难。问问自己,这个版本(没有嵌套)是不是比之前的(两层条件嵌套)更好,可读性更高?


但对于我,我会保留先前的版本(包含两层嵌套)。这是因为:



  • 代码比较短且直接,包含if嵌套的更清晰

  • 倒置判断条件可能加重思考的负担(增加认知载荷)



因此,应当尽力减少嵌套和尽早return,但不要过度。如果你感兴趣的话,可以看一下关于这个话题的一篇文章和 StackOverflow 上的讨论。



  • Avoid Else, Return Early by Tim Oxley

  • StackOverflow discussion on if/else coding style


3.使用默认参数和解构

我猜下面的代码你可能会熟悉,在JavaScript中我们总是需要检查 null / undefined的值和指定默认值:

09bbba26f1bd44a2a32ce550d4ab8118.png

实际上,我们可以通过声明 默认函数参数 来消除变量 q。

2e126c96efaf4c198d4305e2fea8f266.png

这更加直观,不是吗?注意,每个声明都有自己的默认参数.


例如,我们也能给fruit分配默认值:function test(fruit = 'unknown', quantity = 1)。


如果fruit是一个object会怎么样?我们能分配一个默认参数吗?


99b5250b9ddc4a46a1331237764a0c79.png

看上面这个例子,我们想打印 fruit 对象中可能存在的 name 属性。否则我们将打印unknown。我们可以通过默认参数以及解构从而避免判断条件 fruit && fruit.name

cf71741ca7b24403882b9ac65f6df141.png

由于我们只需要 name 属性,我们可以用 {name} 解构出参数,然后我们就能使用变量 name 代替 fruit.name。


我们也需要声明空对象 {} 作为默认值。如果我们不这么做,当执行 test(undefined) 时,你将得到一个无法对 undefined 或 null 解构的的错误。因为在 undefined 中没有 name 属性。


如果你不介意使用第三方库,这有一些方式减少null的检查:



  • 使用 Lodash get函数

  • 使用Facebook开源的idx库(with Babeljs)



这是一个使用Lodash的例子:

cb2109e5290d4ed69e9ae05795529fe2.png

你可以在jsbin运行demo代码。除此之外,如果你是函数式编程的粉丝,你可能选择使用 Lodash fp,Lodash的函数式版本(方法变更为get或者getOr)。

4.倾向于对象遍历而不是Switch语句

让我们看下面这个例子,我们想根据 color 打印出水果:

8d8101a0c69d4371a647beece1553b8d.png

上面的代码看起来没有错误,但是我找到了一些累赘。用对象遍历实现相同的结果,语法看起来更简洁:


364a6fc63f57401fa892e4f8ac88d117.png

或者你也可以使用 Map实现相同的结果:

f1a7404abcb34bb39562fd92a7bf9120.png

Map是一种在 ES2015 规范之后实现的对象类型,允许你存储 key 和 value 的值。


但我们是否应当禁止switch语句的使用呢?答案是不要限制你自己。从个人来说,我会尽可能的使用对象遍历,但我并不严格遵守它,而是使用对当前的场景更有意义的方式。


Todd Motto有一篇关于 switch 语句对比对象遍历的更深入的文章,你可以在这个地方阅读

TL;DR; 重构语法

在上面的例子,我们能够用Array.filter 重构我们的代码,实现相同的效果。


20d62a35213a4d3dbdb420dbbb6f06bd.png

有着不止一种方法能够实现相同的结果,我们以上展示了 4 种。

5.对 所有/部分 判断使用Array.every & Array.some

这最后一个建议更多是关于利用 JavaScript Array 的内置方法来减少代码行数。看下面的代码,我们想要检查是否所有水果都是红色:


daf5ee10991142449e1793073119e6cb.png

代码那么长!我们可以通过 Array.every减少代码行数:

af5bd12371e2488883ba2865eefa1263.png

现在更简洁了,不是吗?相同的方式,如果我们想测试是否存在红色的水果,我们可以使用 Array.some 一行代码实现。

af644e15fb8448ff991a4ae96384eedb.png

6.总结

让我们一起生产更多可读性高的代码。我希望你能从这篇文章学到东西。

这就是所有的内容。编码快乐!


英文:ecelyn Yeen 译文:眠云(杨涛)

github.com/dawn-plex/translate/blob/master/articles/5-Tips-to-Write-Better-Conditionals-in-JavaScript.md