The context of the closure after the loop should be changed, currently
it points to the `window` object. It seems like a typo, as it's even
noted that you have to "...make sure that it's properly scoped...".
})(); } -> }).call(this); }
On Dec 18, 2:08 pm, Pablo Fernandez <fernandezpabl...@gmail.com>
wrote:
> Hi, I'm reading "Pro Javascript Techniques" from John Resig, and I'm
> confused with an example. This is the code:
>
> // Create a new user object that accepts an object of properties
> function User( properties ) {
> // Iterate through the properties of the object, and make sure
> // that it's properly scoped (as discussed previously)
> for ( var i in properties ) { (function(){
> // Create a new getter for the property
> this[ "get" + i ] = function() {
> return properties[i];
> };
> // Create a new setter for the property
> this[ "set" + i ] = function(val) {
> properties[i] = val;
> };})(); }
> }
>
> // Create a new user object instance and pass in an object of
> // properties to seed it with
> var user = new User({
> name: "Bob",
> age: 44});
>
> // Just note that the name property does not exist, as it's private
> // within the properties object
> alert( user.name == null );
> // However, we're able to access its value using the new getname()
> // method, that was dynamically generated
> alert( user.getname() == "Bob" );
> // Finally, we can see that it's possible to set and get the age using
> // the newly generated functions
> user.setage( 22 );
> alert( user.getage() == 22 );
> Now running that on Firebug console (on FF3) throws that user.getname
> () is not a function. I tried doing this:
>
> var other = User
> other()
> window.getname() --> this works!
> And it worked!
>
> Any idea why? thanks everybody!
>
> PS: I know this is not an specific Jquery thing but a Javascript
> question, but I didn't know where to ask.
