передать событие другой функции

Question

var that = this; 

for(var i=0;i<x;++i){ 
// some code 
events={ 
    click: function(event){ 
    that.doClick(event,this); 
} 
} 
} 

Теперь с приведенным выше кодом я получаю ошибку JShint «Не выполняйте функции в цикле».

Для устранения указанной выше ошибки я делаю это сейчас

var that = this; 
    function clickHandler() { 
       return function() { 
        that.doClick(this); 
       }; 
      } 

for(var i=0;i<x;++i){ 
// some code 
events={ 
    click: clickHandler() 
} 
} 

ли этот штраф? Если да, то как я могу передать событие из события щелчка в clickHandler, а затем сделать doClick funciton

Answer 1

Нет, это не нормально. Проблема в создании функций внутри цикла заключается в том, что вы создаете новую копию функции на каждой итерации.

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

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

var that = this; 
function clickHandler(event) { 
    that.doClick(event, this); 
} 
for(var i=0; i<x; ++i){ 
    var events = {click: clickHandler}; 
} 

Однако, поскольку events, кажется, не зависит от i, вы можете переместить его снаружи:

var that = this, 
    events = {click: function(event){ 
     that.doClick(event, this); 
    }}; 
for(var i=0; i<x; ++i){ 
    /* Use `events` here */ 
}