type
status
date
slug
summary
tags
category
icon
password
Property
Feb 21, 2024 08:40 AM
notion image

前言

不以规矩,不能成方圆。
本人有幸经历了团队从缺乏标准到逐渐规范的一个过程,在此当做记录供大家参考。
本文从为什么需要规范以及建立规范的重要性出发,逐步引申出如何去建立适合自己团队的规范,详细讨论了前端具体涉及到的规范都有哪些,部分小节有具体的配置步骤和操作链接。
文中涉及到的所有配置均放在 github 上的 Demo 中,觉得不错的点个赞 ❤️❤️❤️。
下面我们开始吧🏃。

为什么需要规范

规范能给我们带来什么好处,如果没有规范会造成什么后果?这里主要拿代码规范来说。
统一代码规范的好处:
  1. 提高代码整体的可读性、可维护性、可复用性、可移植性和可靠性,这会从根本上降低开发成本,也是最重要的一点。
  1. 保证代码的一致性:软件系统中最重要的因素之一就是编码的一致性。如果编码风格一致,也更加易于维护,因为团队内任何人都可以快速理解并修改。
  1. 提升团队整体效率:开发人员通常需要花费大量的时间来解决代码质量问题,如果都按照规范编写,也有助于团队尽早发现问题,甚至完全预防问题,这将提高整个交付过程的效率。
  1. 减少 code review 期间一系列的争议,因为缺乏标准,在争议过程中双方很难妥协(没少因为这事争论过😕)。
若不统一代码规范,可能会造成的后果:
  1. 由于缺乏规范,导致代码风格不一,增加团队成员间的心理负担,极端情况下,某段代码只有某个人能修改(俗称屎山💩)。
  1. 团队间协作更加困难:因为开发人员得适应不同的风格,会导致效率低下(阅读代码是我们花费时间最多的地方)。
  1. 在 code review 期间可能经常为类似的事情做过多的讨论。
  1. 影响团队的生产力和质量,严重的甚至会影响团队和谐。

为什么依然有很多团队缺乏规范

在这件事上,很难达成一致是我认为最重要的原因。并且,仅仅只是拥有规范也是不够的:
  • 当开发人员被要求在短时间内完成任务时,通常会回避质量标准。
  • 团队中总是有一些有个性的人不会为了团队去改变自己的习惯。
  • 有些人在会议上就约定达成了一致,在会下依旧我行我素。
  • ...

如何保持规范

我曾想通过会议讨论的方式来制定规范,但效果却差强人意。将失败的原因总结为大致几点:
  1. 在会议中,思维很容易发散,经常出现的情况是讨论了很多,却很难有实际性的成效,在开发中依然有不少人选择无视规则
  1. 正式的会议往往很难组织,大家很难一起有空闲的时间来讨论,一次 / 两周 都很困难。
  1. 会议中对实际案例分析,提出若干点优化建议后,没有对问题的优先级和侧重点进行划分,导致实际效果并不好。
  1. 还有一点也是我自己的原因,组织会议的能力有待提升...😭
经历了上述的挫败之后,经过反复复盘总结,决定换一种方式来执行:
  1. 对规范问题进行归纳分析并通过文档记录(wiki等),寻找业内最佳解决方案,在团队内进行统一。
  1. 采用小步快跑的方式,有问题就解决问题,按照优先级和重要性进行排序划分,依次将问题纳入迭代,每个迭代重点解决其中几个即好。
  1. 本迭代的规范问题绝不留到下个迭代,防止堆积(当然,有时候还是得向项目经理妥协😮‍💨)。
  1. 在 code review 过程中严格把关,拒绝睁一只眼👀闭一只眼🧐。
  1. 当团队成员对具体某个规范有争议时,及时讨论并定出结论。
  1. 没有规则只是为了规则,制定规范的目的并不是一定要按照某某标准来执行,更多的是团队成员达成一致即可。
  1. 鼓励大家大胆的质疑规则,若不能提高代码的可读性,可维护性,可复用性,可移植性和可靠性的规则都应该被受到质疑。
  1. 以身作则,船头的方向不能偏航。
经过两个月的迭代后,发现效果出奇的好👏,大家的规范意识普遍增强,当遇到规范问题时也不再畏畏缩缩,而是大胆的抛出在群里讨论。

开发者需要建立和遵守的规范

大致可以划分成这几个方向:
  • 开发流程规范
  • 代码规范
  • git commit 规范
  • 项目文件结构规范
  • UI 设计规范

1. 开发流程规范

