Python 脚本的优化建议

[Schwi]

几个月前为了自动签到和了解一下 Python，所以 fork 了一个仓库改了改，现在回头看过去总感觉写的好乱，希望大家能给出一些修改建议(指出问题和方向就行)

https://github.com/cyb233/mimikkoAutoSignIn

PS：在某些情况下出现的例如 action#768 的问题，有没有好的处理方案？
https://github.com/cyb233/mimikkoAutoSignIn/actions/runs/1046444500

[wuwukai007]

看了下代码都放在外面了，我觉得代码应该统计放在一个目录下 例如 src ，外面提供一个入口和配置就好了就好了，

[Schwi]

@wuwukai007 感谢，关于代码内容有什么建议吗

[FurN1]

（免责：我水平不高）看起来有些命名风格不统一，可以按照 pep8 和 google style guide 处理一下

[Schwi]

@IgniteWhite 谢谢，我去看看这两个文档

[pigspy]

- main 函数太长了，像参数处理可以抽出单独的函数
- 有一些很奇怪的变量名，像 rs1，rs2，不知道为什么要这么命令
- login_path 这种常量最好抽出来，不要在函数里定义

随便看了下，就这些

[pigspy]

@pigspy 命令 => 命名

[Schwi]

@pigspy 非常感谢，rs 开头的变量我是用来判断多个推送有哪些失败的，不过命名确实不太合适

[godblessumilk]

try catch 的异常处理可以再严谨些

[009694]

1. 变量名驼峰 蛇形混用

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

3. 修改全大写变量的值

[jaredyam]

随便两点：
1. 你有声明依赖的 requirements.txt ，但却在 Actions 中又重复了一遍；
2. 你的 try-except 是在捕捉什么异常呢，可不可以更具针对性；