Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Feature:优化了战斗识别逻辑 #56

Merged
merged 2 commits into from
Dec 15, 2024
Merged

Feature:优化了战斗识别逻辑 #56

merged 2 commits into from
Dec 15, 2024

Conversation

NaNExist
Copy link
Contributor

  • 缩短了匹配失败的等待时间
  • 将角色死亡,退出战斗这一触发频率较低的事件判断放到靠后位置
  • 使用常量替换了Magic Number,便于后续修改

一次匹配流程就要好几秒,感觉失败了没必要等那么久

@KIYI671
Copy link
Owner

KIYI671 commented Dec 15, 2024

  • 缩短了匹配失败的等待时间
  • 将角色死亡,退出战斗这一触发频率较低的事件判断放到靠后位置
  • 使用常量替换了Magic Number,便于后续修改

一次匹配流程就要好几秒,感觉失败了没必要等那么久

之前MD4还是比较需要死亡检测的,不过现在12人,确实不太需求了。不过放在那么后面实际也没有效果了,反而拖延时间,不如删了。
减少匹配失败时间的话,建议增加匹配机会,因为:你永远无法知晓用户的网络环境是怎么样的.jpg

@NaNExist
Copy link
Contributor Author

NaNExist commented Dec 15, 2024

  • 缩短了匹配失败的等待时间
  • 将角色死亡,退出战斗这一触发频率较低的事件判断放到靠后位置
  • 使用常量替换了Magic Number,便于后续修改

一次匹配流程就要好几秒,感觉失败了没必要等那么久

之前MD4还是比较需要死亡检测的,不过现在12人,确实不太需求了。不过放在那么后面实际也没有效果了,反而拖延时间,不如删了。 减少匹配失败时间的话,建议增加匹配机会,因为:你永远无法知晓用户的网络环境是怎么样的.jpg

那我把初始值改大一点,然后跑一会镜牢看看效果怎么样
死亡检测还是留一下比较好,毕竟万一要是发生了不可名状的事情导致死完了,或者小金后面又改了镜牢就不好说了

@KIYI671
Copy link
Owner

KIYI671 commented Dec 15, 2024

  • 缩短了匹配失败的等待时间
  • 将角色死亡,退出战斗这一触发频率较低的事件判断放到靠后位置
  • 使用常量替换了Magic Number,便于后续修改

一次匹配流程就要好几秒,感觉失败了没必要等那么久

之前MD4还是比较需要死亡检测的,不过现在12人,确实不太需求了。不过放在那么后面实际也没有效果了,反而拖延时间,不如删了。 减少匹配失败时间的话,建议增加匹配机会,因为:你永远无法知晓用户的网络环境是怎么样的.jpg

那我把初始值改大一点,然后跑一会镜牢看看效果怎么样
死亡检测还是留一下比较好,毕竟万一要是发生了不可名状的事情导致死完了,或者小金后面又改了镜牢就不好说了

现在放最后,不可名状也没办法,会战至最后一人,除非搏一搏还没死完前识别到这一步的几率了,那还是留着吧。

@NaNExist
Copy link
Contributor Author

NaNExist commented Dec 15, 2024

  • 添加了战斗中匹配成功和失败的统计
  • 增加了自适应等待时间,匹配失败后会增加等待时间,直到等待时间上限

挂了一个多小时,走完了一个完整流程,没遇到问题,感觉时间步进有点小了,我看看还有没有能优化的

@KIYI671 KIYI671 merged commit 0868a4c into KIYI671:main Dec 15, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants