1

I have picked up the habit of wrapping all of my case statements in curly brackets from programming in C because of this but JSLint is throwing a fit. It stops validating at that point.

My question is: Is this such a bad practice in JS? Do I not have to worry about the scope issue because JS has function scope (I understand how that would be the case, I just want a good reason not to be 'consistent' on this)?

(I know that different languages call for different practices, but i am trying to be as consisten as possible across languages to help protect my sanity.)

Community
  • 1
  • 1
griotspeak
  • 13,022
  • 13
  • 43
  • 54

1 Answers1

4

Good question.

The reason that JSLint complains about this is because it is actually contrary to the language specification for the switch/case statments:

http://www.ecma-international.org/publications/files/ECMA-ST/ECMA-262.pdf

https://developer.mozilla.org/en/JavaScript/Reference/Statements/switch

The JavaScript compier will tolerate it, however JSLint is about ensuring your code adheres to a stricter and more correct subset of JavaScript. ("The Good Bits" as Douglas Crockford puts it!)

Moreover, the extra bracing is extra characters that you will have to transmit with the website. If you don't need them, why transmit them?

Referring to your above link, the same problem does not present itself in JavaScript. So, the following will work:

var x = 0;
switch(x){
    case 0:
        var y = 1;
        alert(y);
}

See it at this JSFiddle: http://jsfiddle.net/LKWwB/

Finally, regarding your sanity, I would relinquish the tenuous grip you have on it. I did years ago and am much happier for it :-)

James Wiseman
  • 29,946
  • 17
  • 95
  • 158
  • Many thanks. Good to know that the answer really is as simple as "because that is incorrect!" – griotspeak Jan 25 '11 at 17:11
  • I see no proof that this is "contrary to the language specification". Where does it say that in the links you provided? – Gili May 31 '13 at 16:16