Discussion :

[agh]

Bonjour à tous,

Je m'arrache les cheveux :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
for (var i = 0 ; i < 3 ; i++)
{
        setInterval (function ()
        {
            toto(i) ;
        } , 1000) ;
}
 
function toto (i)
{
     console.log (i) ;
}

Voilà un code tout bête... Qui selon ma logique devrait afficher :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
0
1
2

toutes les secondes...
Or, ça m'affiche :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
3
3
3

toutes les secondes.

Je vois d'où vient le problème :

La boucle est d'abord parcourue en entier, jusqu'à ce que i=3.
Puis, le setInterval est fait (avec un i=3)...
MAIS ? Comment faire pour avoir le comportement attendu ?

Je vous remercie beaucoup

[Invité]

En faisant un wrapper appelé immédiatement pour mémoriser l'index

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
for (var i = 0 ; i < 3 ; i++)
{
    (function(i)
    {
        setInterval (function ()
        {
            toto(i) ;
        } , 1000) ;
    })(i);
}
 
function toto (i)
{
    console.log (i) ;
}

D'ailleurs je trouve ça plus lisible comme suit, mais bon, question de goût^^

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
for (var i = 0; i < 3; i++) {
  (function(i) {
    setInterval(function() {
      toto(i);
    }, 1000);
  })(i);
}
 
function toto(i) {
  console.log(i);
}

[p3ga5e]

Salut agh !

Enerian a très bien répondu à ta question. Toutefois si ton but est de trouver le meilleur moyen de repartir la charge CPU dans un même processus Node, je te conseille d’utiliser process.nextTick à la place de setInterval ou d’utiliser la fonction eachLimit du célèbre module async .

[Kaamo]

C'est un problème récurrent en javascript.
En effet, i dans la fonction anonyme de setInterval ne sera évalué que lorsque celle-ci sera appelée.
Donc, lors de l'appel de cette fonction (au bout d'une seconde en l'occurrence), i vaudra déjà son max (2 en l'occurrence).
Pour avoir le comportement voulu, il faut "enfermer" la variable i au moment de son inclusion dans la fonction anonyme. Comme ci-dessus, ou de cette façon en utilisant un "créateur" de fonction qui "ferme" le scope à chaque tour de boucle :

Code javascript :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
for (var i = 0 ; i < 3 ; i++){
  setInterval(creerToto(i), 1000);
}
 
function creerToto(i) {
  return function() { toto(i); };
}
 
function toto (i) {
  console.log (i) ;
}

Mais pourquoi mettre le setInterval dans la boucle ? Je ferais plutôt le contraire !

Code javascript :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
setInterval(function() {
  for (var i = 0 ; i < 3 ; i++){
    console.log(i);
  }
}, 1000);

edit : j'avais zapé Node. +1 @p3ga5e

[agh]

Merci beaucoup pour vos réponses !