V2EX = way to explore
V2EX 是一个关于分享和探索的地方
现在注册
已注册用户请  登录
推荐学习书目
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
Schwi
V2EX  ›  Python

Python 脚本的优化建议

  •  
  •   Schwi · 83 天前 via Android · 1637 次点击
    这是一个创建于 83 天前的主题,其中的信息可能已经有所发展或是发生改变。
    几个月前为了自动签到和了解一下 Python,所以 fork 了一个仓库改了改,现在回头看过去总感觉写的好乱,希望大家能给出一些修改建议(指出问题和方向就行)

    https://github.com/cyb233/mimikkoAutoSignIn

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

    随便看了下,就这些
    pigspy
        6
    pigspy   82 天前
    @pigspy 命令 => 命名
    Schwi
        7
    Schwi   82 天前 via Android
    @pigspy 非常感谢,rs 开头的变量我是用来判断多个推送有哪些失败的,不过命名确实不太合适
    godblessumilk
        8
    godblessumilk   82 天前 via Android
    try catch 的异常处理可以再严谨些
    009694
        9
    009694   82 天前
    1. 变量名驼峰 蛇形混用

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

    3. 修改全大写变量的值
    jaredyam
        10
    jaredyam   81 天前
    随便两点:
    1. 你有声明依赖的 requirements.txt ,但却在 Actions 中又重复了一遍;
    2. 你的 try-except 是在捕捉什么异常呢,可不可以更具针对性;
    关于   ·   帮助文档   ·   FAQ   ·   API   ·   我们的愿景   ·   广告投放   ·   感谢   ·   实用小工具   ·   2156 人在线   最高记录 5497   ·     Select Language
    创意工作者们的社区
    World is powered by solitude
    VERSION: 3.9.8.5 · 20ms · UTC 12:57 · PVG 20:57 · LAX 05:57 · JFK 08:57
    ♥ Do have faith in what you're doing.