一、项目描述
- 结合设计稿和PRD演示程序功能,确保需求的完整性。
- 确认开发过程中是否引入了新的依赖项,比如:组件、node模块。
- 确认本次需求的代码变动范围,提早准备ChangeList,便于高效评审。
- 确认本次需求是否涉及到前端和后端是否有纠缠不清的业务。
- 确认本次需求是否做了文字材料方面的归档记录。
二、重点业务
- 确认是否涉及到真金白银的业务,比如:预约、领券、购买、结算等。
- 确认是否涉及到服务端CRUD逻辑的业务,因为错误可能会透传。
- 确认再确认本次需求的核心业务代码。
三、编码设计
- 确认代码设计的层次结构和数据流转。
- 确认组件拆分和设计是否合理,比如:职责单一、耦合度等。
- 确认模块或组件中的方法是否可以移动到单独的文件模块中维护,使模块或组件更加简洁易读。
- 确认项目是否有README文件内容。
四、编码规范
- 确认命名规范。
- 确认注释是否清楚且有用,并只用来解释why而非what,对比较绕的逻辑要有必要的注释。
- 确认硬编码的数据是否正确,比如:链接地址。
- 确认定时器的是否合理,禁止滥用定时器。
- 确认变量是否做了必要的判空和类型判断。
- 确认深层次的逻辑嵌套是否设计正确且合理。
- 确认是否有直接或间接的使用到递归,确保递归可以退出。
- 确认条件表达式是否过于复杂,确保条件表达式清晰明了。
- 确认是否使用了全局变量进行通信,禁止这种行为。
- 确认在遍历数组的时候又是否修改数组,会造成数组长度变化而造成程序错误。
- 确认函数体代码量是否太多,太多了不易阅读和维护。
- 确认监控场景。
五、环境配置
- 确认测试和正式环境相关的域名及数据配置是否正确。
- 确认CICD(持续集成和持续部署)相关配置是否有变动。
- 确认前端工程构建的配置是否变动(webpack、eslint、babel、package.json等)。
六、合规问题
- App&小程序防违规运营指南1.0。
- 确认抽奖类活动是否有相关信息公示。
- 确认兜底页面或组件是否有相应的用户引导。
- 确认移动终端dark模式的UI界面是否正常。
七、技术沉淀
- 确认是否有通用组件、方法、常用业务处理方案的输出。