menu activo en as3

Hola, me gustaría me comentaran mi script ya que no estoy seguro si es la mejor forma de realizarlo, me funciona pero no estoy seguro si estoy limpiando bien los "Listener" o si esta muy rebuscado. mu chas gracias


//Estoy creando un menu en flash con script en la linea de tiempo



tengo un clip en el scenario llamado "logo_mc" en su interior ay otro clip llamado "menu" el cual esta conformado por otros clips que representan las secciones de un menu, y mi script es este"

[as]
//inicio
var menu:Array = new Array();
menu[0] = logo_mc.menu_mc.servi_mc;
menu[1] = logo_mc.menu_mc.reel_mc;
menu[2] = logo_mc.menu_mc.wor_mc;
menu[3] = logo_mc.menu_mc.contac_mc;
for (var f in menu)
{
menu[f].addEventListener(MouseEvent.ROLL_OVER,over,false);
menu[f].addEventListener(MouseEvent.ROLL_OUT,out,false);
menu[f].addEventListener(MouseEvent.CLICK,sec,false);
}
//funciones

function over(e:MouseEvent):void
{
e.target.gotoAndStop(2);
e.target.buttonMode = true;
}
//
function out(e:MouseEvent):void
{
e.target.buttonMode = false;
e.target.gotoAndStop(1);
}
//

//secciones

function sec(event:MouseEvent):void
{
switch (event.currentTarget.name)
{
case "servi_mc" :
servicios(event);
break;
case "reel_mc" :
rell(event);
break;
case "contac_mc" :
contacto(event);
break;
case "empleo_mc" :
rell(event);
break;
case "wor_mc" :
work(event);
break;
}
}
//
function servicios(event:MouseEvent)
{
for (var f in menu)
{
menu[f].addEventListener(MouseEvent.ROLL_OVER,over,false);
menu[f].addEventListener(MouseEvent.ROLL_OUT,out,false);
menu[f].addEventListener(MouseEvent.CLICK,sec,false);
menu[f].buttonMode = true;
menu[f].gotoAndStop(1);
}
logo_mc.menu_mc.servi_mc.gotoAndStop(3);
logo_mc.menu_mc.servi_mc.removeEventListener(MouseEvent.ROLL_OUT,out);
logo_mc.menu_mc.servi_mc.removeEventListener(MouseEvent.ROLL_OVER,over);
logo_mc.menu_mc.servi_mc.removeEventListener(MouseEvent.CLICK,sec);
logo_mc.menu_mc.servi_mc.buttonMode = false;
trace("servicios");
}
//
//
function rell(event:MouseEvent)
{
for (var f in menu)
{
menu[f].addEventListener(MouseEvent.ROLL_OVER,over,false);
menu[f].addEventListener(MouseEvent.ROLL_OUT,out,false);
menu[f].addEventListener(MouseEvent.CLICK,sec,false);
menu[f].buttonMode = true;
menu[f].gotoAndStop(1);
}
logo_mc.menu_mc.reel_mc.gotoAndStop(3);
logo_mc.menu_mc.reel_mc.removeEventListener(MouseEvent.ROLL_OUT,out);
logo_mc.menu_mc.reel_mc.removeEventListener(MouseEvent.ROLL_OVER,over);
logo_mc.menu_mc.reel_mc.removeEventListener(MouseEvent.CLICK,sec);
logo_mc.menu_mc.reel_mc.buttonMode = false;
trace("reel");
}
//etc
{/as}
//espero se entienda gracias

Te recomiendo jamas usar el timeline para cambiar la interfaz de usuario. el timeline solo debe usarse para animacion, de lo contrario te puede llegar a dar dolores de cabeza muy grandes y codigos feos en muchos casos. Se podría ver mejor el codigo si no usaras el timeline y fuera mas orientado a objetos.

Muchas gracias por tu opinión. lo tendré en cuenta.