How to create a unique eventlistener for a label/button in Actionscript in a for loop?

Question

At the moment I have this in Actionscript (for Flex) to create 26 labels in a BorderContainer (ID letters):

var lettersArray:Array = ['A', 'B', 'C', 'D', 'E', 'F', 'G', 'H', 'I', 'J', 'K', 'L', 'M', 'N', 'O', 'P', 'Q', 'R', 'S', 'T', 'U', 'V', 'W', 'X', 'Y', 'Z'];

for(var indexLetter:String in lettersArray) {
    var labelLetter:Label = new Label();
    labelLetter.text = lettersArray[indexLetter];
    labelLetter.addEventListener(MouseEvent.CLICK, function():void { getUsers(lettersArray[indexLetter]); });

    letters.addElement(labelLetter);
}

But when I click on a letter, only the letter Z is sent to the function getUsers. So when I'm creating labels from a for loop, only the eventlistener works for the last created label. How do I fix this?

Richard Walton · Accepted Answer · 2012-02-18T09:24:42.497

2

I'm assuming that getUsers(lettersArray[indexLetter]) is an error as you can't access array elements like that

var lettersArray:Array = ['A', 'B', 'C', 'D', 'E', 'F', 'G', 'H', 'I', 'J', 'K', 'L', 'M', 'N', 'O', 'P', 'Q', 'R', 'S', 'T', 'U', 'V', 'W', 'X', 'Y', 'Z'];

for each(var letter:String in lettersArray) {
    var labelLetter:Label = new Label();
    labelLetter.text = letter;
    labelLetter.addEventListener(MouseEvent.CLICK, function():void { getUsers(e.currentTarget.text); });

    letters.addElement(labelLetter);
}

edited Feb 18 '12 at 09:24

answered Feb 17 '12 at 14:04

Richard Walton

4,789
3
38
49

Finally! Thank you! I've added _e:MouseEvent_ to the function. _getUsers(lettersArray[indexLetter])_ was always 'Z'. – Harmjan Feb 17 '12 at 14:09
Why not simply **labelLetter.text = indexLetter;** – sch Feb 17 '12 at 14:36
I copied the example from the original question and only skimmed it to change the relevant part :) Fixed now. – Richard Walton Feb 18 '12 at 09:24

How to create a unique eventlistener for a label/button in Actionscript in a for loop?

1 Answers1