这里可能有小伙伴有疑问了,开发流程规范不是项目经理定的吗🤔️,跟我有什么关系?
这里想告诉大家的是,开发流程在一定程度上应该是由我们自己来掌控。不管是传统开发的模式还是敏捷开发的模式,对于开发者来说核心依旧是高质高效的完成用户提出的需求
笔者曾见过不少开发者在拿到产品经理的需求后就开始急匆匆的写代码,以此来体现他们的高效,但往往却因为需求理解不到位和前期代码欠缺设计导致 bug 率高和返工。
如何找到适合自己的开发流程是需要依靠经验来支撑的,需要反复总结和思考,最终达到高质高效完成的目的。
说一说笔者自己比较喜欢的开发流程:
notion image
在接收到需求后应第一时间去了解这个需求的背景是什么?这么做到底有没有解决用户的痛点?或者说用户更深层次的需求是什么?如果团队的产品经理经验不丰富,往往可以在这个阶段砍掉很多不合理的需求(这一点真的很重要)
对于复杂大功能往往还需要进行技术方案调研技术方案设计,并输出详细的设计文档。涉及到细节上,则需要将数据流走向、组件设计等通过脑图的形式呈现出来。

2. 代码规范之格式化规范

由于每个开发者的 IDE 不同,即使 IDE 相同也会因为每个人的配置不一样导致格式化的结果不一样。如何确保团队内开发人员采用统一的格式化配置呢?
这里给推荐大家使用 prettier,它内置了一套格式化的规则,具体配置:
1). 安装依赖:
2). 创建一个空配置文件,让编辑器和其他工具知道你正在使用 Prettier:
3). 创建一个.prettierignore 文件,让 Prettier CLI 和编辑器知道哪些文件不能格式化,example:
4). 配置编辑器(VScode 为例)
IDE 中安装 Prettier-Code Formater 插件:
notion image
找到 IDE 中设置模块,搜索 format On Save,勾上这个就可以了。
notion image
现在当我们 Ctrl + S 保存代码时,插件就会帮助我们自动格式化了。
这里有小伙伴要问了,要是有人将没有格式化的代码提交上去怎么办?
这时候就需要在 git commit 的阶段自动将提交的代码进行格式化,这里我们借助工具 husky,它主要可以帮助我们在 git 阶段检查提交消息、运行测试、检查代码等。没接触过的小伙伴可以去官网了解一下,配置如下:
  • 然后将以下内容添加到 package.json 中:
这段配置的意思是:当执行 git commit 阶段前,先执行 lint-stagedlint-staged 中的内容就是对暂存区的文件执行格式化的命令。
其他:若使用的是脚手架工具搭建的项目,会自带 eslint 配置(eslintConfig)。prettier 和 eslint 会有一些配置上的冲突,这个时候需要安装 eslint-config-prettier 以使 ESLint 和 Prettier 相互配合,安装完后在.eslintrc 中配置(以 Create-React-App 为例):
这样就可以用 "prettier" 的部分规则覆盖前面的规则,让它们都能正常工作。
更多详情见:Prettier

3. 代码规范之 JS/TS 规范

JS/TS 主流的大致有这几种:
比较推荐使用 Airbnb JavaScript Style Guide,它在 Github 上足有 12 万 star,几乎覆盖了 JavaScript 的每一项特性。
具体配置:
1). 安装依赖
2). 生成配置文件
跟着终端中的提示,按照自身需求一步步选择即可。
有了具体的规范后,我们同样需要使用工具去约束:还是通过在 git commit 阶段校验,若不通过则取消提交。
配置(还是在 package.json 中的 lint-staged ):
注意: 这里如果选用的 Typescript,则会默认使用 @typescript-eslint/parser 解析器,官方为了追求更快的解析速度,并不会对.ts 文件中的类型进行检查,只会做语法检测。
如果需要对类型也进行检测,需要在 extends 中加上 plugin:@typescript-eslint/recommended-requiring-type-checking
但是在笔者的使用中发现效果并不好,一些基本的类型依然检测不出来,索性这里换了另一种方式:在 pre commit 中执行 yarn run tsc,这里的意思是对项目中 ts 文件进行类型检测,默认会读取根目录中的 tsconfig.json 配置。
这种方式并不完美,它的弊端就在于全量检测,如果项目不大还好,若是项目代码量够多,检测 10-20s 也是常有的事。
更多详情查看 eslint 官网:eslint.org/docs/user-g…

4. 代码规范之 CSS 规范

CSS 检查代码规范使用 stylelint 插件,规范则推荐使用 stylelint-config-standard
1). 安装
2). 在项目的根目录中创建一个配置文件.stylelintrc.json,内容如下:
3). 解决与 prettier 配置的冲突:
4). 将下面配置复制到.stylelintrc.json 中:
5). 在 git commitv 阶段进行检测:

5. 代码规范之自定义其他规范

