推荐学习书目
› 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 · Dec 9, 2020 · 6784 views
    This topic created in 2154 days ago, the information mentioned may be changed or developed.

    程序功能是使用 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 replies  •  2020-12-11 17:10:50 +08:00
    guyskk0x0
    guyskk0x0
    Dec 10, 2020
    1
    1. 判断是顺序穷举,图片越多越耗时,可以对图片建索引,能直接从 N 个图中选出最相似的。
    2. 大量的映射关系就应该用表,写代码里或是配置文件里都可以。
    J55s5
    J55s5
    Dec 10, 2020
    2
    采用注入的方式,会好一些

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

    fun1() {}
    fun2() {}

    main() {
    var tmp = getKey()
    func_map[tmp]()
    }
    chenliangngng
    chenliangngng
    Dec 10, 2020
    3
    没看题,推荐状态设计模式
    zhuangzhuang1988
    zhuangzhuang1988
    Dec 10, 2020
    4
    还好 一看就知道含义
    就怕瞎优化
    crclz
    crclz
    Dec 10, 2020
    5
    { {图片名 = "开始_结束比赛.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
    LadyChunsKite
    Dec 10, 2020
    6
    看着跟我大二时用按键精灵写的挂机脚本有点像。。
    为了判断游戏可能出现的各种状况,在代码里使用了大量的 goto 语句,。
    现在想想挺有意思的。。
    ericgui
    ericgui
    Dec 10, 2020
    7
    用 table
    laminux29
    laminux29
    Dec 10, 2020
    8
    我觉得用表更好。这不是啥优雅不优雅的事情,而是方便管理。

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