What’s wrong with my code?

Question

var quantidade = "200";

setInterval(function(){
    var tempo     = document.getElementById("banner").textContent; 
    var match     = tempo.match(/[^\d](\d+)/);
    var nr        = match && match[1];
    if(nr === "5"){
        var red   = document.getElementById("red").textContent;
        var green = document.getElementById("green").textContent;
        var black = document.getElementById("black").textContent;
        if(red>black){
            var corfinal = "black";
            $("#enviarpreto").click();
        }else{
          var corfinal = "red";
          $("#enviarvermelho").click();
        }
    }

   if(tempo[5] === "d"){
       var red2     = document.getElementById("red").textContent;
       var green2   = document.getElementById("green").textContent;
       var black2   = document.getElementById("black").textContent;
       var meured   = document.getElementById("meured").textContent;
       var meublack = document.getElementById("meublack").textContent;
       if(meured < 0){
           quantidade = quantidade*2;
       }else if(meupreto < 0){
           quantidade = quantidade*2
       }else{
           quantidade = 200;
       }
       document.getElementById('betAmount').value = quantidade;
   }
}, 1000);

What I intend to do is that when the time condition[5] === d happens, it will multiply if those values are negative. But I just want you to multiply it once. But we have a problem as it is in setinterval, it is constantly multiplying, because the time[5], is for 4 seconds.

How can I so that it only makes me 1x?

Thank you.

Answer 1

You should clear the interval as soon as your condition is satisfied.

Thus:

var myVar = setInterval(function(){ 
                  // definir condição.
                  if(n == 5){
                    // limpe o interval para parar a execução.  
                    clearInterval(myVar);
                  }       
            }, 1000);

So I believe, just define the clearInterval within your if(tempo[5] === "d") to perform the routine only once.

Updating :

Tested here worked without giving error in my browser, however I do not I would use a script like this. It’s infinite loop. I did it because you asked me to. So the people who are going to face the wrong way and think about downvotes. Know; I would never use that!

CallTimeOut = function(){
       var myTimeOut = setTimeout(function(){
       // execute aqui sua rotina 
       //Aqui embaixo sua condição para encerrar o TimeOut  if(tempo[5] === "d")
       if(1==1) clearTimeout(myTimeOut);
   },1000);
}

setInterval(function(){
    if(typeof(myTimeOut) == 'undefined'){
        CallTimeOut(); 
    }   
},1000);

Just an additional:

remember that setTimeout will perform your routine after X milliseconds, in the case above 1000 ms which is equal to a second and will not perform more.

and setInterval, performs a routine every X milliseconds in the case there also 1 second, so it doesn’t even need that clearTimeout it will run and stop and again will be invoked by setInterval.