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

John Au-Yeung
来源：medium
译者：前端小智

点赞再看，养成习惯

本文 GitHub https://github.com/qq44924588… 上已经收录，更多往期高赞文章的分类，也整理了很多我的文档，和教程资料。欢迎 Star 和完善，大家面试可以参照考点复习，希望我们一起有点东西。

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 语句相同的检查，这样有利于阅读。

代码部署后可能存在的 BUG 没法实时知道，事后为了解决这些 BUG，花了大量的时间进行 log 调试，这边顺便给大家推荐一个好用的 BUG 监控工具 Fundebug。

原文：https://levelup.gitconnected….

文章每周持续更新，可以微信搜索「大迁世界」第一时间阅读和催更（比博客早一到两篇哟），本文 GitHub https://github.com/qq449245884/xiaozhi 已经收录，整理了很多我的文档，欢迎 Star 和完善，大家面试可以参照考点复习，另外关注公众号，后台回复福利，即可看到福利，你懂的。

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

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

简化函数

使用卫语句代替嵌套语句

总结

交流