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 · 2021-07-25 20:21:53 +08:00 via Android · 2450 次点击
    这是一个创建于 1215 天前的主题,其中的信息可能已经有所发展或是发生改变。
    几个月前为了自动签到和了解一下 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  
       2021-07-25 20:28:37 +08:00
    看了下代码都放在外面了,我觉得代码应该统计放在一个目录下 例如 src ,外面提供一个入口和配置就好了就好了,
    Schwi
        2
    Schwi  
    OP
       2021-07-25 20:42:25 +08:00 via Android
    @wuwukai007 感谢,关于代码内容有什么建议吗
    IgniteWhite
        3
    IgniteWhite  
       2021-07-25 21:07:08 +08:00 via iPhone
    (免责:我水平不高)看起来有些命名风格不统一,可以按照 pep8 和 google style guide 处理一下
    Schwi
        4
    Schwi  
    OP
       2021-07-25 21:11:04 +08:00 via Android
    @IgniteWhite 谢谢,我去看看这两个文档
    pigspy
        5
    pigspy  
       2021-07-25 22:10:45 +08:00
    - main 函数太长了,像参数处理可以抽出单独的函数
    - 有一些很奇怪的变量名,像 rs1,rs2,不知道为什么要这么命令
    - login_path 这种常量最好抽出来,不要在函数里定义

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

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

    3. 修改全大写变量的值
    jaredyam
        10
    jaredyam  
       2021-07-26 22:58:22 +08:00
    随便两点:
    1. 你有声明依赖的 requirements.txt ,但却在 Actions 中又重复了一遍;
    2. 你的 try-except 是在捕捉什么异常呢,可不可以更具针对性;
    关于   ·   帮助文档   ·   博客   ·   API   ·   FAQ   ·   实用小工具   ·   1040 人在线   最高记录 6679   ·     Select Language
    创意工作者们的社区
    World is powered by solitude
    VERSION: 3.9.8.5 · 32ms · UTC 22:30 · PVG 06:30 · LAX 14:30 · JFK 17:30
    Developed with CodeLauncher
    ♥ Do have faith in what you're doing.