BetterGenshinImpact项目中实时任务调度机制的问题与优化
项目地址: https://gitcode.com/gh_mirrors/be/better-genshin-impact 在BetterGenshinImpact项目中，开发者发现了一个关于实时任务调度的重要机制问题。该项目是一个基于C#开发的游戏辅助工具，其中包含了一个任务调度系统用于管理各种自动化功能。
问题背景
项目中的任务调度系统通过addTimer方法来添加实时任务，但实际实现中存在一个设计缺陷。从表面上看，addTimer方法应该是一个"添加"操作，允许同时存在多个实时任务。然而，在底层实现中，该方法实际上是执行了一个"设置"操作，这意味着每次调用都会覆盖之前设置的所有实时任务。
问题影响
这种实现方式导致了几个严重的问题：
-
无法同时运行多个实时任务：由于每次调用都会覆盖之前的设置，开发者无法通过JavaScript脚本同时启用多个实时任务。
-
功能冲突：例如，当用户同时启用了自动剧情和自动拾取功能时，后启用的功能会覆盖前一个功能的设置，导致功能无法同时工作。
-
预期行为不符：从方法命名(
addTimer)来看，开发者会预期这是一个累加操作而非覆盖操作，这违反了最小意外原则。
技术分析
在底层实现中，GameTaskManager类的AddTimer方法(位于第83行)直接替换了现有的定时器，而不是将新定时器添加到现有定时器集合中。这种实现方式虽然简单，但限制了系统的灵活性。
解决方案
项目维护者已经意识到这个问题，并在代码中添加了TODO注释标记这个问题。后续的修复方案应该考虑：
- 修改
addTimer的实现，使其真正支持添加多个定时器 - 或者提供一个新方法
setTimer来明确表示覆盖行为 - 维护一个定时器集合，允许同时运行多个定时任务
总结
这个案例展示了API设计中的常见陷阱：方法命名与实际行为不符。在系统设计中，保持API的语义清晰和一致性至关重要，特别是当项目需要支持脚本扩展时。BetterGenshinImpact项目通过识别和修复这个问题，将提高其任务调度系统的灵活性和可靠性。
创作声明：本文部分内容由AI辅助生成（AIGC），仅供参考
扫一扫
被折叠的 条评论
为什么被折叠?
到【灌水乐园】发言
