|
|
|
## 评审目标
|
|
|
|
|
|
|
|
```buildoutcfg
|
|
|
|
1. 实现方案的正确性
|
|
|
|
2. 代码的坏味道
|
|
|
|
3. 规范性
|
|
|
|
```
|
|
|
|
|
|
|
|
### 评审日期
|
|
|
|
|
|
|
|
```
|
|
|
|
2021-12-09
|
|
|
|
```
|
|
|
|
|
|
|
|
### 评审人
|
|
|
|
|
|
|
|
```
|
|
|
|
蒋家升、范召贤
|
|
|
|
```
|
|
|
|
|
|
|
|
### 被评审人
|
|
|
|
|
|
|
|
```
|
|
|
|
李子健
|
|
|
|
```
|
|
|
|
|
|
|
|
### 参考链接
|
|
|
|
|
|
|
|
- [代码入口](http://tech.pingansec.com/granite/project-collie-app/-/tree/master/app_juzi_data)
|
|
|
|
|
|
|
|
### 流程
|
|
|
|
|
|
|
|
```buildoutcfg
|
|
|
|
1. 被评审人需先整体描述需要解决的问题、解决流程 (被评审人讲解过程中,评审人可以记录问题,不要打断被评审者的思路)
|
|
|
|
2. 被评审人讲完,评审人和与会人员可以提问题
|
|
|
|
3. 评审人进行评审 (被评审者或者与会人员记录评审待改进的内容,有时并不是只针对被评审者,而是所有编码者)
|
|
|
|
4. 评审完成之后,落实待修改项,主要是缺陷和规范性
|
|
|
|
```
|
|
|
|
|
|
|
|
### 值得学习的地方
|
|
|
|
|
|
|
|
```buildoutcfg
|
|
|
|
暂无
|
|
|
|
```
|
|
|
|
|
|
|
|
### 建议 by蒋家升
|
|
|
|
1. udms太多,从 [爬虫业务](http://tech.pingansec.com/granite/kb/-/wikis/data_stream) 看大类其实就3个;<br>
|
|
|
|
应思考此处是否应该做成判断任务分发的形式:
|
|
|
|
1. 从sql开始就做groupby;
|
|
|
|
2. 大类udms中逻辑判断分发;
|
|
|
|
3. (沟通协调)以爬虫datatype等分组
|
|
|
|
|
|
|
|
2. sql/其它db 的轮子
|
|
|
|
3. 代码规范 [remark]
|
|
|
|
4. 逻辑优化 [remark]
|
|
|
|
|
|
|
|
---
|
|
|
|
|
|
|
|
### 改进落实
|
|
|
|
|
|
|
|
```buildoutcfg
|
|
|
|
时间:
|
|
|
|
|
|
|
|
改进人:
|
|
|
|
|
|
|
|
监督人:
|
|
|
|
``` |