Comments on Inner Exception: Tip: Make sure you declare JavaScript For loops co...

It looks like the let block scopes variable...

2007-04-30T12:59:00.000-04:00

It looks like the let block scopes variables to the block level in JavaScript 1.7 with Firefox 2.

@brook, joe, + andrew: From Crockford: "JavaScript...

2007-04-30T11:43:00.000-04:00

@brook, joe, + andrew: From Crockford: "JavaScript's C-like syntax, including curly braces and the clunky for statement, makes it appear to be an ordinary procedural language. This is misleading because JavaScript has more in common with functional languages like Lisp or Scheme than with C or Java."

This sums up my confusion on scoping in a nutshell. It never occurred to me that if you did for(var i... that i was scoped at the function anyway. It also feels like I living in crazy town that if you don't use var, that i was globally scoped, I expected it was either scoped at the loop, or if you had quizzed me I would have said the function before reading all these comments, never would I have answered global. So yeah brook, I don't like the design of JavaScript here. It looks and feels like a C-derived language, but when things like this don't work like I expect from C-derived language experience, well its time to get back to basics. I am going to pick up JavaScript: The Definitive Guide based on Crockford's recommendation:
http://www.amazon.com/exec/obidos/ASIN/0596101996/wrrrldwideweb

@Joshua Paine: The developer that orignally wrote ...

2007-04-30T11:32:00.000-04:00

@Joshua Paine: The developer that orignally wrote the JavaScript was unintentionally declared i in the global namespace. I don't know you would ever want to do this on purpose either, but maybe someone has. And I am sorry, I dislike your if example, sacrificing clarity for brevity (and reducing 1 line at that) just doesn't seem worth it to me. You might understand what you wrote when you wrote it, but if you have to go back to this code and you have been using a language that has = as an equality operator, not just an assignment, the fog of confusion can set in.

@anonymous: I haven't been schooled like that in a...

2007-04-30T11:18:00.000-04:00

@anonymous: I haven't been schooled like that in a long time ;-). Here is some sample code (greatly simplified from the original) that will cause Firefox to use 100% CPU for a while:

function multipleLoops() {
var startIndex = 0;
var endIndex = 1000;
var count = 0;

for(i=startIndex; i < endIndex; i++) {
count = count + doSomeStuff();
}

alert(i);
}

function doSomeStuff() {
var count = 0;

for(i=0; i < 10; i++) {
count = count + i;
}

return count;
}

This is pretty low complexity code, no branches obviously, and now both Firefox 2.0.0.3 and IE 7.0.5730.11 prompt to kill the "unresponsive script", though IE does it nearly immediately, Firefox lets it spin for 5-10 seconds. So Anon you were totally right that something else was "manipulating i", but you are wrong that we have separate code running for IE and Firefox, its all the same.

Also, javascript limits scope only within function...

2007-04-29T17:40:00.000-04:00

Also, javascript limits scope only within functions. Putting var within a for loop does not limit the scope of i to the loop. You can only limit the scope of i to within it's enclosing function. So to avoid confusion (as in the above) it's often recommended to var all your variables at the start of functions, rather than when they are first assigned.

Simplifying what Brook said: this is how variable ...

2007-04-29T11:14:00.000-04:00

Simplifying what Brook said: this is how variable scope is supposed to work in JavaScript.

As a result, if the body of the "for" loop decrements the value of the counter, you can get an infinite loop.

Your complaint isn't with the browsers. Your comp...

2007-04-29T09:27:00.000-04:00

Your complaint isn't with the browsers. Your complaint is with ECMA-262 12.6.3, which explicitly permits for() without var in the first expression list. Were IE and FF to get sticky over that point, they'd not be compliant with the specification.

Anonymous is correct (about everything), and I'm g...

2007-04-29T08:00:00.000-04:00

Anonymous is correct (about everything), and I'm glad no browsers flag failing to use var there as an error, because it's not. Two reasons:

1) This isn't really correct:

function bob()
{
var str='';
for(var i=0; i<5; i++)
str+='yo';
for(var i=0; i<5; i++)
str+='hi';
alert(str);
}

Because i has already been declared with var, there's no need to do it again in the second loop, though Firefox at least doesn't raise an error there either.

2) Not declaring your counter with var at all and thus polluting the global namespace and leaving yourself open to the weird interactions you experienced is something I have yet to hear of anyone doing knowingly on purpose, and it should never be done. But never is a very long time, and there might be a reason to do it that I can't think of (especially if, e.g., you did it in an inner function and the counter was declared in the containing function), so I'd hesitate to disallow it.

In a less extreme example, Crockford and (I'm sure) many others think this is a never-do:

if(resultINeedLater = someFunction())

but I like the line of code it saves me, and I rarely make mistakes about assignment instead of equality, and if I do I do not find them subtle bugs but catch them easily. Just some preference of my brain that I'm glad I can indulge even though it seems like a bad idea to some.

My apologies, code snippet should be (hopefully th...

2007-04-29T02:22:00.000-04:00

My apologies, code snippet should be (hopefully this comes out right):

/* prints 0 to 999 in both IE and Firefox */
var endIndex=1000;
var startIndex=0;

for(i=startIndex; i < endIndex; i++) {
document.write(i + '<br>');
//Do some stuff here
}

Your evaluation of what is going on in the Javascr...

2007-04-29T02:18:00.000-04:00

Your evaluation of what is going on in the Javascript is incorrect. "i" is not initialized to startIndex at the beginning of each iteration of the loop in Firefox:

/* prints 0 to 999 in both IE and Firefox */
var endIndex=1000;
var startIndex=0;

for(i=startIndex; i < endIndex; i++) {
document.write(i + '>br>');
//Do some stuff here
}

My guess is that your "//Do some stuff here" manipulated (or called functions which manipulated) a variable "i" as well, and in Javascript, if you use a variable without declaring it with "var", the scope is global (otherwise it is local to the containing function). Why this does not affect Internet Explorer is anyone's guess. Without seeing the remainder of the source code, I'd speculate that there is separate code to support IE and Firefox and only the Firefox code uses another global instance of "i".

However, as the sample code above demonstrates (tested in Firefox 2.0.0.3 and IE 7), the loop completes normally. There is no bug, error or differences between the way IE and Firefox handle the scoping rules and loop.