中小型前端團隊代碼規范工程化最佳實踐 - ESLint


前言

There are a thousand Hamlets in a thousand people's eyes.

一千個程序員,就有一千種代碼風格。在前端開發中,有幾個至今還在爭論的代碼風格差異:

  • 單引號還是雙引號?
  • 代碼行結束是否需要分號?
  • 兩個空格還是四個空格?
  • ...

這幾個代碼風格差異在協同開發中經常會被互相吐槽,甚至不能忍受。

除此之外,由於 JavaScript 的靈活性,往往一段代碼能有多種寫法,這時候也會導致協同時差異。並且,有一些寫法可能會導致不易發現的 bug,或者這些寫法的性能不好,開發時也應該避免。

為了解決這類靜態代碼問題,每個團隊都需要一個統一的 JavaScript 代碼規范,團隊成員都遵守這份代碼規范來編寫代碼。當然,靠人來保障代碼規范是不可靠的,需要有對應的工具來保障,ESLint 就是這個工具。

有的讀者看到這里,可能會說:Prettier 也可以保證代碼風格一致。是的,Prettier 確實可以按照設置的規則對代碼進行統一格式化,后面的文章也會有對應的介紹。但是需要明確的一點是,Prettier 只會在格式上對代碼進行格式化,一些隱藏的代碼質量問題 Prettier 是無法發現的,而 ESLint 可以。

關於 ESLint

關於 ESLint,它的 Slogan 是 Find and fix problems in your JavaScript code。如上文所說,它可以發現並修復你 JavaScript 代碼中的問題。來看一下官網上描述 ESLint 具備的三個特性:

  • Find Problems。ESLint 通過靜態代碼分析可以快速發現代碼中的問題。ESLint 可以運行在大多數文本編輯器中,並且也可以在工作流中接入 ESLint
  • Fix Automatically。ESLint 發現的很多問題都可以自動修復
  • Customize。可以定制 ESLint 檢查規則

基於以上描述,我們在前端工程化中可以這樣使用 ESLint:

  1. 基於業界現有的 ESLint 規范和團隊代碼習慣定制一套統一的 ESLint 代碼規則
  2. 將統一代碼規則封裝成 ESLint 規則包接入
  3. 將 ESLint 接入腳手架、編輯器以及研發工作流中

快速上手

先簡單介紹一下如何使用 ESLint,如果已經有所了解的同學,可以直接跳過這一節。

新建一個包含 package.json  的目錄(可以在空目錄下執行 npm init -y),新建一個 index.js

// index.js
const name = 'axuebin';

安裝 eslint :

npm install eslint --save-dev

然后執行 ./node_modules/.bin/eslint --init  或者 npx eslint --init  生成一個 ESLint  配置文件 .eslintc.js

module.exports = {
  env: {
    es2021: true,
  },
  extends: 'eslint:recommended',
  parserOptions: {
    ecmaVersion: 12,
  },
  rules: {},
};

生成好配置文件之后,就可以執行 ./node_modules/.bin/eslint index.js 或者 npx eslint index.js 命令對文件進行檢查。結果如下:
image.png
index.js  中的代碼命中了 no-unused-vars  這個規則,默認情況下,這個規則是會報 error  的,也就是 ESLint 不允許代碼中出現未被使用的變量。這是一個好習慣,有利於代碼的維護。

簡單配置

我們來嘗試配置 ESLint 的檢查規則。以分號和引號舉例,現在你作為團隊代碼規范的指定人,希望團隊成員開發的代碼,都是單引號帶分號的。

打開 .eslintrc.js  配置文件,在 rules  中添加相關配置項:

module.exports = {
  env: {
    es2021: true,
  },
  extends: 'eslint:recommended',
  parserOptions: {
    ecmaVersion: 12,
  },
  rules: {
    semi: ['error', 'always'],
    quotes: ['error', 'single'],
  },
};

然后我們將 index.js  中的代碼改成:

// index.js
const name = 'axuebin';

執行 eslint  命令之后:
image.png
可以看到檢查結果如下:

  • **[no-unused-vars] **'name' is assigned a value but never used。定義了 name 變量卻未使用。
  • **[quotes] **Strings must use singlequote。字符串必須使用單引號。
  • **[semi] **Missing semicolon。缺失分號。

