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

[bbxiong]

程序功能是使用 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   

[guyskk0x0]

1. 判断是顺序穷举，图片越多越耗时，可以对图片建索引，能直接从 N 个图中选出最相似的。
2. 大量的映射关系就应该用表，写代码里或是配置文件里都可以。

[wzzzx]

采用注入的方式，会好一些

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

fun1() {}
fun2() {}

main() {
var tmp = getKey()
func_map[tmp]()
}

[chenliangngng]

没看题，推荐状态设计模式

[zhuangzhuang1988]

还好 一看就知道含义
就怕瞎优化

[crclz]

{ {图片名 = "开始_结束比赛.bmp",点击 = {x=1147,y=547}}; {图片名 = "开始_结束比赛_是否退出.bmp",点击 = {x=384,y=375}}; }

你提到的这种方案其实已经是很简洁的了。如果要更简洁，可以用人类友好的配置语言，例如 yaml：

对于图片判断比较耗时的结果（游戏出现 1 个画面，但是要比较 n 次，如果我没理解错的话），你可以采用：
1. 神经网络。用训练好的模型的末尾某一层作为 embedding （例如，可以理解为一个 256 维的向量），预先计算保存数据库中所有图片的 embedding 。当遇到游戏画面的时候，计算一次 embedding （记为 A ），然后把 A 和数据库中的 embedding 做夹角余弦，找出最大的值。（非常快）
2. 手动设计一种图片的相似哈希，例如平均的 RGB，或者分块的平均 RGB 。
3. 进入游戏的虚拟内存空间去做事情（你懂的）

[LadyChunsKite]

看着跟我大二时用按键精灵写的挂机脚本有点像。。
为了判断游戏可能出现的各种状况，在代码里使用了大量的 goto 语句，。
现在想想挺有意思的。。

[ericgui]

用 table

[laminux29]

我觉得用表更好。这不是啥优雅不优雅的事情，而是方便管理。

特别是用 csv 或 xls 来组织数据，用 excel 管理起来很舒服。

[xcstream]

200 行问题不大

[yousabuk]

大俗即大雅，简单易懂，大家都能看懂挺好。

[ungrown]

如果各分支的判断是同型的，可以通过特定的数据结构来“查表”，拿我所熟悉的 python 来说，可以所 dict 字典数据类型，其实`case...switch`本质上也是查表（吧？）。
如果各分支是异型的，或者不完全同型的，可以花些心思整理一下，稍微分个层级，嵌套判断。
特别复杂的情况，可以灵活混用不同思路，必要的时候将一个判断分成多个判断流程。

[hdbzsgm]

用一个 static map 替换大串 if else / switch

[zencoding]

状态模式 解君愁
https://design-patterns.readthedocs.io/zh_CN/latest/behavioral_patterns/state.html

[zunceng]

类比到 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]

策略模式，可以看看这篇文章 https://zhuanlan.zhihu.com/p/288072115

[ytymf]

状态迁移啊，用状态机

[cmostuor]

一般用大量 if else 写代码的基本上是新手 这行混得久的会有更优更聪明的写 比如查表

[WilliamYang]

@cmostuor 一般情况确实是，但实际仍然有一大堆写了 10 年如一年代码的同事

[cmostuor]

@WilliamYang 那这种程序员确实该淘汰掉招 985 、211 的

[cmostuor]

@cmostuor 大量 if else 写代码是及其低效的 这样低效的代码写十几年 简直无法想象

[lithbitren]

动态语言分支超过 3 个就比不上表驱动的速度了，静态语言的条件分支性能上限在十几个到几十个不等

[bbxiong]

```lua
{
{图片名 = "开始_结束比赛.bmp",点击 = {x=1147,y=547}};
{图片名 = "开始_结束比赛_是否退出.bmp",点击 = {x=384,y=375}};
}
```
目前为止只有用 table 配置的方式最简单...

[bianz103]

采用 for 循环代替呢

[Cloutain]

switch case 即可，本质就是建表

[Leigg]

不嵌套还好

[jwchen]

搞个字典吧 k:func
还可以改成工厂方法