Добавление кросс-браузера прослушивателя событий

моя проблема здесь: почему этот код не работает?????... сообщение не отображается

Я пытаюсь добавить прослушиватель событий к элементу x при нажатии события

    function test()
    {
    alert("test");
    }

var EventsCrossBrowsers = 
 {
    addEvents:(function(element,event,func)
              {
                    if(element.addEventListener)
                    {
                        return elemenet.addEventListener(event,func,false);
                    }
                    else if(elemenet.attachEvent)
                    {
                        return elemenet.attachEvent("on"+event,func);
                    }
              }());
 }

 var x =document.getElementById("test");

EventsCrossBrowsers.addEvents(x,"click",test);

спасибо alot jfriend00.... Я думаю, самый лучший способ: -

function test()
{
    alert("test");
}

function addEventsCrossBrowsers(elemenet,event,func)
{
    if(elemenet.addEventListener)
    {
        elemenet.addEventListener(event,func,false);
    }
    else if(elemenet.attachEvent)
    {
        elemenet.attachEvent("on"+event,func);
    }
}
var x =document.getElementById("test");
addEventsCrossBrowsers(x,"click",test);

ваш второй способ почти такой же, за исключением того, что я не понимаю возврата... еще раз спасибо...

ОТВЕТЫ

Ответ 1

В вашей функции я вижу, что вы используете как elemenet, так и element, где они должны быть одинаковыми. Это будет по крайней мере частью проблемы.

Я также вижу, что ваша функция addEvents - это функция самоисполнения, которая не имеет смысла в этом отношении. Похоже, что это должна быть нормальная функция.

Здесь функция кросс-браузера. Помимо создания одной функции для добавления обработчиков событий, она также нормализует указатель this и объект events, чтобы их можно было обрабатывать одинаково в любом браузере.

// add event cross browser
function addEvent(elem, event, fn) {
    // avoid memory overhead of new anonymous functions for every event handler that installed
    // by using local functions
    function listenHandler(e) {
        var ret = fn.apply(this, arguments);
        if (ret === false) {
            e.stopPropagation();
            e.preventDefault();
        }
        return(ret);
    }

    function attachHandler() {
        // set the this pointer same as addEventListener when fn is called
        // and make sure the event is passed to the fn also so that works the same too
        var ret = fn.call(elem, window.event);   
        if (ret === false) {
            window.event.returnValue = false;
            window.event.cancelBubble = true;
        }
        return(ret);
    }

    if (elem.addEventListener) {
        elem.addEventListener(event, listenHandler, false);
        return {elem: elem, handler: listenHandler, event: event};
    } else {
        elem.attachEvent("on" + event, attachHandler);
        return {elem: elem, handler: attachHandler, event: event};
    }
}

function removeEvent(token) {
    if (token.elem.removeEventListener) {
        token.elem.removeEventListener(token.event, token.handler);
    } else {
        token.elem.detachEvent("on" + token.event, token.handler);
    }
}

Если вам нужна более простая версия без опций распространения и предотвращения по умолчанию, но с нормализацией this и event, это будет следующим:

// add event cross browser
function addEvent(elem, event, fn) {
    if (elem.addEventListener) {
        elem.addEventListener(event, fn, false);
    } else {
        elem.attachEvent("on" + event, function() {
            // set the this pointer same as addEventListener when fn is called
            return(fn.call(elem, window.event));   
        });
    }
}