Node.js et setInterval et boucle

Version imprimable

11/04/2013, 13h44
agh

Node.js et setInterval et boucle
Bonjour à tous,

Je m'arrache les cheveux :
Code:

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:

1 2 3 0 1 2
toutes les secondes...
Or, ça m'affiche :
Code:

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

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

Code:

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:

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);
}

11/04/2013, 14h38
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 .
11/04/2013, 14h44
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:

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:

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
12/04/2013, 09h51
agh

Merci beaucoup pour vos réponses !