Can't trace an index of an array

[daz1761]

I'm not sure if its just getting to late in the day, but basically I copied some code over to a similar function, which traces out the array index number using indexof(e.target). For some reason I'm getting the usual undefined property errors, but it is tracing other properties in the array using e.target?

Heres my code in the document class:

var _soloButtons:Array;

            _soloButtons = new Array();

            var count3:uint = 4;
            
            var _soloIndex:soloButton;

            for (var iii:int = 0; iii < count2; iii++)
            {
                _soloIndex = new soloButton();
                _soloIndex.x = soloPosX;
                _soloIndex.y = soloPosY;
                _soloIndex.addEventListener(MouseEvent.CLICK, soloTrack);
                soloPosY = soloPosY + 34.9;
                _soloIndex.soloONOFF = soloONOFF;
                _soloButtons.push(_soloIndex);
                addChild(_soloIndex);
            }

And function:

public function soloTrack(e:MouseEvent):void
        {
            trace("solo");
            trace(e.target.soloONOFF);
            trace(_soloButtons.indexOf(e.target)); //this is the line it rejects??
        }

Thanks

[PBFrO]

Instead of e.target, try e.currentTarget and see if it works. Current target applies to the item that has the event listener.

[adaminaudio]

Hi,

It might be possible that your event.target is not in your array. Your movieclip might have children that are becoming event.target.

What happens when you trace event.currentTarget ?

Maybe try adding the handlers via the array

for(var i:uint =0; i<_soloButtons.length; i++){
      
     var soloButton:SoloButton = _soloButtons[i] as SoloButton;
     soloButton.addEventListener(MouseEvent.CLICK, soloTrack);
   //a for-each could also work  

}

just a thought maybe....not sure if that would even help. You're right, it is late I'll just wait til the smarter guys give you a better answer...

-ADam

[daz1761]

Thanks guys

I've tried some of your suggestions but no luck sadly, I don't understand as the other 2 trace statements are tracing ok. Even the e.target.soloONOFF, which is simple a boolean property. And I'm using the exact same process near enough in my other custom class I'm using and it works a treat.

The error is 1120: Access of undefined property _soloButtons?

Cheers

[Poony]

An other approach would be to give all items a variable called e.g "myId".

And when you create the objects just assign it, also set mouseChildren to false to avoid any children becoming the e.target:

_soloIndex.myId = i;
_soloIndex.mouseChildren = false;

Then you would just trace

e.target.myId

Hope it helps!

[daz1761]

This has escalated even further now...

I can't get anywhere near my array index for properties, I just keep getting 1120 access of undefined property of _muteButtons and _soloButtons!!

Also, I'm getting an error flashing up quickly saying migration error or something before the .swf file loads after adding gotoAndPlay???

This is what I'm trying to trace and change to test if the function is working:

Document class:

_muteButtons = new Array();

            var count2:uint = 4;

            var _muteIndex:muteButton;

            for (var ii:int = 0; ii < count2; ii++)
            {
                _muteIndex = new muteButton();

                with (_muteIndex)
                {
                    x = mutePosX;
                    y = mutePosY;
                    buttonMode = true;
                    gotoAndStop("1"); //migration error
                    addEventListener(MouseEvent.CLICK, muteTrack);
                }
                mutePosY = mutePosY + 34.9;
                _muteIndex.muteONOFF = muteONOFF;
                _muteButtons.push(_muteIndex);
                addChild(_muteIndex);
            }

            var _soloButtons:Array;

            _soloButtons = new Array();

            var count3:uint = 4;

            var _soloIndex:soloButton;

            for (var iii:int = 0; iii < count2; iii++)
            {
                _soloIndex = new soloButton();

                with (_soloIndex)
                {
                    x = soloPosX;
                    y = soloPosY;
                    buttonMode = true;
                    gotoAndStop("start"); //migration error??
                    addEventListener(MouseEvent.CLICK, soloTrack);
                }
                soloPosY = soloPosY + 34.9;
                _soloIndex.soloONOFF = soloONOFF;
                _soloButtons.push(_soloIndex);
                addChild(_soloIndex);
            }

        }

        public function muteTrack(e:MouseEvent):void
        {
            if (e.target.muteONOFF == true)
            {
                _muteButtons[0].trackVolume = 0; //doesnt work
                trace("mute");
                trace(e.target.muteONOFF);
            }
            else
            {
                trace("no luck");
            }
        }

        public function soloTrack(e:MouseEvent):void
        {
            trace("solo");
            trace(e.target.soloONOFF);
            trace(_soloButtons.indexOf(e.currentTarget)); //doesnt work
        }
    }