老老實實地按照規范修改代碼,使用單引號並將加上分號。當然,如果你們希望是雙引號和不帶分號,修改相應的配置即可。

具體各個規則如何配置可以查看:https://eslint.org/docs/rules

自動修復

執行 eslint xxx --fix  可以自動修復一些代碼中的問題,將無法自動修復的問題暴露出來。比如上文中提到的引號和分號的問題,就可以通過 --fix  自動修復,而 no-unused-vars  變量未使用的問題,ESLint 就無法自動修復。
image.png

使用配置包

init  生成的配置文件中,我們看到包含這一行代碼:

module.exports = {
  extends: 'eslint:recommended',
};

這一行代碼的意思是,使用 ESLint 的推薦配置。 extends: 'xxx'  就是 繼承,當前的配置繼承於 xxx  的配置,在此基礎上進行擴展。

因此,我們也可以使用任意封裝好的配置,可以在 NPM 上或者 GItHub 上搜索 eslint-config 關鍵詞獲取,本文我們將這類封裝好的配置稱作 “配置集”。比較常見的配置包有以下幾個:

  • eslint-config-airbnb: Airbnb 公司提供的配置集
  • eslint-config-prettier: 使用這個配置集,會關閉一些可能與 Prettier 沖突的規則
  • eslint-config-react: create react app 使用的配置集
  • eslint-config-vue: vuejs 使用的配置集
  • ...

最佳實踐

簡單了解完 ESLint 之后,對於 ESLint 的更多使用細節以及原理,在本篇文章就不展開了,感興趣的朋友可以在官網詳細了解。本文重點還是在於如何在團隊工程化體系中落地 ESLint,這里提幾個最佳實踐。

抽象配置集

對於獨立開發者以及業務場景比較簡單的小型團隊而言,使用現成、完備的第三方配置集是非常高效的,可以較低成本低接入 ESLint 代碼檢查。

但是,對於中大型團隊而言,在實際代碼規范落地的過程中我們會發現,不可能存在一個能夠完全符合團隊風格的三方配置包,我們還是會在 extends 三方配置集的基礎上,再手動在 rules 配置里加一些自定義的規則。時間長了,有可能 A 應用和 B 應用里的 rules 就不一樣了,就很難達到統一的目的。

這時候,就需要一個中心化的方式來管理配置包:根據團隊代碼風格整理(或者基於現有的三方配置集)發布一個配置集,團隊統一使用這個包,就可以做到中心化管理和更新

除此之外,從技術層面考慮,目前一個前端團隊的面對的場景可能比較復雜。比如:

  • 技術選型不一致:框架上 PC 使用 React,H5 使用 Vue;是否使用 TypeScript
  • 跨端場景多:Web 端和小程序端,還有 Node
  • ...

以上問題在真實開發中都是存在的,所以在代碼規范的工程化方案落地時,一個單一功能的配置集是不夠用的,這時候還需要考慮這個配置集如何抽象。

為了解決以上問題,這里提供一種解決方案的思路:
image.png
具體拆解來看,就是有一個類似 eslint-config-standard 的基礎規則集(包括代碼風格、變量相關、ES6 語法等),在此基礎之上集成社區的一些插件(Vue/React)等,封裝成統一的一個 NPM Package 發布,消費時根據當前應用類型通過不同路徑來 extends 對應的配置集。

這里有一個 Demo,感興趣的朋友可以看一下:eslint-config-axuebin

開發插件

ESLint 提供了豐富的配置供開發者選擇,但是在復雜的業務場景和特定的技術棧下,這些通用規則是不夠用的。ESLint 通過插件的形式賦予了擴展性,開發者可以自定義任意的檢查規則,比如 eslint-plugin-vue / eslint-plugin-react 就是 Vue / React 框架中使用的擴展插件,官網也提供了相關文檔引導開發者開發一個插件。

一般來說,我們也不需要開發插件,但我們至少需要了解有這么個東西。在做一些團隊代碼質量檢查的時候,我們可能會有一些特殊的業務邏輯,這時候 ESLint 插件是可以幫助我們做一些事情。

這里就不展開了,主要就是一些 AST 的用法,照着官方文檔就可以上手,或者可以參考現有的一些插件寫法。

腳手架 / CLI 工具

