CODEWITHSUNDEEP
flashactionscript-3eventsrace-condition
user223097
user223097

Reputation:

Race condition firing events in AS3

I have some troubles firing and removing events in the right chronicle order. The code below gives the following output:

of course this should be more something like:

Can someone help me with this? I'm running out of ideas on how to tackle this.

thx!

    for(var i:int = 0;i< 3;i++){
        createPoster();         
    }

    function createPoster(){
        Main.db.savePoster();
        Main.db.addEventListener(Config.evt_SAVEPOSTER_READY, callService);
    }

    function callService(){
       Main.db.removeEventListener(Config.evt_SAVEPOSTER_READY, callService);
    }

Upvotes: 1

Views: 718

Answers (3)

Amarghosh
Amarghosh

Reputation: 59451

Is the db call (Main.db.savePoster();) synchronous - does it return only after the action is completed? Since you are calling addEventListener after the db call, the event listener (for the first iteration at least) will not be called if the db-call is synchronous.

Is the Main.db the same instance in all the three iterations? If it is, you don't have to register the same event listener thrice for it - once would be enough. Call addEventListener before starting the for-loop. Keep a counter for tracking the number of calls to the callService and call removeEventListener once the counter hits loop count (3, in this case).

Upvotes: 0

bhups
bhups

Reputation: 14885

The problem is that you are registering same function callService for same event Config.evt_SAVEPOSTER_READY on single EvenDispatcher objectdb. So as soon first savePoster dispatches the event after successfully saving the poster, db receives the event and three eventHandlers (in this case callService) are called because callService is registered thrice. So one solution would be dispatching the events from Poster.

for(var i:int = 0;i< 3;i++){
  createPoster();
}
function createPoster(){
  poster = Main.db.savePoster();
  poster.addEventListener(Config.evt_SAVEPOSTER_READY, callService);
}
function callService(e:PosterEvent){
  e.target.removeEventListener(Config.evt_SAVEPOSTER_READY, callService);
}

Upvotes: 1

Rey
Rey

Reputation: 125

Have you checked what happens when you loop through one item only? Seems to me you are not queuing your routines properly.

You might probably want to add an event listener to your Main.db object only once and remove it when you have gotten all your 'posters' successfully saved.

Upvotes: 0

Related Questions