Solved

problem passing a variable from one function to another AS3

Posted on 2008-06-21
4
2,052 Views
Last Modified: 2013-11-11
Hi
it is my first try of writing in AS3 in a class format. so i`m building a gallery and i built a grid so now i wanted to insert the dynamic values in. i bring the xml into flash (it works) and i want the number of "images" to take its value from the xml. but i cannot make it work.
i have the value from the xml store in a variable but i cannot take it and pass it into another method.

i attach my code as a whole because maybe the problem is with my writing of the class.

best regards

ron
package{

	

	import flash.display.MovieClip;

	import flash.net.URLLoader;

	import flash.net.URLRequest;

	import flash.display.Loader;

	import flash.events.Event;

	import flash.display.Sprite;

	

	

	public class SGclass extends MovieClip{

		

		private var siteLoader:URLLoader;

		private var siteRequest:URLRequest;

		private var imageLoader:Loader;

		private var doxml:XML;

		private var boxNum:int;

		private var cols:int= 6;

		private var rows:int = Math.ceil(boxNum / cols);

		private var boxCount:int = 0;

		private var box:Sprite;

		

		

		public function SGclass(){

			

			siteLoader = new URLLoader();

			siteRequest = new URLRequest("sites2.xml");

			siteLoader.load(siteRequest);

			

			siteLoader.addEventListener(Event.COMPLETE , takesites);

			placeImages();

			//trace(boxNum);

			

		}

		

		

		public function takesites (e:Event):void {

			

			doxml = XML(e.target.data);

			//trace(doxml.site);

			

			boxNum = doxml.site.length();

			//trace(boxNum);

			

			

		

		}

		

	private function placeImages(){

			

			for (var py:int = 0; py < rows; py ++) {

				for (var px:int = 0 ; px < cols ; px ++) {

					//see that you are not exiding the boxNum variable

					if(boxCount < boxNum){

						box= new Sprite();

						box.graphics.beginFill(0xCCCCCC);

						box.graphics.drawRoundRect(0,0,80,80,15,15);

						box.graphics.endFill();

						box.buttonMode = true;

						box.x = 100 + px * (box.width + 10);

						box.y = 40 + py * (box.height+ 10);

						addChild(box);

						boxCount++;

						

						}

					}

				}

				

			
 

		}
 

		

		

	}

	

}

Open in new window

0
Comment
Question by:derrida
  • 2
  • 2
4 Comments
 
LVL 34

Expert Comment

by:Aneesh Chopra
Comment Utility
problem is you are callling "placeImages()" function just with XML load call, which is not correct.
XML load is an asyncronous function that so we have 'complete' event to check when XML gets loaded and data is available for use.

solution is:
You should call "placeImages()" function after XML get loaded.
below is the updated code which should work, if it does not, I suggest to upload your sample source with XML file for review:


package{

	

	import flash.display.MovieClip;

	import flash.net.URLLoader;

	import flash.net.URLRequest;

	import flash.display.Loader;

	import flash.events.Event;

	import flash.display.Sprite;

	

	

	public class SGclass extends MovieClip{

		

		private var siteLoader:URLLoader;

		private var siteRequest:URLRequest;

		private var imageLoader:Loader;

		private var doxml:XML;

		private var boxNum:int;

		private var cols:int= 6;

		private var rows:int = Math.ceil(boxNum / cols);

		private var boxCount:int = 0;

		private var box:Sprite;

		

		

		public function SGclass(){

			

			siteLoader = new URLLoader();

			siteRequest = new URLRequest("sites2.xml");

			siteLoader.load(siteRequest);

			

			siteLoader.addEventListener(Event.COMPLETE , takesites);

			

			//trace(boxNum);

			

		}

		

		

		public function takesites (e:Event):void {

			

			doxml = XML(e.target.data);

			//trace(doxml.site);

			

			boxNum = doxml.site.length();

			//trace(boxNum);

			placeImages();		

		}

		

	private function placeImages(){

			

			for (var py:int = 0; py < rows; py ++) {

				for (var px:int = 0 ; px < cols ; px ++) {

					//see that you are not exiding the boxNum variable

					if(boxCount < boxNum){

						box= new Sprite();

						box.graphics.beginFill(0xCCCCCC);

						box.graphics.drawRoundRect(0,0,80,80,15,15);

						box.graphics.endFill();

						box.buttonMode = true;

						box.x = 100 + px * (box.width + 10);

						box.y = 40 + py * (box.height+ 10);

						addChild(box);

						boxCount++;

						

						}

					}

				}

				

			

 

		}

 

		

		

	}

	

}

Open in new window

0
 
LVL 1

Author Comment

by:derrida
Comment Utility
hi
it still does not work. i uploaded the files here:
https://filedb.experts-exchange.com/incoming/ee-stuff/7187-mysitegalleryWithClass.zip

and can you explain why i should call that function there and not in the constructor please?


ron
0
 
LVL 34

Accepted Solution

by:
Aneesh Chopra earned 500 total points
Comment Utility
Got the fix:

another problem was you were intializing variable 'rows' also before setting boxNum, and 'rows' value is dependent over 'boxNum' variable
so now I have intialized the variable 'rows' also after XML get loaded and all is working fine.

BELOW is the working code:

and
here I am explaining it again,
very basic logic is you can use a variable only once have its source available,
here you want to set the boxNum using XML, and XML is being loaded externally, so XML data will be available only once flash gets it loaded into memory.
and we have 'complete' event to know the XML is loaded,
hence we have to initiate the boxNum and rows variable inside 'takesites' function, because it gets executed when we have XML ready.

and 'placeImages()' function is also using variables 'boxNum' and 'rows' which are being set inside function 'takesites()' that so we have to call function 'placeImages()' only after setting variables 'boxNum' and 'rows'

I hope all would be clear.
package{

	

	import flash.display.MovieClip;

	import flash.net.URLLoader;

	import flash.net.URLRequest;

	import flash.display.Loader;

	import flash.events.Event;

	import flash.display.Sprite;

	

	

	public class SGclass extends MovieClip{

		

		private var siteLoader:URLLoader;

		private var siteRequest:URLRequest;

		private var imageLoader:Loader;

		private var doxml:XML;

		private var boxNum:int;

		private var cols:int= 6;

		private var rows:int;

		private var boxCount:int = 0;

		private var box:Sprite;

		

		

		public function SGclass(){

			

			siteLoader = new URLLoader();

			siteRequest = new URLRequest("sites2.xml");

			siteLoader.load(siteRequest);

			

			siteLoader.addEventListener(Event.COMPLETE , takesites);

			

			//trace(boxNum);

			

		}

		

		

		public function takesites (e:Event):void {

			

			doxml = XML(e.target.data);

			//trace(doxml.site);

			

			boxNum = doxml.site.length();

			rows = Math.ceil(boxNum / cols);

			placeImages();

			trace(boxNum);

		}

		

	private function placeImages(){

			

			for (var py:int = 0; py < rows; py ++) {

				for (var px:int = 0 ; px < cols ; px ++) {

					//see that you are not exiding the boxNum variable

					if(boxCount < boxNum){

						box= new Sprite();

						box.graphics.beginFill(0xCCCCCC);

						box.graphics.drawRoundRect(0,0,80,80,15,15);

						box.graphics.endFill();

						box.buttonMode = true;

						box.x = 100 + px * (box.width + 10);

						box.y = 40 + py * (box.height+ 10);

						addChild(box);

						boxCount++;

						

						}

					}

				}

				

			
 

		}
 

		

		

	}

	

}

Open in new window

0
 
LVL 1

Author Comment

by:derrida
Comment Utility
Hi
thanks it works and i do get the explaination. while i`m writing AS not as class the flowing of the code is more natural for me then this sort of writing, but i hope to learn this type of writing.

thank you for your efforts.

best regards

ron
0

Featured Post

What Should I Do With This Threat Intelligence?

Are you wondering if you actually need threat intelligence? The answer is yes. We explain the basics for creating useful threat intelligence.

Join & Write a Comment

I know the transition can be hard. We got used to the ease of use ActionScript 2 had, but honestly, it became problematic later on, especially if designers were involved in the project and found it easy to add code as they saw fit. So, this artic…
I have been doing hardcore actionscripting for some time; and needless to say I have faced a lot of problems in just understanding others' code rather than understanding what the code executes. A programmer's life can become hell when there are a lo…
The goal of the tutorial is to teach the user how to how to load their YouTube profile onto Flash Media Live Encoder.
The goal of the tutorial is to teach the user how to set there setting in Adobe Flash Media Live Encoder and YouTube for optimal video and audio quality.

744 members asked questions and received personalized solutions in the past 7 days.

Join the community of 500,000 technology professionals and ask your questions.

Join & Ask a Question

Need Help in Real-Time?

Connect with top rated Experts

17 Experts available now in Live!

Get 1:1 Help Now