Cant использовать замыкание в качестве слушателя событий для PushState

Question

Я хочу использовать замыкание как EventListener (им работать в библиотеке маршрутизации), поэтому я создал класс, который делает именно это назначение

'use strict'; 
var RouteManager = RouteManager || {}; 

RouteManager.OnClickDelegate = function(UrlHandler) 
{ 
    var handler = UrlHandler; 

    return function delegate(e) 
    { 
     e = e || window.event; 
     var element = e.target || e.srcElement; 
     if (element.tagName == 'A') { 
      console.log("link click"); 
      window.history.pushState({},"",e.href); 
      //this.handler.handle(element.href); 
      return false; 
     } 
    }; 
}; 

:

document.onclick = RouteManager.OnClickDelegate(new UrlHandler()); 

Однако, несмотря на назначение закрытия в качестве слушателя событий работает («ссылка нажмите» регистрируются правильно) состояние разве толкнул

сделать дела даже незнакомым, это работает правильно:

document.onclick = (function(UrlHandler){ 
    var uh = UrlHandler; 
    return function delegate(e) 
    { 
     e = e || window.event; 
     var element = e.target || e.srcElement; 
     if (element.tagName == 'A') { 
      console.log("link click"); 
      window.history.pushState({},"",e.href); 
      //this.handler.handle(element.href); 
      return false; 
     } 
    }; 
})(new UrlHandler()); 

Любые идеи?

Кроме того, пожалуйста, не предлагают библиотеку работать маршрутизации, его не то, что им просят

Answer 1

состояние разве толкнул

Вероятно потому, что e.href является undefined. Вы хотите element.href.

---

Обратите внимание, что в вашем закрытии вы можете использовать переменную параметра напрямую, вам не нужно это объявление локальной переменной. Просто используйте это:

RouteManager.OnClickDelegate = function(handler) { 
    return function delegate(e) { 
     e = e || window.event; 
     var element = e.target || e.srcElement; 
     if (element.tagName == 'A') { 
      console.log("link click"); 
      window.history.pushState({}, "", element.href); 
      handler.handle(element.href); 
      return false; 
     } 
    }; 
};