Problème de valeur de variables

Bonsoir,

J’utilise jQuery. J’aimerais appliquer une fonction liée à un évenement à un grand nombre d’élément, et pour cela, j’utilise une boucle. Mon code est le suivant:

 1
 2
 3
 4
 5
 6
 7
 8
 9
10

for (var i = 1; i < 9; i++) {
        for (var j = 0; j < 9; j++) {
            $('#' + parseInt(i) + parseInt(j)).keypress(function(e) {
                switch (e.keyCode) {
                    // Ici i vaut toujours 9
                }
            });
        }
    }
});

Mon souci est le suivant: dans la fonction appelée par l’événement, i et j ne correspondent pas aux valeurs que je souhaiterais. Après avoir un peu cherché, j’ai trouvé qu’il fallait apparemment utiliser des closures. J’ai essayé, mais sans succès.

Si quelqu’un pouvait m’aider… Merci !

Lorsque je souhaite avoir une valeur fixe par element, j’utilise la fonction "data" de jQuery.

Donc tu n’auras plus besoin d’appeler keypress dans une boucle, mais juste fixer la valeur.

1
2
3
4
5
6
7
8

$("div.maclass").keypress(fonction(e) {
  $(this).data("i"); // toujours i
});
for (var i = 1; i < 9; i++) {
  for (var j = 0; j < 9; j++) {
    $('#' + parseInt(i) + parseInt(j)).data("i", i);
  }
}

Modifier le DOM pour ça, c’est (très) loin d’être optimisé.

Autant savoir plus précisément ce qu’il y a à faire dans cet événement :

• est-ce qu’il y a vraiment besoin de ces variables i et j ?
• pourquoi appliquer autant de listeners quand on pourrait déléguer ?
• autant utiliser une class pour récupérer nos éléments plutôt que des id (pas besoin de boucle, donc) comme l’indique victor
• à quoi sert parseInt quand on sait d’avance que la variable est un entier et qu’en plus on va la caster en string ?

Tu es sûr que data modifie le DOM ?

http://www.nicoespeon.com/en/2013/05/jquery-data-and-performance/

EDIT :

Voici un exemple : http://jsbin.com/nuqadurozu/edit?html,js,output

My bad. J’ai banni jQuery de tous mes projets, j’ai pas suivi les nouveautés.

Dans tous les cas je suis pas bien sûr que ça ait un intérêt ici : il y a déjà des variables déclarées, ça sert à rien d’en créer d’autres.

Je ne sais pas, c’est ce que j’ai cru comprendre dans son problème. En effet, il peut récupérer directement l’id puis utiliser une expression régulière et le tour est joué.

Mais cette portion de code est peut-être simplement un exemple d’un problème/utilisation plus complexe.

Une expression régulière ?!

Dans son exemple il veut récupérer i, la dizaine, il y a plusieurs solutions pour y arriver.

Effectivement. La question est donc : y a-t-il une pire solution qu’une expression régulière pour y arriver ?

Au final, j’ai effectivement rajouté une classe à mes cases pour pouvoir toutes les sélectionner avec jQuery, et je récupère l’id avec la fonction attr() de jQuery. Ça change la façon dont j’attaquais le problème lors de mon post initial; ce qui m’embêtais vraiment à ce moment, c’est que je n’arrivais pas à avoir les valeurs correctes de i et j dans ma callback…