How to stop timer if function executed properly?

Question

The Section() is responsible to look on database and get any HTML code if some things are matched. If nothing matched, there is no callback (but I can callback an empty thing).

Then it calls addIt() and display the HTML. Works fine.

My question is when there is some HTML to get from database, to stop the timer? Because for now it adds HTML every 10 seconds.

function addIt(data) { $(data).insertAfter('#hello'); }

setInterval(function Section(){
      $.getJSON("domain"+ number, addIt);
}, 10000);

use clearInterval(); w3schools.com/jsref/met_win_clearinterval.asp — yugi
– yugi, Commented May 19, 2015 at 8:36

AshBringer · Accepted Answer · 2015-05-19 08:37:20Z

1

Maybe something like this :

var x = setInterval(function Section(){
  $.getJSON("domain"+ number, addIt);
}, 10000);

if( // something ) {
   clearInterval(x);
 }

answered May 19, 2015 at 8:37

AshBringer

2,6732 gold badges24 silver badges43 bronze badges

Sign up to request clarification or add additional context in comments.

Comments

mplungjan · Accepted Answer · 2015-05-19 09:57:16Z

0

I would never call AJAX in an interval. Instead use the callback to call again

function Section(){
   $.getJSON("domain"+number,addIt);
}
function addIt(data) { 
    if (data) $(data).insertAfter('#hello'); 
    else setTimeout(Section,10000); // call again in 10 seconds unless data was returned
}
Section(); // first call

If the call results in an ERROR when not returning data, try this:

$.getJSON("domain"+number, function(data) {
  if (data) $(data).insertAfter('#hello'); 
})
.fail(function() { 
  setTimeout(Section,10000);
});

edited May 19, 2015 at 9:57

answered May 19, 2015 at 8:43

mplungjan

180k29 gold badges183 silver badges246 bronze badges

9 Comments

EnexoOnoma Over a year ago

hello, thank you for your answer. I am a newbie in AJAX, jQuery and these, what I am trying is to repeat the Section() every x seconds. What is another way to do this, a better way ?

mplungjan Over a year ago

Like I showed you. It will call Section (after a successful call) in 10 seconds but only if no data. If data, it will stop. My method will make sure the loop stops on error too.

EnexoOnoma Over a year ago

I see, this is nice. Can you please tell me why, meaning in a theoretical level why is it bad ?

mplungjan Over a year ago

If your server is slower that the timeout seconds to respond, you will call it again and again while it is still processing

EnexoOnoma Over a year ago

Meaning that if I set the timer to 1 sec, but for a reason it took 2 seconds to respond, that means it will run 2 times. But with your case, it will run upon completion. right?

|

Miraage · Accepted Answer · 2015-05-19 08:40:53Z

0

var timerId;

function addIt(data) { 
  $(data).insertAfter('#hello');
}

function section() {
  $.getJSON().done(function(data) {
    addIt(data);

    if (timerId) {
      clearInterval(timerId);
    }
  });
}

timerId = setInterval(section, 10000);

answered May 19, 2015 at 8:40

Miraage

3,4943 gold badges28 silver badges43 bronze badges

Collectives™ on Stack Overflow

How to stop timer if function executed properly?

3 Answers 3

Comments

9 Comments

Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

Comments

9 Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related