前端代码 review 流程规范

2023-02-27

预测试
0、明确任务需求和实现效果(需要产品和UI协同)

1、单元测试、集成测试通过

2、todo : C2C 测试通过(验证组件)

3、本地开发环境中,eslint 代码规范检查,无报错无警告

本地测试
4、基本功能正常;界面样式正常;移动端兼容良好;谷歌浏览器显示良好(浏览器兼容性和系统兼容性尽量满足)

5、极端情况正常:错误输入(数据类型不对等)、数据量巨大(1000行数据)、请求异常处理(403-404-500)

6、代码兼容性:是否兼容老代码和老数据结构(如果不能兼容,至少界面不报错)

代码查验
7、多余的空行注释删除

8、命名规范:函数名变量名不规范、代码可读性(避免生僻Hook)

9、性能优化:减少时间空间复杂度;减少全局变量;React 中,减少 render 次数(减少不必要的state,生命周期函数优化等)

10、代码安全:代码是否存在 XSS CFRS 攻击

学习点
1、review 其他人的优点:组件化,可拓展,可复用性

2、项目整体的框架,不同组件在文件上和逻辑上的依赖关系;视图、逻辑的分离等;使用了哪些设计模式

3、大量阅读优秀项目的源码,这样才能看到自己的不足之处;让优秀同事review自己的代码