从一次重写原生方法遇到的坑，总结一下Web中的事件系统

写在前面
前段时间，我写过一篇文章前端开发中的Error以及异常捕获。 在文章中，我提到了这个问题：

经过不断探索(不想再喷自己了)，我找到了原因。下面一一道来。本文主要讲解自己找问题原因的思路，如果想看结论和总结，请直接跳到文末。

问题复现
我是在自己以前的项目中测试addEventListener的重写的。这里直接上精简后的问题代码：

import React from 'react';
import ReactDOM from 'react-dom';

const nativeAddEventListener = EventTarget.prototype.addEventListener;
EventTarget.prototype.addEventListener = function (type, func, options) {
    const wrappedFunc = function (...args) {
        try {
             return func.apply(this, args);
        } catch (e) {
             const errorObj = {
                 error_msg: e.message || '',
                 error_stack: e.stack || (e.error && e.error.
                 error_native: e
             };
        }
    }
    return self.nativeAddEventListener.call(this, type, wrappedFunc, options);
};

const App = function() {
    return <div>11111</div>
};

ReactDOM.render(<App/>, document.body);

问题初探索
删掉那一点重写addEventListener的代码后，表现符合预期了。应该是重写那儿的问题。但是仔细看了过后，那段代码并没有什么问题。并且这段代码我在其他地方也试过，表现一直是正常的。是不是和React哪里冲突了？我使用的React版本是16.4

我搜索了react-dom源码中的addEventListener关键字，总共出现了四次。初步看了一下，并没有什么问题，只是注册了一些事件而已。没有具体分析这些代码的含义，我选择了先更换React的版本试一试，于是，我换成了15.6.2的版本。令人吃惊的是，表现符合预期了。难道真的和React的版本有关系? 在我的认知中，两个版本中最大的不同就是：React v16采用了全新的Fiber架构，而我对Fiber的理解大概就是：重新设计了react node的数据结构，模拟实现了自己的任务堆栈，结合时间分片来进行任务的调度，从而更新整个系统。另外，React有自己的一套事件系统，addEventListener和事件也是紧密相关的，难道影响到了这个?

继续探索
我决定从ReactDOM.render()这个方法入手，调试一下ReactDOM的源代码。之前并没有研究过React的源码，压力有点大。调试了一翻之后，我并没有发现什么问题，并且已经有点懵逼了。我准备同时调试react v15和react v16的代码，看看有什么不同。为了方便，我将问题代码全部抽了出来，全部写到了一个html文件中，并且直接引用React的cdn地址。这个时候，我发现了一个神奇的问题：直接引用cdn地址后，不管React是什么版本，就算是v16版本，也不会出现之前问题，表现都是符合预期的。我更加懵逼了。

发现问题
静下心来仔细观察后，我发现了，我cdn引用的都是react的production版本，而我在项目中使用的react代码，却是development版本的，难道是development和production的diff代码，导致了上面的问题。于是我重新仔细看了一下v16的development的代码，找到了代码中一段长长的注释：

大意就是：在开发版本中，react不会采用try{}catch(){}的方式来捕获错误，而是会把所有开发者定义的callback用一个叫做invokeGuardedCallback的函数包裹起来，然后使用一个假的dom，监听、触发自定义事件来执行invokeGuardedCallback，并且通过一个全局的错误捕捉函数来捕获错误。

在这段注释的下面，就是注释中提到的invokeGuardedCallback的代码。

我仔细研究了这个invokeGuardedCallback的代码，其核心就是：

function invokeGuardedCallback(name, func, context, a, b, c, d, e, f){
     ...
     var fakeNode = document.createElement('react');
     var evt = document.createEvent('Event');
     var evtType = 'react-' + (name ? name : 'invokeguardedcallback');
     var callCallback = function(){
        ...
        fakeNode.removeEventListener(evtType, callCallback, false); // 这里很重要！！！
        ...
        func.apply(context, funcArgs); // 这里是真正执行react中的逻辑代码
     } 
     fakeNode.addEventListener(evtType, callCallback, false);
     evt.initEvent(evtType, false, false);
     fakeNode.dispatchEvent(evt);
     ...
}
 

react将所有容易出错的函数，都用这个invokeGuardedCallback包了起来。每一次都重新造一个虚拟的element，然后监听其自定义事件，并且立即触发这个自定义事件。调试了这个invokeGuardedCallback后，我发现在react v16中，发现很多函数被多次执行。
为什么会多次执行呢? 终于，我找到了问题的原因：

我重写了addEventListener, 在函数外包了一层try{}catch(){}，返回的是一个新的函数，所以，最终注册在事件监听器上的，并不是我传入的那个函数。这个时候，调用removeEventListener时，无法移除我传入addEventListener的函数。

在invokeGuardedCallback中，removeEventListener的逻辑相当于并没有生效。于是，在Fiber的调度中，某个函数被多次重复执行了，而被重复执行的函数并不是幂等的，问题便产生了。

问题的总结与思考
问题终于定位了，一句总结，就是：

重写了addEventListener，却并没有考虑到与之对应的removeEventListener，导致removeEventListener无法正常工作。

下面是一些思考：

一开始，如果我仔细看一下react源码中addEventListener周围的代码，或许能更早发现这个问题，就不用绕这么大一个圈了。
自己对于第三方库的development版本和production版本，并没有一个很强烈的认知、意识，以前上线的不少项目，线上竟然还是用的第三方库的development版本，这个毛病，一定得改掉。
分析问题的能力还很欠缺，不够敏感。考虑问题的全面性需要提高。
真的不要随便重写原生方法。。。