|
## 评审目标
|
|
## 评审目标
|
|
|
|
```
|
|
1. 实现方案的正确性
|
|
1. 实现方案的正确性
|
|
2. 代码的会味道
|
|
2. 代码的会味道
|
|
3. 规范性
|
|
3. 规范性
|
|
|
|
```
|
|
|
|
|
|
|
|
|
|
### 流程
|
|
### 流程
|
|
|
|
```
|
|
1. 被评审人需先整体描述需要解决的问题、解决流程 (被评审人讲解过程中,评审人可以记录问题,不要打断被评审者的思路)
|
|
1. 被评审人需先整体描述需要解决的问题、解决流程 (被评审人讲解过程中,评审人可以记录问题,不要打断被评审者的思路)
|
|
2. 被评审人讲完,评审人和与会人员可以提问题
|
|
2. 被评审人讲完,评审人和与会人员可以提问题
|
|
3. 评审人进行评审 (被评审者或者与会人员记录评审待改进的内容,有时并不是只针对被评审者,而是所有编码者)
|
|
3. 评审人进行评审 (被评审者或者与会人员记录评审待改进的内容,有时并不是只针对被评审者,而是所有编码者)
|
|
4. 评审完成之后,落实待修改项,主要是缺陷和规范性
|
|
4. 评审完成之后,落实待修改项,主要是缺陷和规范性
|
|
|
|
```
|
|
|
|
|
|
|
|
|
|
### 代码的坏味道
|
|
### 代码的坏味道
|
|
|
|
```
|
|
1. 无效代码
|
|
1. 无效代码
|
|
|
|
|
|
a、321行remain_cnt不会小于0,最小为0,当remain_cnt为0时就没有任务了,此时程序break,337行if语句冗余
|
|
a、321行remain_cnt不会小于0,最小为0,当remain_cnt为0时就没有任务了,此时程序break,337行if语句冗余
|
... | @@ -47,19 +54,36 @@ |
... | @@ -47,19 +54,36 @@ |
|
|
|
|
|
b、处理并发不够完美,不同进程会拿到相同任务,解决办法使用队列,一个进程写任务,多个进程消费任务
|
|
b、处理并发不够完美,不同进程会拿到相同任务,解决办法使用队列,一个进程写任务,多个进程消费任务
|
|
|
|
|
|
|
|
```
|
|
|
|
|
|
### 规范性
|
|
### 规范性
|
|
|
|
```
|
|
1. 缺少项目文档 (app_environ_protection_grade/readme.md)
|
|
1. 缺少项目文档 (app_environ_protection_grade/readme.md)
|
|
2. data_pump配置文件,缺少流程必要注释说明, 缺少profile (方便随时测试、复现)
|
|
2. data_pump配置文件,缺少流程必要注释说明, 缺少profile (方便随时测试、复现)
|
|
|
|
```
|
|
|
|
|
|
---
|
|
---
|
|
|
|
|
|
### 改进落实
|
|
### 改进落实
|
|
|
|
```
|
|
缺陷:
|
|
缺陷:
|
|
|
|
|
|
规范性:
|
|
规范性:
|
|
|
|
|
|
时间:
|
|
时间:
|
|
|
|
|
|
负责人: |
|
负责人:
|
|
\ No newline at end of file |
|
```
|
|
|
|
|
|
|
|
### 已改进问题
|
|
|
|
``````
|
|
|
|
缺陷:
|
|
|
|
|
|
|
|
规范性:
|
|
|
|
|
|
|
|
时间:
|
|
|
|
|
|
|
|
负责人:
|
|
|
|
```
|
|
|
|
|
|
|
|
``` |
|
|
|
\ No newline at end of file |