前端代码 review 流程规范

预测试

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自己的代码