一、前言
这是团队协作系列的第一篇,字数 8000+,长文警告 ⚠
有经验的工程师应该了解,开发一个项目前我们需要一份团队开发规范,以此为共识,共同构建我们的项目开发。
去年公司的一个项目需要重构,为什么需要重构?除了产品设计发生变化以外,前端的 UI 样式发生了大变革。产品设计的变化意味着功能的变动,我们要对功能代码进行调整;UI 的大变革体现在高度定制化,我们需要构建一个自己的组件库。作为团队新人,在我看到项目代码,尤其让我印象深刻的是,核心代码无注释、ES6 使用较少、变量命名无章法、文件结构组织混乱、组件封装性有待提高…… 以上想必也是大部分工程师面临的困境……
需要做的事情很多,但古语有云:“三军未动,粮草先行”。面对上述诸多问题,我查阅了很多资料、与很多大佬交流,起草并拉着团队成员制定了第一版前端开发规范,再拉会宣讲、统一团队认知,对内在开发规范层面达成共识,对外展现我们是会思考、有保障的工程师。
本着“取其精华,去其糟粕”的原则,在进行开发工作的过程中,我们对规范进行了适当补充和删减,不断反馈和完善,当然过程非常艰辛。就这样,在大家的共同努力下,代码质量得到了显著提升。
1.1 文档类规范的局限性
在这个项目上的试验成功以后,我准备将这套规范推广到更多项目,正巧那时我已开始负责公司所有的前端开发管理工作。但是遇到了很多问题,其中明显感觉心有余而力不足的一点是,随着项目需要团队开始扩充人数,然而绝大多数都是实习生,这意味着我需要在实习生培养与不同项目开发之间达到动态平衡。实习生没有太多开发经验,对于开发规范的理解相对是有难度的,我想要通过 Code Review 来保证代码质量以及最佳实践的引导和传授。事实证明这很难,我不得不花费大量精力保证代码质量和风格,并且有时迫于紧张的工期压力,没有办法全身心地关注实习生的每一次 pull request,于是只好抓大放小。逐渐地,我发现即便是有文档,也很难实现代码风格的同步,散落在项目代码中的“不规范代码”在将来会产生巨大的隐患,比如:“make a shit mountain of code”。
1.2 团队组织的合作问题
1999 年 9 月 23 日,NASA 发射的火星探测器到达火星,准备执行任务,但与地面失联。按照原计划早就重新绕到火星前面,但地面无信号。最后发现是因为负责制造推进器的公司和负责制造中央处理器的公司采用了不同的单位,力学单位换算出错,最后引发了这场坠毁事故,可想而知这样的代价多么高昂……
NASA 曾经因为力学单位不一致遭受严重的损失。对于团队组织来说,如果开发规范不统一,意味着整体步调不一致,合作将受阻,最后引发一系列问题。为什么呢?站在 10 人以内团队管理经验的角度下,我认为至少存在以下 6 种可能性:
第一、存在不同成员需要开发同一个项目的可能性;
第二、存在不同成员需要维护不属于自己那部分代码的可能性;
第三、存在不同成员需要同时开发一个模块的可能性;
第四、存在不同成员需要同时兼顾不同项目开发的可能性;
第五、存在不同成员使用不同 IDE 的可能性;
第六、存在异地团队协同开发的可能性。
如果不统一开发规范,短期来看,好像开发得更加迅速,可以更快推出产品。但长期来看,以上六点叠加的,不管是技术上还是管理上,它们附带的显性的或者是隐患的成本都极其高。
1.3 使用工具的本质
所以,基于以上两大问题,如何统一开发规范就成了关键问题。ESLint + Prettier (+ editorconfig + husky + commitlint) 这套工具组合可以说是神技了,可以在一定程度上实现代码层面的自动化,至少可以完成团队层面的代码规范统一(代码逻辑仍然需要有经验的工程师进行 Code Review)。这种统一工作表面上是在增加开发人员的心智负担,看上去还和业务没有任何关系,但本质上却提升了代码质量,在测试之前就减少了大量的 bug,在测试之后,减少了代码维护成本和后续开发的认知成本。如此一来,让产品稳定上线,提升用户使用体验,为业务发展保驾护航。否则是在浪费时间,我依稀还记得那一次我花了 3 个多小时合并代码的窘境和痛苦……
在设置 ESLint 和 Prettier 之前,要知道它们是什么,我的概括如下:
ESLint 是什么?检测 JS 代码,发现代码质量问题并修复问题,还可以自己根据项目需要进行规则的自定义配置以及检查范围等等。
Prettier 是什么?代码格式化工具,可以让团队的代码风格保持一致。可支持的源码类型包括:JavaScript, JSX, Angular, Vue, Flow, TypeScript, CSS, HTML, JSON, YAML……
简单来说,前者用于代码潜在问题的发现,后者用于代码风格的保持。
我不准备根据官方文档娓娓道来,而是通过几个常用的脚手架工具直接让大家上手体验效果。
二、浅尝 Prettier
以 VSCode 为 IDE,需要安装好 Prettier 扩展插件。
2.1 设置:保存时格式化
在设置中搜索找到 Editor: Format On Save,勾选它:
勾选之后,通过 Ctrl+Shift+P 搜索找到 settings.json
点击进入后可以看到最下方新增了一条 VSCode 配置设定: "editor.formatOnSave": true,
设置完成后,当你在保存文件时,VSCode 就可以帮你进行格式化了。到这你一定有疑惑,VSCode 根据什么来格式化?是的,格式化的形式需要你自己选择。别担心,这个后面我会提到,go on。
2.2 在 VSCode 中使用 Prettier
打开设置找到 Prettier: Single Quote,勾选它:
同理, settings.json 新增了一条 VSCode 配置设定:"prettier.singleQuote": true,
在前言部分提到,Prettier 是用来保持代码风格的,使用单引号而非双引号就是一种代码风格。在勾选了这个设定以后。找个目录,新建一个 index.js:
const express=require("express");
第一行中就用了双引号,当保存时就会应用 2.1 部分的自动格式化,将双引号修复为单引号。
现在按下保存键试试,会有通知提示你需要进行配置。
点进去,选择 Prettier-Code formatter 即可 (其他前端相关的代码也选择 Prettier 插件即可)。
基本可以猜到,只要是有关 VSCode 的配置都会在你设置后,让 settings.json 新增一条相关配置:
再次保存,你会发现修复好了。
注意,这里的修复不仅是引号问题被格式化修复,赋值号的前后添补了空格,原因在于 Prettier 作为 VSCode 的插件,它自带有一套默认的代码风格配置。
在哪里看配置?新建终端 ⇒ 左边找到【输出】tab ⇒ 右边找到 Prettier。
在上面的截图中,除了 Prettier Options,还有
- 检测 ignore file( .prettierignore) 是否存在
- 检测本地配置文件( .prettierrc or .editorconfig)是否存在
好了,此处又可以猜到检测的目的。如果既安装了插件又存在忽略文件或者配置文件,那么 VSCode 插件的权重则更低,也就是会被本地配置文件覆盖。
在 github 上验证了这个猜想是正确的:github.com/prettier/pr…
从优先级可以看出,对于团队项目开发来说,使用 prettier configuration file 更为实际。另外,理解这个优先级是非常重要的。在项目中,如果你发现配置的和你预想的不一样,你就要想想看是不是这三处冲突了,因为他们是可以并存的。
三、为什么是 ESLint + Prettier
在前言中提到,ESLint 是用于发现并修复代码问题(包括代码风格格式化),这和 Prettier 美化代码风格显得很矛盾,因为它们存在重叠的部分,那么业界为什么要将二者结合在一起使用呢?
3.1 疑惑一
一开始我是很疑惑的,后来在 Prettier 的官方文档中找到如下解释,让我有点豁然开朗:prettier.io/docs/en/com…
ESLint 对于代码问题的检测采用两类规则,一类是格式化规则,一类是代码质量规则。前者与 Prettier 做的事情是一致的,因此,我们通过 Prettier 执行格式化代码的工作,而代码质量的控制由 ESLint 处理。换句话说,前者美化代码,后者捕获 bugs。
3.2 疑惑二
那么问题又来了,既然 ESLint 可以把事情全都做了,为什么不直接用 ESLint?这个问题困扰我很久,我的字节朋友(强哥)给了我答案: ESLint 之类的 Linters 对于代码格式化的能力是有限的,不如 Prettier 那么专业,这是常见的二者冲突问题。
3.3 疑惑三
所以,怎么处理重叠部分的冲突?Prettier 提供了两个插件:
eslint-config-prettier
:
- github.com/prettier/es…
- 解决 ESLint 中的样式规范(即代码风格)和 Prettier 中样式规范的冲突,以 Prettier 的样式规范为准,使 ESLint 中的样式规范自动失效。对应 .eslintrc 配置
extends-"prettier"
eslint-plugin-prettier
:
- github.com/prettier/es…
- 将 Prettier 样式规范作为 ESLint 代码质量规范来使用,同样将格式问题以 error 的形式抛出,即
rule-"prettier/prettier"
,可在rules-"prettier/prettier"
中进行自定义配置。对应 .eslintrc 配置plugin-"prettier"