當有了團隊的統一 ESLint 配置集和插件之后,我們會將它們集成到腳手架中,方便新項目集成和開箱即用。但是對於一些老項目,如果需要手動改造還是會有一些麻煩的,這時候就可以借助於 CLI 來完成一鍵升級。

本文結合上文的 Demo eslint-config-axuebin,設計一個簡單的 CLI Demo。由於當前配置也比較簡單,所以 CLI 只需要做幾件簡單的事情即可:

  • 詢問用戶當前項目的類型(是 JavaScript 還是 TypeScript、是 React 還是 Vue)
  • 根據項目類型寫 .eslintrc.js  文件
  • 根據項目類型安裝所需依賴(比如 vue 需要 eslint-plugin-vue)
  • package.json  的 scripts  中寫入 "lint": "eslint src test --fix"

核心代碼如下:

const path = require('path');
const fs = require('fs');
const chalk = require('chalk');
const spawn = require('cross-spawn');

const { askForLanguage, askForFrame } = require('./ask');
const { eslintrcConfig, needDeps } = require('./config');

module.exports = async () => {
  const language = await askForLanguage();
  const frame = await askForFrame();

  let type = language;
  if (frame) {
    type += `/${frame}`;
  }

  fs.writeFileSync(
    path.join(process.cwd(), '.eslintrc.js'),
    `// Documentation\n// https://github.com/axuebin/eslint-config-axuebin\nmodule.exports = ${JSON.stringify(
      eslintrcConfig(type),
      null,
      2
    )}`
  );

  const deps = needDeps.javascript;
  if (language === 'typescript') {
    deps.concat(needDeps.typescript);
  }
  if (frame) {
    deps.concat(needDeps[frame]);
  }

  spawn.sync('npm', ['install', ...deps, '--save'], { stdio: 'inherit' });
};

可運行的 CLI Demo 代碼見:axb-lint,在項目目錄下執行:axblint eslint 即可,如圖:
image.png

自動化

配置了 ESLint 之后,我們需要讓開發者感知到 ESLint 的約束。開發者可以自己運行 eslint 命令來跑代碼檢查,這不夠高效,所以我們需要一些自動化手段來做這個事情。當然 在開發時,編輯器也有提供相應的功能可以根據當前工作區下的 ESLint 配置文件來檢查當前正在編輯的文件,這個不是我們關心的重點。

一般我們會在有以下幾種方式做 ESLint 檢查:

  • 開發時:依賴編輯器的能力
  • 手動運行:在終端中手動執行 eslint 命令
  • pre-commit:在提交 git 前自動執行 eslint 命令
  • ci:依賴 git 的持續集成,可以將檢查結果輸出文件上傳到服務器

這里提一下 pre-commit 的方案,在每一次本地開發完成提交代碼前就做 ESLint 檢查,保證雲端的代碼是統一規范的。

這種方式非常簡單,只需要在項目中依賴 huskylint-staged 即可完成。安裝好依賴之后,在 package.json 文件加入以下配置即可:

{
  "lint-staged": {
    "*.{js,jsx,ts,tsx}": "eslint --cache --fix"
  },
  "husky": {
    "hooks": {
      "pre-commit": "lint-staged"
    }
  }
}

效果如圖所示:
image.png
如果代碼跑 ESLint 檢查拋了 Error 錯誤,則會中斷 commit 流程:
image.png
這樣就可以確保提交到 GitHub 倉庫上的代碼是統一規范的。(當然,如果認為將這些配置文件都刪了,那也是沒辦法的)

總結

本文介紹了 ESLint 在中小型前端團隊的一些最佳實踐的想法,大家可以在此基礎上擴展,制訂一套完善的 ESLint 工作流,落地到自己團隊中。

本文是前端代碼規范系列文章的其中一篇,后續還有關於 StyleLint/CommitLint/Prettier 等的文章,並且還有一篇完整的關於前端代碼規范工程化實踐的文章,敬請期待(也有可能就鴿了)。


免責聲明!

本站轉載的文章為個人學習借鑒使用,本站對版權不負任何法律責任。如果侵犯了您的隱私權益,請聯系本站郵箱yoyou2525@163.com刪除。



 
粵ICP備18138465號   © 2018-2025 CODEPRJ.COM