1
wuwukai007 2021-07-25 20:28:37 +08:00
看了下代码都放在外面了,我觉得代码应该统计放在一个目录下 例如 src ,外面提供一个入口和配置就好了就好了,
|
2
Schwi OP @wuwukai007 感谢,关于代码内容有什么建议吗
|
3
IgniteWhite 2021-07-25 21:07:08 +08:00 via iPhone
(免责:我水平不高)看起来有些命名风格不统一,可以按照 pep8 和 google style guide 处理一下
|
4
Schwi OP @IgniteWhite 谢谢,我去看看这两个文档
|
5
pigspy 2021-07-25 22:10:45 +08:00
- main 函数太长了,像参数处理可以抽出单独的函数
- 有一些很奇怪的变量名,像 rs1,rs2,不知道为什么要这么命令 - login_path 这种常量最好抽出来,不要在函数里定义 随便看了下,就这些 |
8
godblessumilk 2021-07-26 07:33:07 +08:00 via Android
try catch 的异常处理可以再严谨些
|
9
009694 2021-07-26 13:59:46 +08:00
1. 变量名驼峰 蛇形混用
2. 变量大写开头 小写开头混用 3. 修改全大写变量的值 |
10
jaredyam 2021-07-26 22:58:22 +08:00
随便两点:
1. 你有声明依赖的 requirements.txt ,但却在 Actions 中又重复了一遍; 2. 你的 try-except 是在捕捉什么异常呢,可不可以更具针对性; |