| ... | ... | @@ -16,6 +16,7 @@ |
|
|
|
|
|
|
|
b、如果对于剩余任务数非必须知道，319行get_remain_task操作冗余，影响性能，不查询剩余数量，直接get_one_task，然后在337行做判断
|
|
|
|
|
|
|
|
c、113行update_time后续没有用到不需要处理
|
|
|
|
2. 重复代码
|
|
|
|
|
|
|
|
a、131行和138行功能重复，可以直接全部在131行替换
|
| ... | ... | @@ -23,9 +24,7 @@ |
|
|
|
3. 异常处理
|
|
|
|
|
|
|
|
a、所有的sql读写都没有进行异常处理，写sql时 操作应该用try catch包裹起来，出现异常时rollback
|
|
|
|
|
|
|
|
b、当出现解析
|
|
|
|
|
|
|
|
|
|
|
|
4. 必要注释
|
|
|
|
|
|
|
|
a、很多关键解析部分逻辑复杂，缺少必要注释
|
| ... | ... | @@ -36,7 +35,7 @@ |
|
|
|
|
|
|
|
b、get_one_goods函数的作用是解析json 命名不够直抒胸意，可改为parse_goods
|
|
|
|
|
|
|
|
c、还有 方法、变量的命名不够直抒胸意
|
|
|
|
c、还有很多方法、变量的命名不够直抒胸意
|
|
|
|
|
|
|
|
6. bugs
|
|
|
|
|
| ... | ... | |