The buttons are MovieClip symbols to and they trace fine at the end of each for loop

[object soloButton],[object soloButton]
[object soloButton],[object soloButton],[object soloButton]
[object soloButton],[object soloButton],[object soloButton],[object soloButton]

[northmantif]

did you declare your _soloButtons and _muteButtons array object outside constructor?Other way these two dies as fast as running code exits constructor's {} curly braces.
From provided piece of code your _soloButtons variable is a local variable which does exactly what I wrote above.

[daz1761]

Hi, yes I've just realised I didn't declare them outside of the constructor grr stupid me

I'm still getting a wierd error flashing up quickly within the output window for my gotoAndStop methods? It's working but I don't like the fact that there is 2 errors popping up saying something about migration? Am I not quite directing the method to the right distination?

Thanks for spotting the obvious error

Here is my document class if it helps:

package 
{

    import flash.display.*;
    import flash.events.*;
    import fl.controls.*;

    public class documentClass extends MovieClip
    {
        var _tracks:Array = new Array();
        var _muteButtons:Array = new Array();
        var _soloButtons:Array = new Array();
        var _muteIndex:muteButton;
        var _soloIndex:soloButton;
        var muteONOFF:Boolean = true;
        var soloONOFF:Boolean = true;
        var count:uint = 4;
        var count2:uint = 4;
        var count3:uint = 4;
        var PosY:uint = 10;
        var comboY:uint = 30;
        var volDownY:uint = 41;
        var volUpY:uint = 38;
        var panLeftY:uint = 30;
        var panRightY:uint = 47;
        var mutePosX:Number = 350;
        var mutePosY:Number = 32;
        var soloPosX:Number = 375;
        var soloPosY:Number = 32;

        public function documentClass()
        {
            // constructor code

            trace("its working");

            for (var i:int = 0; i < count; i++)
            {
                var _trackIndex:_channelStrip = new _channelStrip(390,PosY,20,comboY,260,volDownY,277,volUpY,155,panLeftY,159,panRightY);
                PosY = PosY + 35;
                comboY = comboY + 35.2;
                volDownY = volDownY + 35;
                volUpY = volUpY + 35;
                panLeftY = panLeftY + 35;
                panRightY = panRightY + 35;
                _tracks.push(_trackIndex);
                addChild(_trackIndex);
                trace(_tracks);
            }

            for (var ii:int = 0; ii < count2; ii++)
            {
                _muteIndex = new muteButton();

                with (_muteIndex)
                {
                    x = mutePosX;
                    y = mutePosY;
                    buttonMode = true;
                    gotoAndStop("1"); // Errors
                    addEventListener(MouseEvent.CLICK, muteTrack);
                }
                mutePosY = mutePosY + 34.9;
                _muteIndex.muteONOFF = muteONOFF;
                _muteButtons.push(_muteIndex);
                addChild(_muteIndex);
                trace(_muteButtons);
            }

            for (var iii:int = 0; iii < count2; iii++)
            {
                _soloIndex = new soloButton();

                with (_soloIndex)
                {
                    x = soloPosX;
                    y = soloPosY;
                    buttonMode = true;
                    gotoAndStop("start"); // Errors
                    addEventListener(MouseEvent.CLICK, soloTrack);
                }
                soloPosY = soloPosY + 34.9;
                _soloIndex.soloONOFF = soloONOFF;
                _soloButtons.push(_soloIndex);
                addChild(_soloIndex);
                trace(_soloButtons);
            }

        }

        public function muteTrack(e:MouseEvent)
        {
            if (e.target.muteONOFF == true)
            {
                trace(_muteButtons.indexOf(e.target));
                trace("mute");
                trace(e.target.muteONOFF);
            }
            else
            {
                trace("no luck");
            }
        }

        public function soloTrack(e:MouseEvent)
        {
            trace("solo");
            trace(e.target.soloONOFF);
            //trace(_soloButtons.indexOf(e.target));
        }
    }
}

[northmantif]

does gotoAndStop("1"); where "1" is a string name for the frame label or you rather want to gotoAndStop(1)... I just glimpsed at the code, so I can be wrong, just a thought...

[ASWC]

I would get rid of 'with'. That's one of those keyword that sounds awesome but turn out in many cases to be just plain evil. Never use it or be prepared for a whole lot of headaches.