Problem with loop

Question

Possible Duplicate:
Javascript infamous Loop problem?

When a mousemove event is reaised the variable i is equal to last value(In my case = 4) for ALL sectors. Where i can store value of i?

for (var i = 0; i < pieChart.Sectors.length; i++) {
  pieChart.Sectors[i].mousemove(function (event) {
     var percent = (localData[i] * 100) / totalSum;
     pieChart.Popup(event.clientX, event.clientY, [percent, "% всего времени\n Было сделано", localData[i], "звонков"].join(' '));
  });
}

score 1 · Answer 1 · answered Jul 01 '11 at 15:23

1

You need a closure. See here for a nice explanation: http://www.mennovanslooten.nl/blog/post/62

I'll be posting your code modified shortly.

answered Jul 01 '11 at 15:23

Mrchief

73,270
19
138
185

score 1 · Answer 2 · answered Jul 01 '11 at 15:25

Answered like a thousand times, 1001 answers: Your event handler functions will close over the i variable. That means, all functions reference the same variable and therefore all of them have the identical value.

Solution: Introduce a new function(-context):

pieChart.Sectors[i].mousemove((function (myEvent) {
        return function() {
            var percent = (localData[i] * 100) / totalSum;
            // do something with "myEvent"
        };
}(event)));

score 0 · Answer 3 · answered Jul 01 '11 at 15:24

0

Check if you variable $i had been declared as a global variable before.

You can even check this with an alert(window.i); or console.log(window.i); at any time.

answered Jul 01 '11 at 15:24

monsieur_h

1,340
1
10
20

Problem with loop

3 Answers3

Linked