Can't run two for loop statement within same if statement?

Question

I have the following JS code, I feel weird that only the first for loop can run, but the 2nd part doesn't work. I even tried to make two different if statement and include the for loop separately, but the same thing happened. If I run the for loop separately (by deleting another for loop), both of them can run, means both of the logic should be correct. Please help.

var s = "<?php printf($resultDataOrderInfo[0][status]); ?>";

   if(s == "Processing"){
       var t = document.getElementsByTagName("input");
       for (i=0; i<=t.length; i++){
           t[i].setAttribute("readonly","true");
       }  // only this above part can run

       var d = document.getElementsByName("delete");
       for (i=0; i<=d.length; i++){
           d[i].setAttribute("hidden","true");
       }
   }

There's no reason they shouldn't be both be able to run. Can you reproduce this issue in a jsFiddle ? — adeneo
– adeneo, Commented May 5, 2014 at 22:41
"doesn't work" is bad explanation - please use debugger and step through the code to provide good details in the post. (F12 will bring dev tools in most desktop browsers) — Alexei Levenkov
– Alexei Levenkov, Commented May 5, 2014 at 22:45
@stackErr - why using new variable matters? it is global variable anyway, should not really matter if used in one or both loops as it is initialized to 0 anyway in both cases. — Alexei Levenkov
– Alexei Levenkov, Commented May 5, 2014 at 22:46
@stackErr: It can't be changed by async code. Async code is unable to break into the current synchronous execution of code. Even if it's timed such that it would have landed at that exact time, it has to wait. If that wasn't the case, then using a separate variable name wouldn't help since the async code could break in during the loop as well. — cookie monster
– cookie monster, Commented May 5, 2014 at 23:05

Christopher Thomas Nicodemus · Accepted Answer · 2014-05-05 22:51:55Z

2

I think you want to change all of the <= to <. If length is equal to 0, then it is going to execute once, but fail due to the length being zero. Which, I believe is the issue of it not reaching the second for loop, due to an error occurring on the first.

answered May 5, 2014 at 22:51

Christopher Thomas Nicodemus

4074 silver badges9 bronze badges

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

2 Comments

Qantas 94 Heavy Over a year ago

Actually, I don't think it matters whether or not t.length is 0.

Dude Alex Over a year ago

Thank you Christopher. By changing all <= to <, both of of the loop work. Since I know both t.length and d.length are not 0, but I think the script paused on error because if I write i<=t.length, when i=t.length, the script can't actually find the t[i] tag, because it doesn't exist, then the script hangs.

Collectives™ on Stack Overflow

Can't run two for loop statement within same if statement?

1 Answer 1

2 Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

2 Comments

Your Answer

Sign up or log in

Post as a guest

Related