JavaScript 是一种易于学习的编程语言,编写运行并执行某些操作的程序很容易。然而,要编写一段干净的JavaScript 代码是很困难的。
在本文中,我们将研究如何降低函数复杂度。
将重复的代码移到同个位置
我们应该将重复的代码提取出来,合并放到同个位置,这样当有需要修改的,我们只需要改一个地方即可,同时也减少犯错率。
假设我们有可能很写出下面的代码:
const button = document.querySelector('button'); let toggled = true; button.addEventListener('click', () => { toggled = !toggled; if (toggled) { document.querySelector("p").style.color = 'red'; } else { document.querySelector("p").style.color = 'green'; } })
上面代码中有两个document.querySelector(“p”)
,我们可以这样优化,把document.querySelector(“p”)
保存到一个变量中,然后使用该变量即可,如下所示:
const button = document.querySelector('button'); const p = document.querySelector("p"); let toggled = true; button.addEventListener('click', () => { toggled = !toggled; if (toggled) { p.style.color = 'red'; } else { p.style.color = 'green'; } }
这样我们就不必写长长的document.querySelector("p")
,只需写个变量 p 就行了。
另一个常见的代码实例中的数字,光看数字我们很难知道其表示的含义:
let x = 1; let y = 1; let z = 1;
我们不知道上面这三个一表示什么,所以我们可以移除重复的代码,并用一个合适的变量名来表示,如下所示:
const numPeople = 1; let x = numPeople; let y = numPeople; let z = numPeople;
这样我们就可以知道 numPeople
的值为1
,其表示的人的数量。
简化函数
函数应尽可能简单,最好只做一件事,行数也不要太多,最多不能超过 30 行。
我们不应该使用 ES5 类的方式,也不应将IIFE用于模块或块。相反,我们应该使用类语法,其中可以在类中包含该类的多个实例方法。这会大大减少了函数的体量。
同样,只要我们可以定义函数,函数就应该是纯函数,这意味着他们不应该产生副作用。
例如,最好的简单函数是如下:
const add = (a, b) => a + b;
上面的函数没有任何副作用,因为它不会在函数外部修改任何变量。 另外,它也是一个纯函数,对于相同的输入,输出的结果也都一样的。
使用卫语句代替嵌套语句
卫语句的定义用法
卫语句就是把复杂的条件表达式拆分成多个条件表达式,比如一个很复杂的表达式,嵌套了好几层的if – then-else语句,转换为多个if语句,实现它的逻辑,这多条的if语句就是卫语句
有时候条件式可能出现在嵌套n次才能真正执行,其他分支只是简单报错返回的情况,对于这种情况,应该单独检查报错返回的分支,当条件为真时立即返回,这样的单独检查就是卫语句(guard clauses).卫语句可以把我们的视线从异常处理中解放出来,集中精力到正常处理的代码中。
比如我们可能会编写如下的代码 :
const greet = (firstName, lastName, greeting) => { if (typeof firstName === 'string') { if (typeof lastName === 'string') { if (typeof greeting === 'string') { return `${greeting}, ${firstName}${lastName}`; } } } }
我们可以这样优化
const greet = (firstName, lastName, greeting) => { if (typeof firstName !== 'string') { throw new Error('first name is required'); } if (typeof lastName !== 'string') { throw new Error('last name is required'); } if (typeof greeting !== 'string') { throw new Error('greeting is required'); } return `${greeting}, ${firstName}${lastName}`; }
在第二个示例中,如果每个参数都不是字符串,则抛出错误,从而消除了嵌套的if
语句。
这将嵌套的if
语句在执行相同操作时减少为没有嵌套的if
语句。
嵌套很难阅读和理解,我们应该在所有地方都摆脱它们。
总结
重复的代码总是不好的。 我们应该永远记住“不要重复自己(DRY)”的原则。
另外应该使用一些新的方式来代替 ES5 时代的写法。
最后,应将嵌套的if
语句替换为卫语句,因为它们可以执行与嵌套的if
语句相同的检查,这样有利于阅读。