5

我正在研究一个跨浏览器的事件处理系统。我请一些开发人员检查我的代码。一位开发人员说我的实现是基于回调而不是真实事件。有什么区别？

为了您的方便（也作为要点），我在下面提供了我的实现的源代码。到目前为止，我还没有发现任何问题。它适用于我测试过的所有浏览器。

我很抱歉这个问题的描述不好，我不熟悉那个纯事件部分。

var evento = (function (window) {
  var win = window
    , doc = win.document
    , _handlers = {}
    , addEvent
    , removeEvent
    , triggerEvent;

  addEvent = (function () {
    if (typeof doc.addEventListener === "function") {
      return function (el, evt, fn) {
        el.addEventListener(evt, fn, false);
        _handlers[el] = _handlers[el] || {};
        _handlers[el][evt] = _handlers[el][evt] || [];
        _handlers[el][evt].push(fn);

      };
    } else if (typeof doc.attachEvent === "function") {
      return function (el, evt, fn) {
        el.attachEvent(evt, fn);
        _handlers[el] = _handlers[el] || {};
        _handlers[el][evt] = _handlers[el][evt] || [];
        _handlers[el][evt].push(fn);
      };
    } else {
      return function (el, evt, fn) {
        el["on" + evt] = fn;
        _handlers[el] = _handlers[el] || {};
        _handlers[el][evt] = _handlers[el][evt] || [];
        _handlers[el][evt].push(fn);
      };
    }
  }());

  // removeEvent
  removeEvent = (function () {
    if (typeof doc.removeEventListener === "function") {
      return function (el, evt, fn) {
        el.removeEventListener(evt, fn, false);
        Helio.each(_handlers[el][evt], function (fun) {
          if (fun === fn) {
            _handlers[el] = _handlers[el] || {};
            _handlers[el][evt] = _handlers[el][evt] || [];
            _handlers[el][evt][_handlers[el][evt].indexOf(fun)] = undefined;
          }
        });

      };
    } else if (typeof doc.detachEvent === "function") {
      return function (el, evt, fn) {
        el.detachEvent(evt, fn);
        Helio.each(_handlers[el][evt], function (fun) {
          if (fun === fn) {
            _handlers[el] = _handlers[el] || {};
            _handlers[el][evt] = _handlers[el][evt] || [];
            _handlers[el][evt][_handlers[el][evt].indexOf(fun)] = undefined;
          }
        });
      };
    } else {
      return function (el, evt, fn) {
        el["on" + evt] = undefined;
        Helio.each(_handlers[el][evt], function (fun) {
          if (fun === fn) {
            _handlers[el] = _handlers[el] || {};
            _handlers[el][evt] = _handlers[el][evt] || [];
            _handlers[el][evt][_handlers[el][evt].indexOf(fun)] = undefined;
          }
        });
      };
    }
  }());

  // triggerEvent
  triggerEvent = function (el, evt) {
    _handlers[el] = _handlers[el] || {};
    _handlers[el][evt] = _handlers[el][evt] || [];

    for (var _i = 0, _l = _handlers[el][evt].length; _i < _l; _i += 1) {
      _handlers[el][evt][_i]();
    }
  };

  return {
    add: addEvent,
    remove: removeEvent,
    trigger: triggerEvent,
    _handlers: _handlers
  };
}(this));
4

1 回答 1

2

我完全不在乎您的系统是基于回调、事件或 lambda 演算的。对于您在这里公开的内容，它似乎写得很好，并且承诺会做得很好（尽管我很好奇您是如何处理的removeEvent()；））。

但是，我对您的实施有几点意见：

  • 每次添加事件处理程序时，无需检查您在哪个浏览器上执行。

我很惊讶有多少人接受每次打电话时检查属性的存在的做法。没有人会在函数调用过程中将你的 IE 换成 FF（document.addEventListener如果你问我，任何愚蠢到定义一个实际的 ECMA-5 替换之外的属性的人都应该被鞭打死），所以检查你是哪个平台'在开始并完成它，如下所示：

if (doc.addEventListener) {
    addEvent  = // ...
    freeEvent = // ...
}
else if (doc.attachEvent) {
    addEvent  =  // ...
    freeEvent =  // ...
}
/* etc. */
  • 您提供了一个统一的接口来附加处理程序，但是根据您的代码将在其上执行的浏览器，实际处理程序的行为会有所不同。

例如，在 IE8- 中，事件目标的可用方式与 ECMA-5 约定中的方式不同。

如果你想提供一个真正的跨浏览器界面，你应该为事件处理程序提供一个统一的执行上下文。这可能包括一个“取消”功能，该功能将转换为：

cancel = function (e) { e.returnValue = false; }; // IE8-
cancel = function (e) { e.preventDefault();    }; // ECMA-5

IE8-下还应还原this到目标对象，并统一target语义event.target。

如果你真的想对程序员好一点，你也可以解决一些奇怪的问题，比如

  • 加载事件未在 IE8 中触发 - 当图像已被缓存时
  • 复杂得离谱的鼠标滚轮报告系统

可能还有其他几个。

我出于自己的目的这样做的方式是围绕实际处理程序生成一个包装器，它可以处理所有平台特性并在调用实际用户代码之前建立一致的执行上下文。

最后一句话：除了它的美感之外，我不太确定它是否仍然需要支持 Netscape4 风格的事件。但这是信仰问题，所以...

于 2013-12-30T01:05:27.923 回答