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

2020-12-09 22:41:33 +08:00
 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   
5014 次点击
所在节点    Python
26 条回复
guyskk0x0
2020-12-10 00:02:40 +08:00
1. 判断是顺序穷举,图片越多越耗时,可以对图片建索引,能直接从 N 个图中选出最相似的。
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
2020-12-10 00:51:59 +08:00
没看题,推荐状态设计模式
zhuangzhuang1988
2020-12-10 01:04:20 +08:00
还好 一看就知道含义
就怕瞎优化
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
2020-12-10 01:33:28 +08:00
看着跟我大二时用按键精灵写的挂机脚本有点像。。
为了判断游戏可能出现的各种状况,在代码里使用了大量的 goto 语句,。
现在想想挺有意思的。。
ericgui
2020-12-10 01:57:08 +08:00
用 table
laminux29
2020-12-10 02:05:36 +08:00
我觉得用表更好。这不是啥优雅不优雅的事情,而是方便管理。

特别是用 csv 或 xls 来组织数据,用 excel 管理起来很舒服。
xcstream
2020-12-10 05:33:15 +08:00
200 行问题不大
yousabuk
2020-12-10 07:51:24 +08:00
大俗即大雅,简单易懂,大家都能看懂挺好。
ungrown
2020-12-10 09:06:12 +08:00
如果各分支的判断是同型的,可以通过特定的数据结构来“查表”,拿我所熟悉的 python 来说,可以所 dict 字典数据类型,其实`case...switch`本质上也是查表(吧?)。
如果各分支是异型的,或者不完全同型的,可以花些心思整理一下,稍微分个层级,嵌套判断。
特别复杂的情况,可以灵活混用不同思路,必要的时候将一个判断分成多个判断流程。
hdbzsgm
2020-12-10 11:17:40 +08:00
用一个 static map 替换大串 if else / switch
zencoding
2020-12-10 11:24:59 +08:00
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
2020-12-10 12:44:34 +08:00
策略模式,可以看看这篇文章 https://zhuanlan.zhihu.com/p/288072115
ytymf
2020-12-10 12:58:07 +08:00
状态迁移啊,用状态机
cmostuor
2020-12-10 12:59:06 +08:00
一般用大量 if else 写代码的基本上是新手 这行混得久的会有更优更聪明的写 比如查表
WilliamYang
2020-12-10 13:03:16 +08:00
@cmostuor 一般情况确实是,但实际仍然有一大堆写了 10 年如一年代码的同事
cmostuor
2020-12-10 13:04:43 +08:00
@WilliamYang 那这种程序员确实该淘汰掉招 985 、211 的
cmostuor
2020-12-10 13:05:53 +08:00
@cmostuor 大量 if else 写代码是及其低效的 这样低效的代码写十几年 简直无法想象

这是一个专为移动设备优化的页面(即为了让你能够在 Google 搜索结果里秒开这个页面),如果你希望参与 V2EX 社区的讨论,你可以继续到 V2EX 上打开本讨论主题的完整版本。

https://www.v2ex.com/t/733921

V2EX 是创意工作者们的社区,是一个分享自己正在做的有趣事物、交流想法,可以遇见新朋友甚至新机会的地方。

V2EX is a community of developers, designers and creative people.

© 2021 V2EX