前端代码评审执行框架

一、项目描述

1. 结合设计稿和PRD演示程序功能，确保需求的完整性。
2. 确认开发过程中是否引入了新的依赖项，比如：组件、node模块。
3. 确认本次需求的代码变动范围，提早准备ChangeList，便于高效评审。
4. 确认本次需求是否涉及到前端和后端是否有纠缠不清的业务。
5. 确认本次需求是否做了文字材料方面的归档记录。

二、重点业务

1. 确认是否涉及到真金白银的业务，比如：预约、领券、购买、结算等。
2. 确认是否涉及到服务端CRUD逻辑的业务，因为错误可能会透传。
3. 确认再确认本次需求的核心业务代码。

三、编码设计

1. 确认代码设计的层次结构和数据流转。
2. 确认组件拆分和设计是否合理，比如：职责单一、耦合度等。
3. 确认模块或组件中的方法是否可以移动到单独的文件模块中维护，使模块或组件更加简洁易读。
4. 确认项目是否有README文件内容。

四、编码规范

1. 确认命名规范。
2. 确认注释是否清楚且有用，并只用来解释why而非what，对比较绕的逻辑要有必要的注释。
3. 确认硬编码的数据是否正确，比如：链接地址。
4. 确认定时器的是否合理，禁止滥用定时器。
5. 确认变量是否做了必要的判空和类型判断。
6. 确认深层次的逻辑嵌套是否设计正确且合理。
7. 确认是否有直接或间接的使用到递归，确保递归可以退出。
8. 确认条件表达式是否过于复杂，确保条件表达式清晰明了。
9. 确认是否使用了全局变量进行通信，禁止这种行为。
10. 确认在遍历数组的时候又是否修改数组，会造成数组长度变化而造成程序错误。
11. 确认函数体代码量是否太多，太多了不易阅读和维护。
12. 确认监控场景。

五、环境配置

1. 确认测试和正式环境相关的域名及数据配置是否正确。
2. 确认CICD（持续集成和持续部署）相关配置是否有变动。
3. 确认前端工程构建的配置是否变动（webpack、eslint、babel、package.json等）。

六、合规问题

1. App&小程序防违规运营指南1.0。
2. 确认抽奖类活动是否有相关信息公示。
3. 确认兜底页面或组件是否有相应的用户引导。
4. 确认移动终端dark模式的UI界面是否正常。

七、技术沉淀

1. 确认是否有通用组件、方法、常用业务处理方案的输出。

待续。。。