... | @@ -17,13 +17,13 @@ |
... | @@ -17,13 +17,13 @@ |
|
### 整体感知
|
|
### 整体感知
|
|
```buildoutcfg
|
|
```buildoutcfg
|
|
1. 代码干净、简洁,好评
|
|
1. 代码干净、简洁,好评
|
|
2. 文档不全,不适合新手自主上手
|
|
2. 文档不全,不适合新手自主上手,一脸懵逼
|
|
```
|
|
```
|
|
|
|
|
|
### 值得学习的地方
|
|
### 值得学习的地方
|
|
```buildoutcfg
|
|
```buildoutcfg
|
|
1. 代码干净、简洁,没看到多余乱七八糟的代码
|
|
1.
|
|
2. 文档不全,不适合新手自主上手
|
|
|
|
```
|
|
```
|
|
|
|
|
|
|
|
|
... | @@ -50,7 +50,7 @@ |
... | @@ -50,7 +50,7 @@ |
|
|
|
|
|
5、命名
|
|
5、命名
|
|
|
|
|
|
a、从类名来看不知道这个类的作用
|
|
a、ic_pc.py 42行 format_task_params 全是task_params
|
|
|
|
|
|
b、get_one_goods函数的作用是解析json 命名不够直抒胸意,可改为parse_goods
|
|
b、get_one_goods函数的作用是解析json 命名不够直抒胸意,可改为parse_goods
|
|
|
|
|
... | @@ -69,9 +69,9 @@ |
... | @@ -69,9 +69,9 @@ |
|
```
|
|
```
|
|
|
|
|
|
### 规范性
|
|
### 规范性
|
|
```buildoutcfg
|
|
```
|
|
1. 缺少项目文档readme.md
|
|
建议: <1>添加readme.md,整体说明爬虫目的,快速启动以及部署等。
|
|
2. pip freeze > requirements.txt 将环境导出来,新环境可以pip install -r requirements.txt
|
|
<2>pip freeze > requirements.txt 导出当前环境依赖包
|
|
```
|
|
```
|
|
|
|
|
|
---
|
|
---
|
... | | ... | |