Bonjour à tous,
 
j'ai un petit souci avec le passage de variable utilisé au sein d'une fonction: Du fait du "closure" (je ne connais pas le terme Français), je n'obtiens pas la variable escompté.  
 
Je vais donner un exemple simplifié:
 
Imaginons les structures suivantes:

var actions = {
    'UP': 1,
    'DOWN': 2,
    'LEFT': 3,
    'RIGHT': 4
};
 
var buttons = ['UP', 'DOWN', 'LEFT', 'RIGHT'];

Si je procède ainsi:

for (var b in buttons) {
    var index = actions[buttons[b]];
    bindKey(buttons[b], function () { // bindKey() est une fonction que j'ai créer qui me permet d'associer à une touche un callback...
        alert(index);
    });
}

Quand j'appuie sur les touches UP, DOWN, LEFT et RIGHT, alert(index) m'affiche toujours undefined car index c'est fait modifier entre temps (sorte de variable global dû au "closure"...)
 
Par contre, si je procède ainsi:

function bindKey2(button, index){
    bindKey(button, function () {
        alert(index);
    });
}
 
for (var b in buttons) {
    var index = actions[buttons[b]];
    bindKey2(buttons[b], index);
}

Dans ce cas, alert(index) m'affiche bien 1, 2, 3, 4 et non undefined. Ceci est du au fait que en appelant la fonction bindKey2, les variables passé en paramètre sont des instances de ces variable et non des références aux variables global.
 
Ma question est la suivante: Comment écrire ceci sans passer par une fonction intermédiaire (bindKey2 dans mon exemple)?

si c était juste un problème de closure t aurais pas 4 fois "4" plutot ?
je pense plutot que comme ta callback se lance sur un event ca décale ton scope
t as essayé de déclarer index au meme niveau que bindKey ?

à tester

for (var b in buttons)
{
    var index = actions[buttons[b]];
    bindKey.call (buttons[b], buttons[b], function () { alert (this) });
}

Ca marche pas
Tu as le code de ton bindKey ?

Bon ca marche comme ca avec une vrai closure :

for (var b in buttons)
{
    var index = actions[buttons[b]];
    bindKey (buttons[b], (function () { var cIndex = index; return function () { alert (cIndex) } } ) () );
}

cIndex devient une sorte de variable statique et c'est elle qu'on affiche plutot qu'index, qui a évolué entre l'instanciation de la fonction et son affichage.

version plus lisible

var f = function () { var cIndex = index; return function () { alert (cIndex) } };
for (var b in buttons)
   {
    var index = actions[buttons[b]];
    bindKey (buttons[b], f () );
   }

Je me disais bien que la réponse allait être évidente. Suffisait d'y penser
 
Merci bien

De rien
Ca m'a permis de me rendre compte que je passais souvent bcp de temps à gérer le scope à coups de call et apply, alors que dans la majorité des cas, une bonne closure fait l'affaire

Pour que ce soit un peu plus claire dans ma tête j'ai externalisé la closure de la manière suivante:

for (var b in buttons) {
    (function () { // we create a closure in order to keep the current value of index at the time of the loop iteration
        var index = actions[buttons[b]];
        bindKey(buttons[b], function () {
            alert(index);
        });
    })();
}

Merci encore