Erkennen welcher Button gedrückt wurde

[JochenKnochen]

Hallo!

Ich bearbeite grad fürs Studium eine Aufgabe bei der wir mittels Flash eine interaktive Anleitung erstellen sollen.

Meine Daten stehen in einer XML-Datei. Aus den Überschriften, lasse ich mir Buttons erstellen, die dann direkt auf den Hilfetext verweisen sollen (bzw. ihn laden sollen).

Ich steh jetzt nur vor der Frage wie ich abfrage, welcher Button gedrückt wurde. Stehe da irgendwie grad total auf dem Schlauch.

for (var i:int=0;i<=xmlData.kapitel.hilfetext.ueberschrift.length()-1; i++){
		
		this["tf" + i] = new Button();
		menu1.addChild(this["tf"+i]);
		this["tf"+i].label=xmlData.kapitel.hilfetext.ueberschrift[i];
		trace("tf"+i);
		
		
		
		this["tf"+i].addEventListener(MouseEvent.MOUSE_UP, loadxmlelement);
}

Wäre nett, wenn mir jemand nen Denkanstoß verpassen könnte, vielleicht ist ja auch weg, die Buttons so zu generieren, ein Irrweg.

[sonar]

Hi,

du könntest den Button-Instanzen "verwertbare" Namen geben und damit arbeiten, so in der Art:

PHP-Code:

// in der Schleife
this["tf" + i].name = "btn_" + i;

// im MouseUp-Listener
trace( event.target.name.split("_")[1] ); 


Also im MouseUp-Listener das jeweilige 'i' des Buttons rausholen und anhand dessen die Aktion bestimmen, die ausgelöst werden soll o.ä.

[Omega Psi]

Besser wäre es, wenn du eine direkte Assozition herstellst, beispielsweise über ein Dictionary. Dann muss deine Anwendung nicht wissen, wie die Konvention für die Namensgebung der Buttons ist:

const dict:Dictionary = new Dictionary();
const button:Sprite = new Sprite();
button.addEventListener(MouseEvent.CLICK, button_clickHandler);
dict[button] = "Hello World";

function button_clickHandler(event:MouseEvent):void
{
  const sprite:Sprite = event.target as Sprite;
  if (sprite) trace(dict[sprite]);
}

[JochenKnochen]

Hey,

danke für eure Antworten! Hab mich für die erste Version entschieden, da mir Dictionarys nicht so geläufig sind und ich im ersteren auch nachvollziehen kann was überhaupt in den Anweisungen passiert.

[Martin Kraft]

Da Dich ja noch niemand begrüßt hat: Willkommen im FlashForum!

Zitat von JochenKnochen

Hab mich für die erste Version entschieden, da mir Dictionarys nicht so geläufig sind und ich im ersteren auch nachvollziehen kann was überhaupt in den Anweisungen passiert.

Ich würde Dir trotzdem empfehlen Dich mal einem Weg zu beschäftigen*, der ohne dynamische Objektzugriffe auskommt. Die sind nämlich sehr unperformant, fehleranfällig und zudem schwer zu lesen. In AS1/2 war sowas zwar gang und gäbe, aber spätens seit AS3 gibt es bessere Wege. Insbesondere, wenn man zigmal nacheinander dasselbe Objekt mit der Klammersyntax anspricht, ist es an der Zeit stattdessen eine lokale Variable zu verwenden.

Auch beim Durchlaufen Deines XMLs kannst Du ganz auf [] und einen Index verzichten und stattdessen einfach eine for each-Schleife verwenden.

Und Dictionarys sind übrigens kein Hexenwerk, sondern ungemein praktisch. Sie funktionieren genauso wie Arrays, können aber statt Zahlen beliebige Objekte als Schlüssel verwenden.

Alles in allem könnte Dein Code so aussehen:

import flash.utils.Dictionary;
import flash.events.MouseEvent;
 
var btn:Button;
var hilfetextByButton:Dictionary;
 
for (var hilfetext:XML in xmlData.kapitel.hilfetext) {
    btn = new Button(); 
    btn.label = hilfetext.ueberschrift.toString();
    btn.addEventListener(MouseEvent.MOUSE_UP, loadxmlelement);
    
    menu1.addChild(btn);
    hilfetextByButton[btn] = hilfetext;
}
 
function loadxmlelement(event:MouseEvent):void {
    var hilfeText:XML = hilfetextByButton[event.currentTarget];
    trace("Das zum Button gehörige XML: " + hilfetext);
    
    // Deine Aktionen
} 

*gerade im Studium geht's ja darum, was neues zu lernen und nicht nur auf ausgetrampelten (Irr-)Wegen zu bleiben

[Omega Psi]

Eine andere, ähnliche Art und Weise wäre es, über eine zweite Map die Assoziation herzustellen:

const map:Object = {};
const button:Sprite = new Sprite();
button.addEventListener(MouseEvent.CLICK, button_clickHandler);

setTextFor(sprite.name = "key1", "Hello world");

function button_clickHandler(event:MouseEvent):void
{
  const sprite:Sprite = event.target as Sprite;
  if (sprite) trace(getText(sprite.name));
};

function getTextFor(key:String):String
{
  if (!key || key == '')
    throw new ArgumentError('Key must not be empty or null.');;

  if (!map.hasOwnProperty(key))
    throw new ArgumentError('Key not found.');;

  return map[key];
}

function setTextFor(key:String, text:String = ""):void
{
  if (!key || key == '')
    throw new ArgumentError('Key must not be empty or null');;

  map[key] = text;
}

Du benutzt einfach die name Eigenschaft als id für die Map. Zugriffe so zu kapseln erleichtert oft die Fehlersuche im Nachhinein.

[Martin Kraft]

Zitat von Omega Psi

Du benutzt einfach die name Eigenschaft als id für die Map. Zugriffe so zu kapseln erleichtert oft die Fehlersuche im Nachhinein.

Das fängt zwar die Fehler ab, macht IMHO die Sache im Kern aber nicht wirklich besser und bläht sie nebenbei durch zusätzliche Werte und Abhängigkeiten unnötig auf. Was passiert z.B. wenn sich irgendjemand außerhalb dieser Konstruktion an der name-Eigenschaft vergreift?!

[Omega Psi]

Naja, Seiteneffekte können immer auftreten. Daher das schmeißen der Fehler.

Was ist wenn man in deinem Beispiel das Dictionary "nullt"? Das var macht die Sache nicht sicherer und daher sind solche Argumente auch immer schwer haltbar. So gesehen sind so ziemlich alle Beispiele hier im Forum potenziell schlecht, da sie Risiken im Laufzeit verhalten aufweisen.

-> Es gibt keine optimale Lösung, nur eine Menge von Kompromissen und man muss sich darüber im Klaren sein. Und auch wenn die Zugriffsoperationen den code aufblähen, so helfen sie beim Suchen der Fehler und beim Debuggen.

Da man in ActionScript kaum ohne Seiteneffekte arbeiten kann, würde ich auch mit gekapselten Dictionarys arbeiten, aber das ist nunmal nicht unsere Entscheidung.