Мне нужна кросс-браузерная функция для регистрации обработчиков событий и (в основном) согласованный опыт работы с обработчиками. Мне не нужен полный вес или функциональность такой библиотеки, как jQuery, поэтому я написал свою собственную. Я считаю, что достиг своих целей с помощью приведенного ниже кода, и до сих пор мое тестирование было успешным, но я слишком долго смотрел на него. Есть ли недостатки в моей логике или ошибки, которые я упускаю?
РЕДАКТИРОВАТЬ 1: каждый блок прокомментирован с намерением браузера для ясности. Обновлен блок IE, чтобы не вызывать func
сразу (благодаря зоркому глазу Энди Э.).
EDIT 2: Обновлен блок IE для вызова func.call()
с this
вместо elem
.
РЕДАКТИРОВАНИЕ 3: Обновлено для передачи JSLint с "Хорошими частями".
function hookEvent(elem, evt, func)
{
if (typeof elem === "string")
{
elem = document.getElementById(elem);
}
if (!elem)
{
return null;
}
var old, r;
if (elem.addEventListener) //w3c
{
elem.addEventListener(evt, func, false);
r = true;
}
else if (elem.attachEvent) //ie
{
elem[evt + func] = function ()
{
func.call(this, window.event);
};
r = elem.attachEvent("on" + evt, elem[evt + func]);
}
else //old
{
old = elem["on" + evt] ? elem["on" + evt] : function (e) { };
elem["on" + evt] = function (e)
{
if (!e)
{
e = window.event;
}
old.call(this, e);
func.call(this, e);
};
r = true;
}
return r;
}
addEventListener
, ниattachEvent
. - person Tim Down   schedule 26.08.2010