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 · 5236 次点击
    这是一个创建于 1480 天前的主题,其中的信息可能已经有所发展或是发生改变。

    程序功能是使用 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:59 +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   ·   实用小工具   ·   2672 人在线   最高记录 6679   ·     Select Language
    创意工作者们的社区
    World is powered by solitude
    VERSION: 3.9.8.5 · 27ms · UTC 14:59 · PVG 22:59 · LAX 06:59 · JFK 09:59
    Developed with CodeLauncher
    ♥ Do have faith in what you're doing.