As far as I know, function foo() { aaa(); }
is just var foo = function(){ aaa() }
in JavaScript. So adding function foo() { bbb(); }
should either overwrite the foo
variable, or ignore the second definition - that's not the point. The point is that there should be one variable foo
.
So, in this example the me
variable should not be correctly resolved from inside the methods and it is not in Explorer 8 :-). I came to this example by trying to wrap them into another closure where (var
) me
would be, but I was surprised that it's not necessary:
var foo = {
bar1 : function me() {
var index = 1;
alert(me);
},
bar2 : function me() {
var index = 2;
alert(me);
}
};
foo.bar1(); // Shows the first one
foo.bar2(); // Shows the second one
Not quite; they're similar, but also quite different. JavaScript has two different but related things: Function declarations (your first example there), and function expressions (your second, which you then assign to a variable). They happen at different times in the parsing cycle and have different effects.
This is a function declaration:
Function declarations are processed upon entry into the enclosing scope, before any step-by-step code is executed.
This is a function expression (specifically, an anonymous one):
Function expressions are processed as part of the step-by-step code, at the point where they appear (just like any other expression).
Your quoted code is using a named function expression, which look like this:
(In your case it's within an object literal, so it's on the right-hand side of an
:
instead of an=
, but it's still a named function expression.)That's perfectly valid, ignoring implementation bugs (more in a moment). It creates a function with the name
foo
, doesn't putfoo
in the enclosing scope, and then assigns that function to thex
variable (all of this happening when the expression is encountered in the step-by-step code). When I say it doesn't putfoo
in the enclosing scope, I mean exactly that:Note how that's different from the way function declarations work (where the function's name is added to the enclosing scope).
Named function expressions work on compliant implementations. Historically, there were bugs in implementations (early Safari, IE8 and earlier). Modern implementations get them right, including IE9 and up. (More here: Double take and here: Named function expressions demystified.)
Actually, it should be. A function's true name (the symbol between
function
and the opening parenthesis) is always in-scope within the function (whether the function is from a declaration or a named function expression).NOTE: The below was written in 2011. With the advances in JavaScript since, I no longer feel the need to do things like the below unless I know I'm going to be dealing with IE8 (which is very rare these days).
Because of implementation bugs, I used to avoid named function expressions. You can do that in your example by just removing the
me
names, but I prefer named functions, and so for what it's worth, here's how I used to write your object:(Except I'd probably use a shorter name than
publicSymbols
.)Here's how that gets processed:
var foo = ...
line is encountered in the step-by-step code, and then it is called (because I have the()
at the very end).bar1_me
andbar2_me
function declarations are processed and those symbols are added to the scope inside that anonymous function (technically, to the variable object for the execution context).publicSymbols
symbol is added to the scope inside the anonymous function. (More: Poor misunderstoodvar
){}
topublicSymbols
.publicSymbols.bar1 = bar1_me;
andpublicSymbols.bar2 = bar2_me;
, and finallyreturn publicSymbols;
foo
.These days, though, unless I'm writing code I know needs to support IE8 (sadly, as I write this in November 2015 it still has significant global market share, but happily that share is plummetting), I don't worry about it. All modern JavaScript engines understand them just fine.
You can also write that like this:
...since those are function declarations, and thus are hoisted. I don't usually do it like that, as I find it easier to do maintenance on large structures if I do the declaration and the assignment to the property next to each other (or, if not writing for IE8, on the same line).