前端代码评审执行框架
2023-09-27 14:27:34 时间
一、项目描述
- 结合设计稿和PRD演示程序功能,确保需求的完整性。
- 确认开发过程中是否引入了新的依赖项,比如:组件、node模块。
- 确认本次需求的代码变动范围,提早准备ChangeList,便于高效评审。
- 确认本次需求是否涉及到前端和后端是否有纠缠不清的业务。
- 确认本次需求是否做了文字材料方面的归档记录。
二、重点业务
- 确认是否涉及到真金白银的业务,比如:预约、领券、购买、结算等。
- 确认是否涉及到服务端CRUD逻辑的业务,因为错误可能会透传。
- 确认再确认本次需求的核心业务代码。
三、编码设计
- 确认代码设计的层次结构和数据流转。
- 确认组件拆分和设计是否合理,比如:职责单一、耦合度等。
- 确认模块或组件中的方法是否可以移动到单独的文件模块中维护,使模块或组件更加简洁易读。
- 确认项目是否有README文件内容。
四、编码规范
- 确认命名规范。
- 确认注释是否清楚且有用,并只用来解释why而非what,对比较绕的逻辑要有必要的注释。
- 确认硬编码的数据是否正确,比如:链接地址。
- 确认定时器的是否合理,禁止滥用定时器。
- 确认变量是否做了必要的判空和类型判断。
- 确认深层次的逻辑嵌套是否设计正确且合理。
- 确认是否有直接或间接的使用到递归,确保递归可以退出。
- 确认条件表达式是否过于复杂,确保条件表达式清晰明了。
- 确认是否使用了全局变量进行通信,禁止这种行为。
- 确认在遍历数组的时候又是否修改数组,会造成数组长度变化而造成程序错误。
- 确认函数体代码量是否太多,太多了不易阅读和维护。
- 确认监控场景。
五、环境配置
- 确认测试和正式环境相关的域名及数据配置是否正确。
- 确认CICD(持续集成和持续部署)相关配置是否有变动。
- 确认前端工程构建的配置是否变动(webpack、eslint、babel、package.json等)。
六、合规问题
- App&小程序防违规运营指南1.0。
- 确认抽奖类活动是否有相关信息公示。
- 确认兜底页面或组件是否有相应的用户引导。
- 确认移动终端dark模式的UI界面是否正常。
七、技术沉淀
- 确认是否有通用组件、方法、常用业务处理方案的输出。
待续。。。
相关文章
- Sql Server中让程序暂停几秒中,再执行后面的命令!(示例)
- 微信小程序 - 无限循环执行动画(Animation)
- 跟老男孩学Linux运维:Shell编程实战2.6 Shell脚本的建立和执行
- 《SAFe 4.0参考指南:精益软件与系统工程的规模化敏捷框架》一 3.10 迭代执行
- JAVA单元测试框架-12-java代码重复执行失败的测试用例
- Java 线程和多线程执行过程分析
- Glide 4.x工作总体执行流程概述
- 异步编程的本质:面向执行上下文管理的编程
- IIFF(立即执行函数表达式)
- 浅析JavaScript中如何实现并发控制:并发控制的理解和执行流程、使用 asyncPool 实现并发控制及其实现原理、如何手写 Promise.all 和 Promise.race
- Linux 文件的读写执行权限的说明
- 深入理解Java虚拟机06--虚拟机字节码执行引擎
- 从环境变量循环获取 - name: "discovery.zen.minimum_master_nodes" value: "2" 带点类型的环境变量然后再放入数组 最后执行命令作为参数 另一种env的处理方法是启动容器后执行脚本 声明新变量赋值环境变量的值 sed 替换变量到 配置文件
- Jenkins_多项目构建(一):单独建立一个项目按顺序执行其它job