How to use conditions with setTimeout in JavaScript? [duplicate]

Question

I am trying to refresh the page after 3 seconds if two text boxes are not filled with text.

I have it set to a button, it works fine without the conditions.

function refresh() {
  var txt1 = document.getElementById("txtAccountNumber");
  var txt2 = document.getElementById("txtStaffNumber");

  if (txt1.value == "" && txt2.value == "") {
    setTimeout(function() {
      location.reload();
    }, 3000);
  }
}

Your code works if called, and the fields are truly empty: if (txt1.value.trim() === "" && txt2.value.trim() === "") setTimeout(location.reload, 3000); — mplungjan
– mplungjan, Commented Aug 30, 2019 at 13:25
Maybe try !txt1.value && !txt2.value instead. My assumption is .value returns undefined — Rajesh Dixit
– Rajesh Dixit, Commented Aug 30, 2019 at 13:26
Try opening up the developer tools (shortcut is F12 on most browsers) and see if there are any errors showing up in the console. — Kei
– Kei, Commented Aug 30, 2019 at 13:26
Can you clarify what you mean by I have it set to a button, it works fine without the conditions ? — Tom O.
– Tom O., Commented Aug 30, 2019 at 13:44
When you call getElementById you should always check the returned value isn't undefined, never assume that just because your calling it in the same page that it will always be valid. Check txt1 and txt2 before use, then check value with typeof value == "string". — SPlatten
– SPlatten, Commented Aug 30, 2019 at 14:19

Jaydeep · Accepted Answer · 2019-08-30 13:24:49Z

0

You can wrap a condition inside a timeout to do so.

<script>
    function refresh() {
         var txt1 = document.getElementById("txtAccountNumber");
         var txt2 = document.getElementById("txtStaffNumber");    

         setTimeout(function () {
            if (txt1.value == "" && txt2.value == "")
            {
                 location.reload();
            }
         }, 3000);
    }

</script>

answered Aug 30, 2019 at 13:24

Jaydeep

1,7221 gold badge17 silver badges33 bronze badges

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

2 Comments

Rajesh Dixit Over a year ago

But how does introduction of if stops calling setTimeout?

mplungjan Over a year ago

Why is that better than not executing the timeout at all?

Kishan Dudhatra · Accepted Answer · 2019-08-30 14:10:54Z

0

Use this code in your script.

setTimeout(function(){ validation(); }, 3000);
var flag = 0;
function validation(){
  var txt1 = document.getElementById("txtAccountNumber").value;
  var txt2 = document.getElementById("txtStaffNumber").value;
  
  if(flag != 0){
    setTimeout(function(){
      if(txt1 == '' && txt2 == ''){
         location.reload();
      }
      else{
          validation();
      }
    }, 3000);
  }
  else{
  	flag = 1;
  	if(txt1 == '' && txt2 == ''){
      location.reload();
    }
    else{
      validation();
    }
  }
}

<form>
    <input type="text" id="txtAccountNumber">
    <input type="text" id="txtStaffNumber">
</form>

answered Aug 30, 2019 at 14:10

Kishan Dudhatra

246 bronze badges

Comments

stephen ryan · Accepted Answer · 2019-08-30 15:28:07Z

0

Thanks for the help it turn out i was using ID's in a asp:TextBox which act weird. I had to add ClientIDMode = "Static" so the function could get the ID's. Works now.

answered Aug 30, 2019 at 15:28

stephen ryan

11 bronze badge

Collectives™ on Stack Overflow

How to use conditions with setTimeout in JavaScript? [duplicate]

3 Answers 3

2 Comments

Comments

Comments

Linked

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

2 Comments

Comments

Comments

Linked

Related