addChild in .as Datei

[Danjy]

Hallo Leute,
ich hab jetzt schon was länger versucht ohne Hilfe aus dem Forum und nur mittels Bücher, Google etc dieses Problem zu beheben..

Aber ich komm einfach nicht weiter...

Ich habe ein Array in dem mehrere Werte stehen box0 - box5, also 6 Werte.

Diese möchte ich jetzt mittels meiner Funktion buildMc(); als MC erzeugen größe, x Koordinate etc mitgeben und auf die Bühne schmeißen..

sobald ich versuche die Dinger auf die Bühne zu schmeißen bekomme ich folgenden Fehler:

ArgumentError: Error #2024: Ein Objekt kann nicht als untergeordnetes Element seiner selbst hinzugefügt werden.
	at flash.display::DisplayObjectContainer/addChild()
	at meineKlasse/buildMc()
	at Unbenannt_fla::MainTimeline/frame1()

Hier mal ein Auszug aus meiner Klasse:

public function buildMc(objArray:Array):void{
	for(var i:uint = 0; i <= (objArray.length-1); i++){

		objArray[i].graphics.beginFill(0x0000);
		objArray[i].graphics.drawRect(0, 0, 100,100);
		objArray[i].graphics.endFill();
				
		objArray[i].boxName = 'titelbox'+i; 
		objArray[i].x = firstXcor;
		objArray[i].width = _toScaleWidthA;
		objArray[i].height = _toScaleHightA;
		firstXcor += (_toScaleWidthA + _zwischenAbstand);
		trace(objArray[i].boxName);
		addChild(objArray[i]);
	}
}

Was mach ich falsch?

[padavan]

zeig mal dein objArray

[Danjy]

Das bau ich mir in der .fla so zusammen:

var boxes:Array = new Array();
for (var i:uint = 0; i < anzahlDerBoxen; i++) {
	var box:meineKlasse = new meineKlasse();
	box.boxName = 'box'+i;
	boxes.push(box);
}

[Martin Kraft]

Zitat von Danjy

Was mach ich falsch?

Wie die Fehlermeldung schon sagt:

Ein Objekt kann nicht als untergeordnetes Element seiner selbst hinzugefügt werden.

Wenn Du innerhalb eines DisplayObjects (z.B. der Instanz dieser Klasse) addChild verwendest, fügst Du damit eine anderes DisplayObject das vorliegende ein.Das funktioniert natürlich nicht, wenn das DisplayObject, das da eingefügt werden soll, selbst die aktuelle Instanz ist, oder diese enthält(, weil das Resultat eine Art logische Endlosschleife wäre).

• Was befindet sich denn in dem objArray?
• Was ist diese Klasse?
• Warum verwaltet sie Eigenschaften, die ihr von der Hauptzeitleiste übergeben werden?
• Zeig mal etwas mehr Code!

[Martin Kraft]

Zitat von Danjy

Das bau ich mir in der .fla so zusammen:

var boxes:Array = new Array();
for (var i:uint = 0; i < anzahlDerBoxen; i++) {
	var box:meineKlasse = new meineKlasse();
	box.boxName = 'box'+i;
	boxes.push(box);
}

Wenn Du mit diesem boxes-Array die Funktion buildMc einer der meineKlasse-Instanzen aufrufst, ist es kein Wunder, dass es zu dem besagten Problem kommt. Das Array enthält ja bereits die Instanz, die später das addChild ausführen soll...

P.S.: Klassennamen sollten immer mit einem Großbuchstaben beginnen (also MeineKlasse statt meineKlasse) und den Inhalt und die Funktion der Klasse beschreiben (Box wäre daher ein besserer Klassenname).

[Danjy]

Danke schonmal für die Antwort...

Also alles was ich momentan will ist einfach nur dynamisch 6 Kasten ausgeben die eine sich selber nebeneinander Positionieren mit einen gewissen Abstand.

meine Klasse:

