sMythicalBird / ZenlessZoneZero-Auto

绝区零 | ZenlessZoneZero | 零号空洞 | 自动战斗 | 自动化 | 图片分类 | OCR识别
https://fairy.autoscript.site/zh/
GNU General Public License v3.0
1.08k stars 83 forks source link

添加条件任务:最大战斗次数 #187

Closed zidianzhimeng closed 4 weeks ago

zidianzhimeng commented 4 weeks ago

已修改,结算界面的判断条件还是需要的,不然只会在增加战斗次数的时候,条件任务才会执行。我想完成当前战斗再停止

lazydog28 commented 4 weeks ago

那可以将增加战斗次数的代码由选择角色的界面修改到结算界面

zidianzhimeng commented 4 weeks ago

我的理解是结算界面代表一次战斗结束,并不表示开启第二次战斗,简单粗暴的修改并不合理,可以先保持这样,就像我之前提到的:有个问题,现在的战斗次数计算是进入选择角色页面就加1。这会导致如果打不过怪,重新进入选择角色页面,依旧会加1,我的想法是这种战斗失败的情况不要记录战斗次数。当然最好的方法是优化战斗逻辑,不要让战斗失败,但是建议战斗次数记录的逻辑再优化一下。

sMythicalBird commented 4 weeks ago

我的理解是结算界面代表一次战斗结束,并不表示开启第二次战斗,简单粗暴的修改并不合理,可以先保持这样,就像我之前提到的:有个问题,现在的战斗次数计算是进入选择角色页面就加1。这会导致如果打不过怪,重新进入选择角色页面,依旧会加1,我的想法是这种战斗失败的情况不要记录战斗次数。当然最好的方法是优化战斗逻辑,不要让战斗失败,但是建议战斗次数记录的逻辑再优化一下。

目前参数越累越多,手动配置过于复杂了,正在写gui界面,以后参数会统一单独管理。至于战斗失败,这个是可以减计数的。不过目前计数先这样配置,等gui界面和战斗模块完善了在进行优化,你这份pr中task修改了的格式我去掉了,项目是使用black统一格式化的,之前的文件并无格式问题

sMythicalBird commented 4 weeks ago

image @zidianzhimeng 提交之前为什么不测试,变量名一致性为什么不检查

zidianzhimeng commented 4 weeks ago

抱歉,之前没注意config中的变量命名 10E4BA30