推荐学习书目
› Learn Python the Hard Way
Python Sites
› PyPI - Python Package Index
› http://diveintopython.org/toc/index.html
› Pocoo
值得关注的项目
› PyPy
› Celery
› Jinja2
› Read the Docs
› gevent
› pyenv
› virtualenv
› Stackless Python
› Beautiful Soup
› 结巴中文分词
› Green Unicorn
› Sentry
› Shovel
› Pyflakes
› pytest
Python 编程
› pep8 Checker
Styles
› PEP 8
› Google Python Style Guide
› Code Style from The Hitchhiker's Guide
这是一个创建于 1254 天前的主题，其中的信息可能已经有所发展或是发生改变。
几个月前为了自动签到和了解一下 Python，所以 fork 了一个仓库改了改，现在回头看过去总感觉写的好乱，希望大家能给出一些修改建议(指出问题和方向就行)
https://github.com/cyb233/mimikkoAutoSignIn
PS：在某些情况下出现的例如 action#768 的问题，有没有好的处理方案？
https://github.com/cyb233/mimikkoAutoSignIn/actions/runs/1046444500
https://github.com/cyb233/mimikkoAutoSignIn
PS：在某些情况下出现的例如 action#768 的问题，有没有好的处理方案？
https://github.com/cyb233/mimikkoAutoSignIn/actions/runs/1046444500
 |
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 是在捕捉什么异常呢，可不可以更具针对性； |
Select Language