Fire Event execute multiple time.

Question

Fire Event execute multiple time.

You must Login before you can answer or comment on any questions.

Hi,I am developing android application with Titanium, android sdk 1.8.0.1.In my application there are three tabs.Every Time when I click on tab my window get refreshed.So my code structure looks like:

////// m.js  /////////////////
var explore = Titanium.UI.createWindow(
{
  //navBarHidden:true,
  backgroundColor:'#f8f8f8'
});explore.open({animated:true});
 
Ti.App.addEventListener('feed_partial_action',function(e)
{
  alert('inside event')
})
var new = Titanium.UI.createButton(
{
 
});explore.add(new);
new.addEventListener('click', function(e)
{
  var explore_new = Titanium.UI.createWindow(
  {
    navBarHidden:true,
    backgroundColor:'#f8f8f8'
  });explore_new.open({animated:true});
 
});

/////// explore_new.js/////////////////
var explore_new = Titanium.UI.currentWindow;
Ti.App.fireEvent('feed_partial_action',{page_type:'new'});

so my problem is that on first load alert inside the event listner in m.js executed once but when I again call m.js it shows alert twice.on third time it shows alert 3 times and so on and after some time it forcefully close the app.I think event listener of window still open after refreshing widow.So is there any way to handle this problem.Thank you

3 Answers

Your Answer

Think you can help? Login to answer this question!

Accepted Answer

Accepted Answer

hi nilesh,

this can be happened because when m.js window open then feed_partial_action add every time but you did not remove that listener so try this

var test = function(e){
    alert(' << TEST >> ');
}
 
Ti.App.addEventListener('feed_partial_action',test);

and on the close event of the window

explore.addEventListener('close',function(e){
    Ti.App.removeEventListener('feed_partial_action',test);
});

— answered 1 year ago by Mitul Bhalia
answer permalink

14 Comments

Hi,Mitul thank you for reply its working for me.I just do modification that instead of calling window close function; I remove event inside event listener.eg:
```
Ti.App.addEventListener('feed_partial_action',function(e)
{
   do task here.......
   Ti.App.removeEventListener('feed_partial_action');
});
```
And its working fine. But one question remain in mind that every time refreshing it's not removing that event not releasing my memory also.

— commented 1 year ago by nilesh kashid

when you remove your eventListener you should pass the function as parameter so that i gave you the code like above..so try that

var test = function(e){
    alert(' << TEST >> ');
    Ti.App.removeEventListener('feed_partial_action',test);
}
 
Ti.App.addEventListener('feed_partial_action',test);

— commented 1 year ago by Mitul Bhalia

Mitul if I do in following way please see the code and suggest me is there any problem.

/////// explore_new.js/////////////////
var explore_new = Titanium.UI.currentWindow;
Ti.App.fireEvent('feed_partial_action',{page_type:'new'});

////// in m.js i am doing this /////////////////
 
Ti.App.addEventListener('feed_partial_action',function(e)
{
    row.add(e.pagetype)
// instead of calling Ti.App.addEventListener('feed_partial_action',test) I am doing that task in event listener
 
//////// after completion of task ////////////////////
     Ti.App.removeEventListener('feed_partial_action');
 
})

Is this proper way to handle event listeners or causes other issues because my code working fine.is it causing memory related issues since i am doing task inside the event listeners. Thank you once again..........

— commented 1 year ago by nilesh kashid

without creating a function for eventListener you can not remove so you have to create one function and do all the stuff in that function..

var test = function(e){
    row.add(e.pagetype);
    Ti.App.removeEventListener('feed_partial_action',test);
}
 
Ti.App.addEventListener('feed_partial_action',test);

/////// explore_new.js/////////////////
var explore_new = Titanium.UI.currentWindow;
Ti.App.fireEvent('feed_partial_action',{page_type:'new'});

look at this guide

addEventListener

removeEventListener

— commented 1 year ago by Mitul Bhalia

Thank you Mitul for your great support.. thankx a lot.

— commented 1 year ago by nilesh kashid

hye Mitul look around this code (Wiki.appcelerator)

deleteButton.addEventListener('click', function(e){
    // when something happens in your app
    database.doDelete(e.whichRecord);
    // fire a global event so components can react
    Ti.App.fireEvent('db_updated');
});
// ... elsewhere in your code
Ti.App.addEventListener('db_updated', function(e){
    theTable.setData(database.getCurrentRecords());
    someOtherComponent.doSomethingElse();
});

They are executing code inside Ti.App.addEventListener (global event). look at this

— commented 1 year ago by nilesh kashid

and what about this one

can't you understand that you have to use function to remove eventListener if you want ....

— commented 1 year ago by Mitul Bhalia
Ya that's right.. Than you mitul for clearing my concept.

— commented 1 year ago by nilesh kashid
have you got....?

and solved your problem...?

— commented 1 year ago by Mitul Bhalia
Yes that solve my event execution multiple time problem....Now my events executing properly.

— commented 1 year ago by nilesh kashid
so mark this answer as accepted so everyone in this forum can get the help from your question...

and good luck

thnx

— commented 1 year ago by Mitul Bhalia
Thanx a lot

— commented 1 year ago by nilesh kashid
Can you please help me to resolve the issue

http://developer.appcelerator.com/question/140905/fireevent-fires-an-api-calls-more-than-once#answer-245090

— commented 11 months ago by Rebin Joseph
This worked fine for me. Try this too. :)

var test = function(e) {
```
 alert(' TEST ');

Ti.App.removeEventListener('feed_partial_action',  test);
```
}

Ti.App.addEventListener('feed_partial_action', test);

— commented 10 months ago by Ishara Amarasekera
Show 11 more comments

Answer 2

Hi Nilesh,

When you call m.js you are actually creating one more window because of every time you have created window in m.js.Try to use like this that may help you.

var explore = Titanium.UI.currentWindow;

Answer 3

This worked fine for me. Try this too. :)

var test = function(e) {

alert(' TEST ');

Ti.App.removeEventListener('feed_partial_action',  test);

}

Ti.App.addEventListener('feed_partial_action', test);