Why do the code snippets below, taken from this article, produce different results due to only a single change in the placement of curly braces?
When the opening curly brace {
is on a new line, test()
returns undefined
, and "no - it broke: undefined" is displayed in the alert.
function test()
{
return
{ /* <--- curly brace on new line */
javascript: "fantastic"
};
}
var r = test();
try {
alert(r.javascript); // does this work...?
} catch (e) {
alert('no - it broke: ' + typeof r);
}
When the brace is on the same line as return
, test()
returns an object, and "fantastic" is alerted.
function test()
{
return { /* <---- curly brace on same line */
javascript: "fantastic"
};
}
var r = test();
try {
alert(r.javascript); // does this work...?
} catch (e) {
alert('no - it broke: ' + typeof r);
}
I personally prefer the Allman Style for readability (vs K&R style).
Instead of…
I like…
But this is a work-around. I can live with it though.
It's because javascript most often puts ";" at the end of each line, so basicly when you have return { in same line, javascript engine see that there will be something more, and when its in new line it thinks you forgot to put ";", and puts it for you.
Javascript doesn't require semicolons at the end of statements, but the drawback is that it has to guess where the semicolons are. Most of the time this is not a problem, but sometimes it invents a semicolon where you didn't intend one.
An example from my blog post about this (Javascript – almost not line based):
If you format the code like this:
Then it is interpreted like this:
The return statement takes it’s parameterless form, and the argument becomes a statement of it’s own.
The same happens to your code. The function is interpreted as:
That's one of the pitfalls of JavaScript: automatic semicolon insertion. Lines that do not end with a semicolon, but could be the end of a statement, are automatically terminated, so your first example looks effectively like this:
See also Douglas Crockford's JS style guide, which mentions semicolon insertion.
In your second example you return an object (built by the curly braces) with the property
javascript
and its value of"fantastic"
, effectively the same as this:The curly braces here indicate the construction of a new object. Thus your code is equivalent to:
which works whereas if you write:
it no longer works.
The problem is indeed semicolon injection as described above. I just read a good blog posting on this subject. It explains this issue and a whole lot more about javascript. It also contains some good references. You can read it here