下面列一些团队内定的其他规范:
(1)命名规范
变量的命名中应尽量减少缩写的情况发生,做到见名知意。
(2)写注释
在每个文件的顶部明确说明该组件做什么,有没有业务理解难点等等,对业务特殊函数 / 变量也需要写注释
(3)变量兜底
(4)辅助函数必须是纯函数
(5)优先使用函数式编程
(6)优先使用函数式组件
除非需要用到错误边界,否则函数式组件应该是首选方法。
(7)组件复杂度
如果一个组件做的事情太多,应适当提取一些逻辑,将其拆分为更小的组件。
如果提取的组件很复杂,则需要依照一定的规则和条件一一提取它。
代码行数并不是一个客观的衡量标准,更多是需要考虑责任划分和抽象。
(8)用错误边界
当需要对大量数据进行渲染处理时,需要通过错误边界组件对其进行降级处理。
更多详情见之前写的专题文章:浅析前端异常及降级处理 。
(9)props 参数传递
props 一层层传递一直是我们很头疼的一个问题,最核心的问题是不清楚 props 是从哪个初始组件传来的,以及 props 中到底有哪些东西,上下文是什么?
因此对于传递较深的场景我推荐直接使用 context,对于 props 中的内容和上下文通过 TS 来解决。
(10)props 传参数量
如果超过 5 个 props,就该考虑是否拆分该组件。在某些情况下,这是需要对组件进行重构的标志。
注意:组件使用的props越多,重新渲染的理由就越多。
(11)避免嵌套三元运算符
三元运算符在第一级之后变得难以阅读,虽然看起来节省了代码空间,但最好在代码中明确意图,保持良好的阅读性。
(12)将列表组件封装成独立组件
(13)避免嵌套渲染函数
(14)组件 / 函数导入导出
...

6. 项目文件结构规范

不得不说,在这件事上我们团队曾经花费了大量的经历去思考和实践,也是信了 React 官方的鬼话:不要花超过五分钟在选择项目文件组织结构上
在项目初期若不重视,到了后期就是到处天马行空,你很难在期望的目录下找到你想要的文件。
文件夹名称全部采用小写 + "-" 来隔开,index.ts 更多是用来做导出作用,要不然最后编辑器中满屏的 index.tsx,很难区分。
通过对工具函数、hooks、api 等加上后缀,更加容易区分引入的文件。

7. Git commit 规范

git commit 规范主要可以帮助开发人员在 code review 期间更容易理解提交的内容,现在大部分主流 commit 规范都是基于 Angular 团队的规范而衍生出来的,它的 message 格式如下:
每个提交消息都包含一个 subject、一个 body 和一个 footer (中间使用空行分割),提交信息的任何一行不能超过 100 个字符。
👉 type 主要有以下几种类型:
  • feat: 一个新特性
  • fix: 修复 bug
  • docs: 文档修改
  • style: 不影响代码含义的更改(空格、格式、缺少分号等)
  • refactor: 代码重构
  • perf: 优化性能
  • test: 测试用例修改
  • chore: 对构建过程或辅助工具和库的更改,例如文档生成
👉 scope:可以是影响范围的任何内容。
👉 subject:包含对更改的简洁描述,规则:
  • 使用陈述语句
  • 第一个字母不要大写
  • 末尾没有点 (.)
👉 body:commit 具体修改内容,可以分为多行,应该包括改变的动机,并与以前的行为进行对比。
👉 footer: 一些备注,通常是修复的 bug 的链接。
截取一张开源库的 commit,example
notion image
有了规范后,我们需要通过工具去约束:commitlint。它要做的就是在我们每次提交 git commit 的时候,都会帮我们检查 commit message 是否符合一定的规范,如果不符合,就让这次提交失败。
具体配置:
更多详情见官网:github.com/conventiona…

8. UI 设计规范

优秀的开发者应该反向推动 UI 的规范,并且能够支撑 UI 规范的落地。
UI 规范的最大好处就是能够提质提效:
  • 在开发者的角度,与设计规范同步形成研发资产,避免重复造轮子;
  • 在测试的角度,能够避免重复的无意义走查;
  • 在 UI 设计师的角度,减少设计成本,提高设计效率,可以快速承接新需求;
  • 站在产品角度,提高产品迭代与优化效率,降低试错成本;
  • 站在用户角度,解决用户体验一致性。
那到底应该怎么去推动 UI 规范?我的做法是先让设计师去给出一系列的规范,没有相关规范就拉上产品经理一起制定规范。然后前端建立一套自己的组件库,再将组件库提供给UI设计师,以此来相互监督是否达成了规范协议。
具体参考 Antd 设计规范

总结

统一规范的最根本目的是为了保证团队成员的一致性,从而减少沟通成本,提高开发效率。
学会热爱标准,但要确保它们不是一成不变的。如果制定的规范不适合您的团队,请确保可以适应和重写所需要的任何规则。它并不是要强制执行一种工作方式,而是为了帮助促进团队之间的互动 👏👏👏。
网站收藏pinia
  • Twikoo