Python 脚本的优化建议 - V2EX
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 2853 次点击
    这是一个创建于 1614 天前的主题,其中的信息可能已经有所发展或是发生改变。
    几个月前为了自动签到和了解一下 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 感谢,关于代码内容有什么建议吗
    FurN1
        3
    FurN1  
       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     Solana     2697 人在线   最高记录 6679       Select Language
    创意工作者们的社区
    World is powered by solitude
    VERSION: 3.9.8.5 29ms UTC 11:46 PVG 19:46 LAX 03:46 JFK 06:46
    Do have faith in what you're doing.
    ubao msn snddm index pchome yahoo rakuten mypaper meadowduck bidyahoo youbao zxmzxm asda bnvcg cvbfg dfscv mmhjk xxddc yybgb zznbn ccubao uaitu acv GXCV ET GDG YH FG BCVB FJFH CBRE CBC GDG ET54 WRWR RWER WREW WRWER RWER SDG EW SF DSFSF fbbs ubao fhd dfg ewr dg df ewwr ewwr et ruyut utut dfg fgd gdfgt etg dfgt dfgd ert4 gd fgg wr 235 wer3 we vsdf sdf gdf ert xcv sdf rwer hfd dfg cvb rwf afb dfh jgh bmn lgh rty gfds cxv xcv xcs vdas fdf fgd cv sdf tert sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf shasha9178 shasha9178 shasha9178 shasha9178 shasha9178 liflif2 liflif2 liflif2 liflif2 liflif2 liblib3 liblib3 liblib3 liblib3 liblib3 zhazha444 zhazha444 zhazha444 zhazha444 zhazha444 dende5 dende denden denden2 denden21 fenfen9 fenf619 fen619 fenfe9 fe619 sdf sdf sdf sdf sdf zhazh90 zhazh0 zhaa50 zha90 zh590 zho zhoz zhozh zhozho zhozho2 lislis lls95 lili95 lils5 liss9 sdf0ty987 sdft876 sdft9876 sdf09876 sd0t9876 sdf0ty98 sdf0976 sdf0ty986 sdf0ty96 sdf0t76 sdf0876 df0ty98 sf0t876 sd0ty76 sdy76 sdf76 sdf0t76 sdf0ty9 sdf0ty98 sdf0ty987 sdf0ty98 sdf6676 sdf876 sd876 sd876 sdf6 sdf6 sdf9876 sdf0t sdf06 sdf0ty9776 sdf0ty9776 sdf0ty76 sdf8876 sdf0t sd6 sdf06 s688876 sd688 sdf86