请教巨大 if elseif 程序的优雅组织方式 - 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
bbxiong
V2EX    Python

请教巨大 if elseif 程序的优雅组织方式

  •  
  •   bbxiong 2020-12-09 22:41:33 +08:00 5796 次点击
    这是一个创建于 1834 天前的主题,其中的信息可能已经有所发展或是发生改变。

    程序功能是使用 lua 大漠插件模拟执行一些简单操作,开启游戏后使用图片查找方式挨个判断当前页面进行操作. 现在使用的 if elseif 结构代码行数已经达到 200+行,还在增加. 1.前面已经过去的游戏画面继续判断比较耗时(不重要) 2.代码看起来非常臃肿

    请教类似逻辑有没有优秀的组织方式?纯代码的组织,不是类似下面使用 table 表的配置类方案(已经在使用中) 排除使用 table 如: 配置_游戏画面识别操作 = { {图片名 = "开始_结束比赛.bmp",点击 = {x=1147,y=547}}; {图片名 = "开始_结束比赛_是否退出.bmp",点击 = {x=384,y=375}}; }

    if game.是游戏界面("开始_结束比赛.bmp") then ldm.点击坐标(1147,547) elseif game.是游戏界面("开始_结束比赛_是否退出.bmp") then 结束开始的比赛 = true ldm.点击坐标(384,375) elseif game.是游戏界面("进入等待界面.bmp") then ldm.KeyPress(32) elseif game.是游戏界面("配置加载.bmp|配置加载 2.bmp|配置_个人设置.bmp") then game.鼠标居中() game.回车() elseif game.是游戏界面("配置加载_继续按钮.bmp|配置加载_继续按钮 2.bmp") then ldm.点击坐标(160,630) elseif game.是游戏界面("广告接受.bmp|广告接受 2.bmp") then game.鼠标居中() game.回车() ... 200+行 end 
    26 条回复    2020-12-11 17:10:50 +08:00
    guyskk0x0
        1
    guyskk0x0  
       2020-12-10 00:02:40 +08:00
    1. 判断是顺序穷举,图片越多越耗时,可以对图片建索引,能直接从 N 个图中选出最相似的。
    2. 大量的映射关系就应该用表,写代码里或是配置文件里都可以。
    wzzzx
        2
    wzzzx  
       2020-12-10 00:29:41 +08:00
    采用注入的方式,会好一些

    var func_map = {
    "fun1" : fun1
    "fun2" : fun2
    }

    fun1() {}
    fun2() {}

    main() {
    var tmp = getKey()
    func_map[tmp]()
    }
    chenliangngng
        3
    chenliangngng  
       2020-12-10 00:51:5 +08:00
    没看题,推荐状态设计模式
    zhuangzhuang1988
        4
    zhuangzhuang1988  
       2020-12-10 01:04:20 +08:00
    还好 一看就知道含义
    就怕瞎优化
    crclz
        5
    crclz  
       2020-12-10 01:05:19 +08:00
    { {图片名 = "开始_结束比赛.bmp",点击 = {x=1147,y=547}}; {图片名 = "开始_结束比赛_是否退出.bmp",点击 = {x=384,y=375}}; }

    你提到的这种方案其实已经是很简洁的了。如果要更简洁,可以用人类友好的配置语言,例如 yaml:
    ![carbon _10_.png]( https://i.loli.net/2020/12/10/8EdHZcLphXzfgAC.png)

    对于图片判断比较耗时的结果(游戏出现 1 个画面,但是要比较 n 次,如果我没理解错的话),你可以采用:
    1. 神经网络。用训练好的模型的末尾某一层作为 embedding (例如,可以理解为一个 256 维的向量),预先计算保存数据库中所有图片的 embedding 。当遇到游戏画面的时候,计算一次 embedding (记为 A ),然后把 A 和数据库中的 embedding 做夹角余弦,找出最大的值。(非常快)
    2. 手动设计一种图片的相似哈希,例如平均的 RGB,或者分块的平均 RGB 。
    3. 进入游戏的虚拟内存空间去做事情(你懂的)
    LadyChunsKite
        6
    LadyChunsKite  
       2020-12-10 01:33:28 +08:00
    看着跟我大二时用按键精灵写的挂机脚本有点像。。
    为了判断游戏可能出现的各种状况,在代码里使用了大量的 goto 语句,。
    现在想想挺有意思的。。
    ericgui
        7
    ericgui  
       2020-12-10 01:57:08 +08:00
    用 table
    laminux29
        8
    laminux29  
       2020-12-10 02:05:36 +08:00
    我觉得用表更好。这不是啥优雅不优雅的事情,而是方便管理。

    特别是用 csv 或 xls 来组织数据,用 excel 管理起来很舒服。
    xcstream
        9
    xcstream  
       2020-12-10 05:33:15 +08:00
    200 行问题不大
    yousabuk
        10
    yousabuk  
       2020-12-10 07:51:24 +08:00 via iPhone
    大俗即大雅,简单易懂,大家都能看懂挺好。
    ungrown
        11
    ungrown  
       2020-12-10 09:06:12 +08:00
    如果各分支的判断是同型的,可以通过特定的数据结构来“查表”,拿我所熟悉的 python 来说,可以所 dict 字典数据类型,其实`case...switch`本质上也是查表(吧?)。
    如果各分支是异型的,或者不完全同型的,可以花些心思整理一下,稍微分个层级,嵌套判断。
    特别复杂的情况,可以灵活混用不同思路,必要的时候将一个判断分成多个判断流程。
    hdbzsgm
        12
    hdbzsgm  
       2020-12-10 11:17:40 +08:00
    用一个 static map 替换大串 if else / switch
    zencoding
        13
    zencoding  
       2020-12-10 11:24:59 +08:00
    zunceng
        14
    zunceng  
       2020-12-10 11:36:26 +08:00
    类比到 web 开发 看到一堆
    if req.uri == "/xxxx/method-a" {
    //do something
    } else if req.uri == "/xxxx/method-b" {
    //do something
    }
    就不能写个好看点的 router 么
    server.Post("/xxxx/method-a", methodA)
    server.Post("/xxxx/method-b", methodB)
    xrr2016
        15
    xrr2016  
       2020-12-10 12:44:34 +08:00
    策略模式,可以看看这篇文章 https://zhuanlan.zhihu.com/p/288072115
    ytymf
        16
    ytymf  
       2020-12-10 12:58:07 +08:00
    状态迁移啊,用状态机
    cmostuor
        17
    cmostuor  
       2020-12-10 12:59:06 +08:00
    一般用大量 if else 写代码的基本上是新手 这行混得久的会有更优更聪明的写 比如查表
    WilliamYang
        18
    WilliamYang  
       2020-12-10 13:03:16 +08:00
    @cmostuor 一般情况确实是,但实际仍然有一大堆写了 10 年如一年代码的同事
    cmostuor
        19
    cmostuor  
       2020-12-10 13:04:43 +08:00
    @WilliamYang 那这种程序员确实该淘汰掉招 985 、211 的
    cmostuor
        20
    cmostuor  
       2020-12-10 13:05:53 +08:00
    @cmostuor 大量 if else 写代码是及其低效的 这样低效的代码写十几年 简直无法想象
    lithbitren
        21
    lithbitren  
       2020-12-10 18:37:25 +08:00
    动态语言分支超过 3 个就比不上表驱动的速度了,静态语言的条件分支性能上限在十几个到几十个不等
    bbxiong
        22
    bbxiong  
    OP
       2020-12-10 22:21:36 +08:00
    ```lua
    {
    {图片名 = "开始_结束比赛.bmp",点击 = {x=1147,y=547}};
    {图片名 = "开始_结束比赛_是否退出.bmp",点击 = {x=384,y=375}};
    }
    ```
    目前为止只有用 table 配置的方式最简单...
    bianz103
        23
    bianz103  
       2020-12-11 07:34:40 +08:00 via iPhone
    采用 for 循环代替呢
    Cloutain
        24
    Cloutain  
       2020-12-11 08:59:43 +08:00
    switch case 即可,本质就是建表
    Leigg
        25
    Leigg  
       2020-12-11 09:27:05 +08:00 via iPhone
    不嵌套还好
    jwchen
        26
    jwchen  
       2020-12-11 17:10:50 +08:00
    搞个字典吧 k:func
    还可以改成工厂方法
    关于     帮助文档     自助推广系统     博客     API     FAQ     Solana     5210 人在线   最高记录 6679       Select Language
    创意工作者们的社区
    World is powered by solitude
    VERSION: 3.9.8.5 118ms UTC 08:43 PVG 16:43 LAX 00:43 JFK 03:43
    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