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

oysun 发布于2019-08-23 14:42 / 463人阅读

摘要：问题初探索删掉那一点重写的代码后，表现符合预期了。每一次都重新造一个虚拟的，然后监听其自定义事件，并且立即触发这个自定义事件。真的不要随便重写原生方法。。。于是，我全面总结一下了中的事件系统，也算是对基础的巩固。

写在前面

前段时间，我写过一篇文章前端开发中的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 11111
};

ReactDOM.render(, document.body);

运行这段代码，浏览器上一片空白，但是却没有任何报错。我一脸懵逼。

问题初探索

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

我搜索了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版本，这个毛病，一定得改掉。

分析问题的能力还很欠缺，不够敏感。考虑问题的全面性需要提高。

真的不要随便重写原生方法。。。

写在后面

在探索这个问题的过程中，我看到了react巧妙应用自定义事件来捕获错误。于是，我全面总结一下了Web中的事件系统，也算是对基础的巩固。由于篇幅已经不够了，这里就直接放文章链接吧：

谈一谈web中的事件
谈一谈web中的事件

欢迎关注我的公众号: 符合预期的CoyPan，
这里只有干货，符合你的预期。

超融合服务器私有云 java方法重写 php继承重写方法 js方法的重写原生javascript事件

文章版权归作者所有，未经允许请勿转载,若此文章存在违规行为，您可以联系管理员删除。

转载请注明本文地址：https://www.ucloud.cn/yun/101006.html

JavasScript重难点知识

摘要：忍者级别的函数操作对于什么是匿名函数，这里就不做过多介绍了。我们需要知道的是，对于而言，匿名函数是一个很重要且具有逻辑性的特性。通常，匿名函数的使用情况是创建一个供以后使用的函数。 JS 中的递归递归, 递归基础, 斐波那契数列, 使用递归方式深拷贝, 自定义事件添加这一次，彻底弄懂 JavaScript 执行机制本文的目的就是要保证你彻底弄懂javascript的执行机制，如果...

forsigner 2019-08-21 18:36 评论0 收藏0
总结移动开发实践中遇到的坑

摘要：博主之前已经推荐了一款神器下面，就总结一下移动端遇见的坑。解决原理虚拟键盘弹出时将元素设置为，虚拟键盘消失时候设置回来。解决方案由于虚拟键盘出现并未抛出事件，而检测或者事件，皆会有一定延迟，会出现闪烁现象。做过很多移动端的项目，在开发调试过程中，一款好的调试工具会让效率大大提高。博主之前已经推荐了一款神器：http://web.jobbole.com/87587/ 下面，就总结一下移...

rockswang 2019-08-29 11:16 评论0 收藏0
2016年终工作总结

摘要：由于初版需求及开发工作都没有参与，在接手项目后过了遍前端结构发现所有交互及组件都是现撸，并未使用市面上已有的优秀前端框架从我个人角度理解上出发，后续需求变更中当需要实现某些常用组件样式或交互时，基本上都需要现撸或者寻找合适的组件。 2016悄无声息的过去了，再过不久便是农历新年这几天相对清闲梳理了一下去年所做的工作，希望在新的一年能发展的更好今年一共研发或升级了五款产品：合伙人、夺...

hoohack 2019-06-27 15:14 评论0 收藏0
FEDAY2016之旅

摘要：前戏补上参会的完整记录，这个问题从一开始我就是准备自问自答的，希望可以通过这种形式把大会的干货分享给更多人。 showImg(http://7xqy7v.com1.z0.glb.clouddn.com/colorful/blog/feday2.png); 前戏 2016/3/21 补上参会的完整记录，这个问题从一开始我就是准备自问自答的，希望可以通过这种形式把大会的干货分享给更多人。 ...

red_bricks 2019-08-19 17:55 评论0 收藏0
移动端开发（使用webuploader上传图片,客户端交互,修改alert弹窗等）

摘要：之前实习做的一个移动端的页面需要的功能有图片上传点击客户端的返回按钮有提示即与客户端有交互遇到不少的坑总结一下问题图片上传功能使用工具百度的暂时遇到的坑删除图片实际上并没有完全删除需要自己在源码上添加详情看的提问上传的图片旋转角度有问题比之前实习做的一个移动端的页面需要的功能有图片上传点击客户端的返回按钮有提示(即与客户端有交互）遇到不少的坑总结一下问题 1.图片上传功能 ...

李昌杰 2019-08-19 17:26 评论0 收藏0