package{
	import fl.transitions.Tween;
	import fl.transitions.easing.*;
	import fl.transitions.TweenEvent;
	import flash.display.*;

	public class meineKlasse extends flash.display.MovieClip{		
		
		public var boxName:String;
		public var _corX:Number;
		public var _scaleX:Number;
		public var _scaleY:Number;
		public static var timeToMove:Number = 0;
		
		public static var firstXcor:Number = 0;
		
		private var _toScaleWidthA:Number = 820;
		private var _toScaleHightA:Number = 500;
		private var _toScaleWidthB:Number = 994;
		private var _toScaleHightB:Number = 606;
		private var _zwischenAbstand:Number = 10;
		
		
		public function meineKlasse():void {
			//constructor
		}
		
		
		public function buildMc(objArray:Array):void{
			for(var i:uint = 0; i <= (objArray.length-1); i++){
				//trace(objArray[i].boxName);
				//var titelbox:MovieClip = objArray[i];
				//titelbox.boxName = 'titelbox'+i;
				//trace(titelbox.boxName);
				
				objArray[i].graphics.beginFill(0x0000);
				objArray[i].graphics.drawRect(0, 0, 100,100);
				objArray[i].graphics.endFill();
				
				objArray[i].boxName = 'titelbox'+i; 
				objArray[i].x = firstXcor;
				objArray[i].width = _toScaleWidthA;
				objArray[i].height = _toScaleHightA;
				firstXcor += (_toScaleWidthA + _zwischenAbstand);
				trace(objArray[i].boxName);
				addChild(objArray[i]);
			}
		}
		
	}
}

und alles was in der .fla passiert:

import meineKlasse;
var anzahlDerBoxen:Number = 6;

stage.addEventListener(Event.ENTER_FRAME, enterFrameFunction);
function enterFrameFunction(evt:Event){
	
}

var boxes:Array = new Array();
for (var i:uint = 0; i < anzahlDerBoxen; i++) {
	var box:meineKlasse = new meineKlasse();
	box.boxName = 'box'+i;
	boxes.push(box);
}

box.buildMc(boxes);

Wie kann ich den verhindern das ich die ineinander Schachtel?
Wahrscheinlich muss ich das addChild in der .fla ausführen oder?

[Danjy]

Ich habe es jetzt einmal so versucht aufzurufen..

var titelbox:meineKlasse = new meineKlasse();
titelbox.buildMc(boxes);

jetzt kommt der Fehler nicht mehr.. jedoch werden meine Kasten noch immer nicht angezeit...

Kann es sein das ich die immer wieder Überschreibe ?
Da ich bei

trace(box.boxName);

nur das letzte Objekt bekomme..

Also irgendwas stimmt da noch nicht..
Könnte ich jetzt mittels getChildByName auf das erzeugte MC zugreifen?

[Martin Kraft]

Wie gesagt: Das addChild bezieht sich auf die Box in der es steht.

Wenn ich mir Deinen Code so ansehen, sollte die Funktion buildMc auch eigentlich nicht in der Box-Klasse stehen, sondern in dem Objekt, das die Boxen verwaltet, dass könnte eine zweite Klasse sein, aber auch einfach die Zeitleiste in der Du z.Z. die Boxen generierst.

[silentx]

Versuchs mal so (habs nicht getestet)

AS-Datei/Klasse

public function buildMc(objArray:Array, pThis:*):void{
  // Dein Code
  pThis.addChild(objArray[i]);
}

FLA-DATEI

box.buildMc(boxes, this);

[Danjy]

Hmm.. Sorry aber ganz versteh ich das noch nicht...

also Generieren des Arrays und Aufruf der buildMc-Funktion:

var boxes:Array = new Array();
for (var i:uint = 0; i < anzahlDerBoxen; i++) {
	var box:meineKlasse = new meineKlasse();
	box.boxName = 'box'+i;
	boxes.push(box);
}

var titelbox:meineKlasse = new meineKlasse();
titelbox.buildMc(boxes);

Und die Funktion selber

public function buildMc(objArray:Array):void{
	for(var i:uint = 0; i <= (objArray.length-1); i++){
				
		objArray[i].graphics.beginFill(0x0000);
		objArray[i].graphics.drawRect(0, 0, 100,100);
		objArray[i].graphics.endFill();
				
		objArray[i].boxName = 'titelbox'+i; 
		objArray[i].x = firstXcor;
		objArray[i].width = _toScaleWidthA;
		objArray[i].height = _toScaleHightA;
		firstXcor += (_toScaleWidthA + _zwischenAbstand);
		trace(objArray[i].boxName);
		addChild(objArray[i]);
	}
}

