| ... | @@ -23,10 +23,10 @@ |
... | @@ -23,10 +23,10 @@ |
|
|
|
|
|
|
|
b、如果对于剩余任务数非必须知道，319行get_remain_task操作冗余，影响性能，不查询剩余数量，直接get_one_task，然后在337行做判断
|
|
b、如果对于剩余任务数非必须知道，319行get_remain_task操作冗余，影响性能，不查询剩余数量，直接get_one_task，然后在337行做判断
|
|
|
|
|
|
|
|
c、113行update_time后续没有用到不需要处理
|
|
c、113行update_time后续没有用到不需要处理 (已修改)
|
|
|
2. 重复代码
|
|
2. 重复代码
|
|
|
|
|
|
|
|
a、131行和138行功能重复，可以直接全部在131行替换
|
|
a、131行和138行功能重复，可以直接全部在131行替换 (已修改)
|
|
|
|
|
|
|
|
3. 异常处理
|
|
3. 异常处理
|
|
|
|
|
|
| ... | @@ -38,15 +38,15 @@ |
... | @@ -38,15 +38,15 @@ |
|
|
|
|
|
|
|
5、命名
|
|
5、命名
|
|
|
|
|
|
|
|
a、从类名来看不知道这个类的作用
|
|
a、从类名来看不知道这个类的作用 (已修改)
|
|
|
|
|
|
|
|
b、get_one_goods函数的作用是解析json 命名不够直抒胸意，可改为parse_goods
|
|
b、get_one_goods函数的作用是解析json 命名不够直抒胸意，可改为parse_goods (已修改)
|
|
|
|
|
|
|
|
c、还有很多方法、变量的命名不够直抒胸意
|
|
c、还有很多方法、变量的命名不够直抒胸意 (已修改了部分，尤其是将get变为parse)
|
|
|
|
|
|
|
|
6. bugs
|
|
6. bugs
|
|
|
|
|
|
|
|
a、读取sql时 拼接 sql语句 有sql注入风险
|
|
a、读取sql时 拼接 sql语句 有sql注入风险
|
|
|
|
|
|
|
|
7. 不足
|
|
7. 不足
|
|
|
|
|
|
| ... | @@ -58,31 +58,19 @@ |
... | @@ -58,31 +58,19 @@ |
|
|
|
|
|
|
|
### 规范性
|
|
### 规范性
|
|
|
```
|
|
```
|
|
|
1. 缺少项目文档 （app_environ_protection_grade/readme.md）
|
|
1. 缺少项目文档 （app_environ_protection_grade/readme.md） (你是啥时候瞎的?明明有readme.md,包含项目背景、基本说明、部署说明)
|
|
|
2. data_pump配置文件，缺少流程必要注释说明, 缺少profile (方便随时测试、复现)
|
|
2. data_pump配置文件，缺少流程必要注释说明, 缺少profile (方便随时测试、复现)
|
|
|
```
|
|
```
|
|
|
|
|
|
|
|
---
|
|
---
|
|
|
|
|
|
|
|
### 改进落实
|
|
|
|
|
```
|
|
|
|
|
缺陷：
|
|
|
|
|
|
|
|
|
|
规范性：
|
|
|
|
|
|
|
|
|
|
时间：
|
|
|
|
|
|
|
|
|
|
负责人：
|
|
|
|
|
```
|
|
|
|
|
|
|
|
|
|
### 已改进问题
|
|
### 已改进问题
|
|
|
```
|
|
```
|
|
|
缺陷：
|
|
见每个问题后面的小括号
|
|
|
|
|
改进后的代码见: tmall_api_review.py
|
|
|
规范性：
|
|
|
|
|
|
|
|
|
|
时间：
|
|
|
|
|
|
|
|
|
|
负责人：
|
|
时间： 2021-11-25
|
|
|
|
改进者： 袁波
|
|
|
|
监督者: 章一锋
|
|
|
```
|
|
```
|
|
|
|
|
|
