JavaScript重構技巧-降低函數(shù)復雜度
本文轉載自微信公眾號「大遷世界」,轉載本文請聯(lián)系大遷世界公眾號。
JavaScript 是一種易于學習的編程語言,編寫運行并執(zhí)行某些操作的程序很容易。然而,要編寫一段干凈的JavaScript 代碼是很困難的。
在本文中,我們將研究如何「降低函數(shù)復雜度」。
將重復的代碼移到同個位置
我們應該將重復的代碼提取出來,合并放到同個位置,這樣當有需要修改的,我們只需要改一個地方即可,同時也減少犯錯率。
假設我們有可能很寫出下面的代碼:
- 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”),我們可以這樣優(yōu)化,把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 就行了。
另一個常見的代碼實例中的數(shù)字,光看數(shù)字我們很難知道其表示的含義:
- let x = 1;
- let y = 1;
- let z = 1;
我們不知道上面這三個一表示什么,所以我們可以移除重復的代碼,并用一個合適的變量名來表示,如下所示:
- const numPeople = 1;
- let x = numPeople;
- let y = numPeople;
- let z = numPeople;
這樣我們就可以知道 numPeople 的值為1,其表示的人的數(shù)量。
簡化函數(shù)
函數(shù)應盡可能簡單,最好只做一件事,行數(shù)也不要太多,最多不能超過 30 行。
我們不應該使用 ES5 類的方式,也不應將「IIFE」用于模塊或塊。相反,我們應該使用類語法,其中可以在類中包含該類的多個實例方法。這會大大減少了函數(shù)的體量。
同樣,只要我們可以定義函數(shù),函數(shù)就應該是純函數(shù),這意味著他們不應該產生副作用。
例如,最好的簡單函數(shù)是如下:
- const add = (a, b) => a + b;
上面的函數(shù)沒有任何副作用,因為它不會在函數(shù)外部修改任何變量。另外,它也是一個純函數(shù),對于相同的輸入,輸出的結果也都一樣的。
使用衛(wèi)語句代替嵌套語句
衛(wèi)語句的定義用法:
衛(wèi)語句就是把復雜的條件表達式拆分成多個條件表達式,比如一個很復雜的表達式,嵌套了好幾層的if - then-else語句,轉換為多個if語句,實現(xiàn)它的邏輯,這多條的if語句就是衛(wèi)語句 |
有時候條件式可能出現(xiàn)在嵌套n次才能真正執(zhí)行,其他分支只是簡單報錯返回的情況,對于這種情況,應該單獨檢查報錯返回的分支,當條件為真時立即返回,這樣的單獨檢查就是衛(wèi)語句(guard clauses).衛(wèi)語句可以把我們的視線從異常處理中解放出來,集中精力到正常處理的代碼中。
比如我們可能會編寫如下的代碼 :
- const greet = (firstName, lastName, greeting) => {
- if (typeof firstName === 'string') {
- if (typeof lastName === 'string') {
- if (typeof greeting === 'string') {
- return `${greeting}, ${firstName}${lastName}`;
- }
- }
- }
- }
我們可以這樣優(yōu)化
- 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}`;
- }
在第二個示例中,如果每個參數(shù)都不是字符串,則拋出錯誤,從而消除了嵌套的if語句。
這將嵌套的if語句在執(zhí)行相同操作時減少為沒有嵌套的if語句。
嵌套很難閱讀和理解,我們應該在所有地方都擺脫它們。
總結
重復的代碼總是不好的。我們應該永遠記住“不要重復自己(DRY)”的原則。
另外應該使用一些新的方式來代替 ES5 時代的寫法。
最后,應將嵌套的if語句替換為衛(wèi)語句,因為它們可以執(zhí)行與嵌套的if語句相同的檢查,這樣有利于閱讀。