Das klappt ja jetzt auch so halb.. Also der Befehl [I]trace(objArray.boxName); gibt mir ja titelbild0 - titelbild5 aus.. das füllen der Mcs klappt nicht.. Es scheint auch als würde kein einziges MC auf der Bühne liegen, und ich weiß auch nicht wie ich die einzelnen Objekte jetzt ansprechen könnte...

Sorry aber wie genau soll ich vorgehen?

[Martin Kraft]

Ich würd's so machen:

Box.as

PHP-Code:

package {
    import flash.display.Sprite;

    public class Box extends Sprite{        
        
        public var boxName:String;
        
        //
        
        private var _toScaleWidthA:Number = 820;
        private var _toScaleHightA:Number = 500;
        private var _toScaleWidthB:Number = 994;
        private var _toScaleHightB:Number = 606;
        
        //////////////////////////////////////////
        
        public function Box() {
            draw();
            
            width = _toScaleWidthA;
            height = _toScaleHightA;
        }
        
        //////////////////////////////////////////
        
        private function draw():void{
            graphics.beginFill(0x0000);
            graphics.drawRect(0, 0, 100,100);
            graphics.endFill();
        }
    }
} 


Zeitleiste:

PHP-Code:

var boxes:Array = [];

function generateBoxes(anzahlDerBoxen:uint = 6):void {
    var xi:Number = 0;
    var zwischenAbstand:Number = 10;

    for (var i:uint = 0; i < anzahlDerBoxen; i++) {
        var box:Box = new Box();
        box.boxName = 'titelbox' + i;
        
        boxes.push(box);
        addChild(box);
        
        box.x = xi;
        xi += box.width + _zwischenAbstand;
    }
}

generateBoxes(); 


P.S.: Wenn Du AS-Tweenings einsetzen möchtest, würde ich von der Flash-eigenen Tweening-Klasse abraten (mit der gibt's immer wieder Probleme). Verwende stattdessen besser eine ausgewachsene Engine, wie Tweensy oder TweenLite.

[Danjy]

Sauber danke @silentx klappt wunderbar.. könntest du mir kurz eine kleine erklärung geben warum :P ?

P.S.: Wenn Du AS-Tweenings einsetzen möchtest, würde ich von der Flash-eigenen Tweening-Klasse abraten (mit der gibt's immer wieder Probleme). Verwende stattdessen besser eine ausgewachsene Engine, wie Tweensy oder TweenLite.

Okay werde es damit mal ausprobieren wenn ich soweit bin

Ich hätte noch eine letzte Frage.. wenn ich jetzt das script so benutze mit der kleinen Änderung von silentx.. Wie genau greif ich jetzt auf die einzelnen Elemente in der .as zu?

[Martin Kraft]

Zitat von silentx

Versuchs mal so (habs nicht getestet)

AS-Datei/Klasse

public function buildMc(objArray:Array, pThis:*):void{
  // Dein Code
  pThis.addChild(objArray[i]);
}

FLA-DATEI

box.buildMc(boxes, this);

Sorry silentx, ich will Dir da wirklich nicht reinkrätschen, aber aus OOP-Sicht ist das nicht mal die zweitbeste Lösung:

• Man sollte wenn irgendmöglich vermeiden auf eine WildCard (*) zu typisieren. In diesem Spezialfall ist so nämlich nicht mal sichergestellt, dass pThis.addChild überhaupt existiert. Wenn überhaupt würde ich daher pThis als DisplayObjectContainer typisieren.
  
  
• Außerdem sollte sich grundsätzlich jedes Objekt um seinen eigenen Sch*** kümmern. Und die Positionierung aller Box-Instanzen gehört definitiv nicht in den Aufgabenbereich einer einzelnen Instanz.

@Danjy: Schau Dir deshalb bitte mal meinen Vorschlag an. Eine saubere Abgrenzung der Klassen ist nämlich die beste Garantie dafür, später keine Probleme zu bekommen...

[silentx]

@Danjy : Erklärung warum es funktioniert
Jetzt bezieht sich das addChild auf die fla-datei und nicht mehr auf sich selbst.
Somit ist es eine andere Klasse. Jedoch ist es nicht wirklich schön gelöst so.

(Wieder ungetestet)
wenn du beim erfassen statt .boxName nur .name verwendest könntest du
in der fla datei folgendermassen abfragen

var mybox:* = getChildByName('Titelbox 1');

[Danjy]

Vielen Dank an euch beide ihr habt mir sehr sehr weitergeholfen