Dynamische Function Namen

[gurki]

ich will eine for schleife basteln die sowohl movieclips erstellt als ihnen auch eigene namen und functions inkl. event listener zuteilt

for (var i:Number=0; i<10; i++) {
	var my_mc = new MovieClip();
	my_mc.graphics.beginFill(0xFF0000);
	my_mc.graphics.drawRect(0, 0, 100, 80);
	my_mc.graphics.endFill();
	my_mc.name="mc"+i;
	my_mc.x=125*i-100;
	addChild(my_mc);
	my_mc.addEventListener(this["testFunction"+i]);
	function "testFunction"+i(event:MouseEvent):void {
	trace(i)
	}
}

bis jetzt hab ich es hinbekommen,dass die schleife die movieclips generiert (wäre jedoch schöner wenn er sie aus der bibliothek ziehen+ duplizieren würde, habe das noch nicht so hinbekommen)
bei der function jedoch bekomm ich allerlei compiler fehler =/
jemand ne idee wie man das problem lösen kann?

P.S.: beschäftige mich erst seit kurzem mit AS3 (hatte vorher aber einige AS2 erfahrungen gesammelt)

[Omega Psi]

Das funktioniert so nicht. Schaue dir mal die Signatur der addEventListener Methode an. Zumal du keinen Gewinn davon trägst, alles super dynamisch zu bauen.

[gurki]

hm ah okay, ja kleiner flüchtigkeitsfehler das ich den MouseEvent vergessen hatte my fault

my_mc.addEventListener( MouseEvent.CLICK,this["testFunction"+i]);
	function this["testFunction"+i](event:MouseEvent):void {
	trace(i)
}

funzt aber trotzdem nicht



das spätere endprodukt soll 40 dieser buttons umfassen(und niemand bastelt 40 mal die selbe funktion mit minimalen änderungen)

ne idee wie man das sonst generieren kann, ohne jede funktion einzeln zu machen?

[Martin Kraft]

Wieso arbeitest Du nicht einfach so:

PHP-Code:

my_mc.addEventListener( MouseEvent.CLICK, function(event:MouseEvent):void {
    trace(event.currentTarget)
}); 


Die Klammersyntax ist nicht typsicher. Und wenn Du die Funktion erst deklarierst nachdem sie an addEventListener übergeben wird ist this["testFunction"+i] dort null.

[Omega Psi]

1. Es hängt von den Änderungen ab. Was soll denn passieren?
2. Anonyme Funktionen sind nicht gerade das gelbe vom Ei, da sie im anonymous Namesapce liegen und man nicht intutiv die Fehler lesen kann.

[gurki]

ah okay event.currentTarget kannte ich noch nicht! ddas hilft mir schon um einiges weiter, danke =)


hab jetzt mal folgendes ausprobiert

trace(event.currentTarget.name.split("mc"))

ich will im grunde nur das er die zahl ausspuckt, die am ende des namens ist, jedoch bekomme ich ", 1" anstatt "1"

[Omega Psi]

var _clips:Array = [];
for (var i:int = 0; i< 10; i++)
{
  var clip:MovieClip = new MovieClip();
  clip.addEventListener(MouseEvent.CLICK, clickHandler);
  clip.graphics.beginFill(0xFF0000);
  clip.graphics.drawRect(0, 0, 100, 80);
  clip.graphics.endFill();
  clip.x=125 * i-100;
  _clips[i] = addChild(clip);
}

function clickHandler(event:MouseEvent):void
{
  trace(_clips.indexOf(event.currentTarget));
}

So in etwa. Es könnte noch ein Tippfehler drin sein.

[Martin Kraft]

Am saubersten wäre natürlich eine eigene Klasse für die Buttons zuschreiben und darin die gesamten Funktionalitäten zu kapseln.

[gurki]

@Omega Psi:

die buttons bekommen allerlei eventListener für dragNdrop, einige tweens(größer/kleiner werden) und werte bezüglich der position an ne variable+ array übertragen (alles schon getan, fehlt halt nur noch ne for schleife zum generieren)

EDIT:
zu deinem code:
funktioniert recht gut!ich glaube das weitere erübrigt sich
den rest sollte ich gebacken bekommen, falls nicht meld ich mich nochmal
vielen dank für die schnelle hilfe

[gurki]

hm okay habs jetzt soweit:

var _clips:Array = [];
var rectangle:Rectangle=new Rectangle(but1.x,330,0,-130);

for (var i:int = 0; i<= 10; i++)
{
  var clip:MovieClip = new MovieClip();
  clip.addEventListener(MouseEvent.MOUSE_DOWN, downHandler);
   clip.graphics.beginFill(0x000000);
  clip.graphics.drawCircle(10, 100, 10);
  clip.graphics.endFill();
  clip.name="mc"+i
  clip.x=50 * i-25;
  _clips[i] = addChild(clip);
}


function downHandler(event:MouseEvent):void
{
	this["mc"+_clips.indexOf(event.currentTarget)].startDrag(false, rectangle);
	//clicked=
  trace("mc"+_clips.indexOf(event.currentTarget));
  trace(event.currentTarget.name)
  }

nur klappt das irgendwie mitm ansprechen nicht ganz.
dachte die namen werden doch zugeteilt und sind dann ansprechbar?

[Martin Kraft]

Der name hat in AS3 nicht mehr zwingend etwas mit der lokalen Variable zu tun, die auf den MC zeigt. Das ist aber in der Regel auch gar nicht mehr nötig:

PHP-Code:

function downHandler(event:MouseEvent):void {
    var mc = event.currentTarget as MovieClip;
    if(mc){
        mc.startDrag(false, rectangle);
    }    
} 


Das ansprechen von DisplayObjekten über die Klammer-Syntax sollte man in AS3 vermeiden.

Aber nochmals mein Tip: Schau Dir mal Klassen an. Die wurden nämlich genau für das erfunden, was Du hier vorhast (viele Objekte mit gleichen Funktionen und verschiedenen Eigenschaftsausprägungen).

[Simse179]

Vielleicht so in etwa?

function downHandler(event:MouseEvent):void
{
    event.currentTarget.startDrag();
    trace(_clips.indexOf(event.currentTarget));
} 

edit: Oh, zu spät...