Accessing mutable variable in an event closure
how to write a closure where I am competing the event parameter in the response
Use a closure either around the whole loop body (as @dandavis) demonstrated), or use it only around the handler:
…
Mousetrap.bind(
[ 'command+' + iKey,
'command+' + iKeyUpper,
'ctrl+' + iKey,
'ctrl+' + iKeyUpper],
(function(_i) { // of course you can use the name `i` again
return function( e ) {
console.log( "you clicked: " + _i );
};
})(i) // and pass in the to-be-preserved values
);
you need to wrap the i variable in a local scope so that it won't sync with the "i" in the for loop:
var keys = [ 'b', 'i', 'u'];
for (var i=0; i < 3; ++i) {
(function(i){
var iKey = keys[i];
var iKeyUpper = iKey.toUpperCase();
Mousetrap.bind(
[ 'command+' + iKey,
'command+' + iKeyUpper,
'ctrl+' + iKey,
'ctrl+' + iKeyUpper],
( function( e ) {
console.log( "you clicked: " + i );
} ) );
}(i));
}
the other alternative is to use functional Array methods, which since they use functions, always have their own scope, and they provide the element value and element index to you intrinsically:
var keys = [ 'b', 'i', 'u'];
keys.map(function(iKey, i){
var iKeyUpper = iKey.toUpperCase();
Mousetrap.bind(
[ 'command+' + iKey,
'command+' + iKeyUpper,
'ctrl+' + iKey,
'ctrl+' + iKeyUpper],
function( e ) {
console.log( "you clicked: " + i );
}); // end bind()
}); // end map()
the 2nd example will work out of the box in IE9+, but you can make it work anywhere with a simple drop-in Array method compat pack, usually included in IE shims...