JavaScript nested function prototype scope

Question

I'm still having trouble figuring on how to manage scopes in JavaScript. In this particular example, I have a draw function containing certain properties and a function that needs to draw lines based on an array.

function Draw (canvas)
{
    this.ctx = canvas.getContext('2d');
    this.street_size = 20;
}

Draw.prototype.street = function (MAP)
{

    MAP.forEach(function (name)
    {
        this.ctx.moveTo(name.start.x,name.start.y);
        this.ctx.lineTo(name.end.x,name.end.y)
        this.ctx.stroke();
    });
}

Of course, "this.ctx" inside the forEach function returns "undefined". How can I make sure that Draw()'s variables are passed to the forEach function (without doing something like ctx = this.ctx)?

Felix Kling · Accepted Answer · 2013-02-11 01:32:21Z

7

You can use .bind ^[MDN]:

MAP.forEach(function (name) {
    this.ctx.moveTo(name.start.x,name.start.y);
    this.ctx.lineTo(name.end.x,name.end.y)
    this.ctx.stroke();
}.bind(this));

Learn more about this.

answered Feb 11, 2013 at 1:32

Felix Kling

820k181 gold badges1.1k silver badges1.2k bronze badges

Sign up to request clarification or add additional context in comments.

Comments

Explosion Pills · Accepted Answer · 2013-02-11 01:34:08Z

4

It's common to declare the object instance variable as a new variable inside the method scope:

var self = this;
MAP.forEach(function (name) {
    self.ctx.moveTo(...

This has the advantage of allowing you to continue to use this as it would be ordinarily.

answered Feb 11, 2013 at 1:34

Explosion Pills

192k56 gold badges341 silver badges417 bronze badges

Comments

the system · Accepted Answer · 2013-02-11 01:32:55Z

3

Pass this as the second argument to forEach().

MAP.forEach(function (name)
{
    this.ctx.moveTo(name.start.x,name.start.y);
    this.ctx.lineTo(name.end.x,name.end.y)
    this.ctx.stroke();
}, this);

The second argument sets the value of this in the callback.

MDN forEach() docs - array.forEach(callback[, thisArg])

answered Feb 11, 2013 at 1:32

the system

9,35642 silver badges47 bronze badges

1 Comment

Felix Kling Over a year ago

+1. Wasn't sure forEach supports it, but it does: es5.github.com/#x15.4.4.18.

Collectives™ on Stack Overflow

JavaScript nested function prototype scope

3 Answers 3

Comments

Comments

1 Comment

Linked

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

Comments

Comments

1 Comment

Linked

Related