记EventTarget和CallbacksInvoker回调顺序错乱的问题

99755349 · 2024年10月13日 13:03

测试代码：
testInvoker() {
const invoker = new EventTarget()
const fn1 = invoker.on(‘event’, () => {
console.log(’-------- handler1’);
invoker.off(‘event’, fn1);
});
const fn2 = invoker.on(‘event’, () => {
console.log(’-------- handler2’);
});
const fn3 = invoker.on(‘event’, () => {
console.log(’-------- handler3’);
});
const fn4 = invoker.on(‘event’, () => {
console.log(’-------- handler4’);
});
console.log("///////////////////////");
invoker.emit(‘event’);
console.log("///////////////////////");
invoker.emit(‘event’);
}

控制台输出：
///////////////////////
-------- handler1
-------- handler2
-------- handler3
-------- handler4
///////////////////////
-------- handler4
-------- handler2
-------- handler3

huaxing_zheng · 2024年10月13日 14:09

同一事件触发的处理逻辑应该是相互独立，和触发先后无关的。

引擎用的 fastremove ，删除第一个事件的处理函数时，会把最后一个该事件的处理函数移动到该位置。核心目的是提高删除性能。

你写逻辑，如果一个事件有前后处理关系的依赖，你应该写在同一个处理函数里面，这样处理逻辑不会分散（体现了耦合逻辑要集中的原则）。要不然理解你的处理逻辑就非常困难：
A 处要先注册先触发，B 处依赖了 A 处的处理才处理自己的逻辑。这简直是要命的设计。

99755349 · 2024年10月14日 00:05

事件系统有时候就是用来解耦的，有些情况下就是会分开注册的呀

huaxing_zheng · 2024年10月14日 00:07

已经解释了，不理解就算了。不强求。

99755349 · 2024年10月14日 00:13

Implement CallbacksInvoker using a linked list to avoid disorderly calling order caused by array.fastRemoveAt. by JoliChen · Pull Request #17705 · cocos/cocos-engine · GitHub 我提交了一个PR，测试看看。

huaxing_zheng · 2024年10月14日 00:29

说几个点：
第一点：数组的索引效率是高于链表。游戏是追求极致性能体验的。所以要性能都是第一考虑。
第二点：乱序到底是不是引擎在实现上的问题。我的观点是：不是设计问题。
第三点：用户依赖同一事件的多个注册函数的派发顺序的需求没办法依赖引擎层实现机制，问题出在谁身上？（这里我坚定的告诉你，是你设计上出了问题了。原因我前面解释过了）
游戏处理逻辑有两块 A 和 B，这里 B 依赖 A 先执行。你分两个地方写，逻辑就分散了。解耦不是这么解的。

开发中一个很大的原则，就是处理逻辑要尽可能集中。你 A 和 B 看似解耦，只是你代码实现上所谓派发以后，你写得更轻松。但是别人维护你代码，绝对是非常坑的。