本文轉載自微信公眾號「大遷世界」，轉載本文請聯系大遷世界公眾號。

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語句相同的檢查，這樣有利于閱讀。