Python 脚本的优化建议

2021-07-25 20:21:53 +08:00
 Schwi
几个月前为了自动签到和了解一下 Python,所以 fork 了一个仓库改了改,现在回头看过去总感觉写的好乱,希望大家能给出一些修改建议(指出问题和方向就行)

https://github.com/cyb233/mimikkoAutoSignIn

PS:在某些情况下出现的例如 action#768 的问题,有没有好的处理方案?
https://github.com/cyb233/mimikkoAutoSignIn/actions/runs/1046444500
2294 次点击
所在节点    Python
10 条回复
wuwukai007
2021-07-25 20:28:37 +08:00
看了下代码都放在外面了,我觉得代码应该统计放在一个目录下 例如 src ,外面提供一个入口和配置就好了就好了,
Schwi
2021-07-25 20:42:25 +08:00
@wuwukai007 感谢,关于代码内容有什么建议吗
IgniteWhite
2021-07-25 21:07:08 +08:00
(免责:我水平不高)看起来有些命名风格不统一,可以按照 pep8 和 google style guide 处理一下
Schwi
2021-07-25 21:11:04 +08:00
@IgniteWhite 谢谢,我去看看这两个文档
pigspy
2021-07-25 22:10:45 +08:00
- main 函数太长了,像参数处理可以抽出单独的函数
- 有一些很奇怪的变量名,像 rs1,rs2,不知道为什么要这么命令
- login_path 这种常量最好抽出来,不要在函数里定义

随便看了下,就这些
pigspy
2021-07-25 22:12:04 +08:00
@pigspy 命令 => 命名
Schwi
2021-07-25 23:20:03 +08:00
@pigspy 非常感谢,rs 开头的变量我是用来判断多个推送有哪些失败的,不过命名确实不太合适
godblessumilk
2021-07-26 07:33:07 +08:00
try catch 的异常处理可以再严谨些
009694
2021-07-26 13:59:46 +08:00
1. 变量名驼峰 蛇形混用

2. 变量大写开头 小写开头混用

3. 修改全大写变量的值
jaredyam
2021-07-26 22:58:22 +08:00
随便两点:
1. 你有声明依赖的 requirements.txt ,但却在 Actions 中又重复了一遍;
2. 你的 try-except 是在捕捉什么异常呢,可不可以更具针对性;

这是一个专为移动设备优化的页面(即为了让你能够在 Google 搜索结果里秒开这个页面),如果你希望参与 V2EX 社区的讨论,你可以继续到 V2EX 上打开本讨论主题的完整版本。

https://www.v2ex.com/t/791679

V2EX 是创意工作者们的社区,是一个分享自己正在做的有趣事物、交流想法,可以遇见新朋友甚至新机会的地方。

V2EX is a community of developers, designers and creative people.

© 2021 V2EX