JavaScript重构技巧-降低函数复杂度

本文转载自微信公众号「大迁世界」，转载本文请联系大迁世界公众号。

JavaScript 是一种易于学习的编程语言，编写运行并执行某些操作的程序很容易。然而，要编写一段干净的JavaScript 代码是很困难的。

在本文中，我们将研究如何「降低函数复杂度」。

[[328307]]

将重复的代码移到同个位置

我们应该将重复的代码提取出来，合并放到同个位置，这样当有需要修改的，我们只需要改一个地方即可，同时也减少犯错率。

假设我们有可能很写出下面的代码：

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; 

上面的函数没有任何副作用，因为它不会在函数外部修改任何变量。另外，它也是一个纯函数，对于相同的输入，输出的结果也都一样的。

使用卫语句代替嵌套语句

卫语句的定义用法：

有时候条件式可能出现在嵌套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语句相同的检查，这样有利于阅读。