评审目标
- 实现方案的正确性
- 代码的会味道
- 规范性
流程
- 被评审人需先整体描述需要解决的问题、解决流程 (被评审人讲解过程中,评审人可以记录问题,不要打断被评审者的思路)
- 被评审人讲完,评审人和与会人员可以提问题
- 评审人进行评审 (被评审者或者与会人员记录评审待改进的内容,有时并不是只针对被评审者,而是所有编码者)
- 评审完成之后,落实待修改项,主要是缺陷和规范性
代码的会味道
- 无效代码
- 重复代码
- 异常处理
- 必要注释
- 。。。
规范性
- 缺少项目文档 (app_environ_protection_grade/readme.md)
- data_pump配置文件,缺少流程必要注释说明, 缺少profile (方便随时测试、复现)
改进落实
缺陷:
规范性:
时间:
负责人: