bei ausgelagerter Klasse wird addChild() nicht ausgeführt.

[Brav]

hallo,

oh, hoffentlich kann mir jemand helfen... wir sollen alle unsere klassen auslagern und das habe ich grad ausprobiert, aber irgendwie ruft er zwar mein addrepository() auf - legt es mir aber nicht auf die Bühne, weiß jemand wo mein Fehler liegen könnte?

Main.as

package  {
	
	import flash.display.MovieClip;
	import flash.events.Event;	
	import view.view;

	
	public class Main extends MovieClip {

		public var dino: view;

		public function Main() {
			this.addEventListener(Event.ENTER_FRAME, test);
			}
			
		public function test(b:Event):void {
			
			this.removeEventListener(Event.ENTER_FRAME, test);
			
			dino = new view();
			
			dino.addrepository();
			}

	}
}

view.as

package view {
	
	import flash.display.MovieClip;
	
	public class view extends MovieClip{

		public var figure:MovieClip;

		public function view() {
			super();			
		}
		
		
		public function addrepository() {
			
			trace ("wird aufgerufen");
			
			figure = new repository();
							
			//Position:
			figure.x= 300;
			figure.y= 300;
			
			addChild(figure);
			
			trace ("wird aufgerufen");
			}
	}
	
}

[Martin Kraft]

Du addest zwar figure in View nicht aber die View-Instanz dino in Main:

addChild(dino); 

P.S.:

• Was verstehst Du unter »alle unsere klassen auslagern«? Klassen (zumindest solche mit Code) sind doch immer in .as-Dateien ausgelagert
• Klassennamen sollte immer mit einem Großbuchstaben beginnen (also View statt view).
• Ich finde es zudem unglücklich das Paket genauso zu nennen wie die Klasse!
• Was ist denn der Sinn dieser seltsamen Event.ENTER_FRAME-Konstruktion in Main?

[Ouroborus23]

Dieses onEnterFrame-Gehüpfe wird gäufig bei Komponenten-Frameworks genutzt ("Deferred Rendering"), hat an dieser Stelle jedoch wenig Sinn. Ich mache da manchmal eine onAddedToStage-Abfrage - was in diesem Fall auch nicht unbedingt notwending ist.

Ich möchte Martins Tipps außerdem wie folgt ergänzen:

• Funktionen sollten im lowerCamelCase geschrieben werden, dass erleichtert die Lesbarkeit unheimlich!
• Funktionen sollten sinnvolle Namen erhalten: Deine Funktion "test" könnte auch "onEnterFrame" heißen, alles andere verwirrt.
• auch auf

  this

  kann mit AS3 verzichtet werden.

Liebe Grüße!

[Brav]

DANKE!! das war meine fehlende Zeile.

zu deinem PS:
1. ist ein projekt, und wir müssen im besten fall für jede Aktion usw. eine eigene Klasse verwenden. hab das vielleicht bisschen komisch ausgedrückt
2. danke für den Tipp mit den Großbuchstaben
3. war bis jetzt nur ein Test um irgendwas auf die Bühne zu bekommen. über genauere Bezeichnungen werde ich mir erst jetzt Gedanken machen wenn ich richtig anfange
4. das ist mehr oder weniger aus anderen Codes kopiert. nehme gerne verbesserungsvorschläge an

Danke nochmal!!

[Ouroborus23]

Zitat von Brav

wir müssen im besten fall für jede Aktion usw. eine eigene Klasse verwenden.

Den Lehrer/Prof, der das verlangt, soll sich hier doch bitte mal melden.