views:

1293

answers:

6

The behavior of "this" when function bar is called is baffling me. See the code below. Is there any way to arrange for "this" to be a plain old js object instance when bar is called from a click handler, instead of being the html element?

// a class with a method

function foo() {

    this.bar();  // when called here, "this" is the foo instance

    var barf = this.bar;
    barf();   // when called here, "this" is the global object

    // when called from a click, "this" is the html element
    $("#thing").after($("<div>click me</div>").click(barf));
}

foo.prototype.bar = function() {
    alert(this);
}
+1  A: 

This is because this is always the instance that the function is attached to. In the case of an EventHandler it is the class that triggered the event.

You can help your self with an anonymous function like this:

function foo() {
  var obj = this;
  $("#thing").after($("<div>click me</div>").click(function(){obj.bar();}));
}

foo.prototype.bar = function() {
  alert(this);
}
Martin
+1  A: 

You may use Function.apply on the function to set what this should refer to:

$("#thing").after($("<div>click me</div>").click(function() {
    barf.apply(document); // now this refers to the document
});
moff
Beside the fact that there is a missing closed bracket at your code - the function *apply* instantly executes the function *barf* instead of returning a function pointer.
Martin
Thanks, I've now corrected the post :)
moff
+13  A: 

Welcome to the world of javascript! :D

You have wondered into the realm of javascript scope and closure.

For the short answer:

this.bar()

is executed under the scope of foo, (as this refers to foo)

var barf = this.bar;
barf();

is executed under the global scope.

this.bar basically means:

execute the function pointed by this.bar, under the scope of this (foo). When you copied this.bar to barf, and run barf. Javascript understood as, run the function pointed by barf, and since there is no this, it just runs in global scope.

To correct this, you can change

barf();

to something like this:

barf.apply(this);

This tells Javascript to bind the scope of this to barf before executing it.

For jquery events, you will need to use an anonymous function, or extend the bind function in prototype to support scoping.

For more info:

Aaron Qian
+2  A: 

There's a good explanation on this keyword in JavaScript available at QuirksMode.

Pawel Krakowiak
+2  A: 

Get the book: JavaScript: the Good Parts.

Also, read as much as you can by Douglas Crockford http://www.crockford.com/javascript/

Chris Brandsma