AngularJS/ JS How do I access variables outside my function?

Question

I'm trying to access the variable np defined from within a function block; however, I am running into some issues when I call this.items.push(plumbers). I get TypeError: Cannot call method push of undefined

myApp.factory('np', function($resource, nearbyPlumbers){
  var np = function(){
    this.items = [];
    this.busy = false;
    this.limit = 5;
    this.offset = 0;
  };

  np.prototype.nextPage = function(){
    if (this.busy) return;
    this.busy = true;

    var temp;

    nearbyPlumbers.nearby({lat: -37.746129599999996, lng: 144.9119861}, function(data){
      angular.forEach(data, function(plumber){
        alert('yay');
        //this.items.push(plumber);
        console.log(plumber);
        console.log(this.items); // <--- This wont work. How do I access this.items
      });
    });
  };
  return np;
});

zs2020 · Accepted Answer · 2013-09-01 05:49:07Z

1

np.prototype.nextPage = function () {
    if (this.busy) return;
    this.busy = true;

    var temp;
    var that = this; // add this line

    nearbyPlumbers.nearby({
        lat: -37.746129599999996,
        lng: 144.9119861
    }, function (data) {
        angular.forEach(data, function (plumber) {
            that.items.push(plumber); //access using "that"
            console.log(plumber);
            console.log(that.items);
        });
    });
};

answered Sep 1, 2013 at 5:49

zs2020

54.7k30 gold badges157 silver badges223 bronze badges

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

1 Comment

Ryan W Kan Over a year ago

AWESEOME! Works great!

Community · Accepted Answer · 2017-05-23 12:13:14Z

0

I'm really curious why you're using this, since it's going to be a different this depending on what scope accessing the singleton from. That would explain the error that you're getting.

I would strongly suggest reading up on factories in Angular and then taking another look at the code. The services docs are a good place to start and this question is good as well.

edited May 23, 2017 at 12:13

CommunityBot

11 silver badge

answered Sep 1, 2013 at 5:50

Jon7

7,2352 gold badges35 silver badges39 bronze badges

2 Comments

Ryan W Kan Over a year ago

I'm kinda hacking it atm, from this demo code binarymuse.github.io/ngInfiniteScroll/demo_async.html so yea, might be bad practice. Was hoping to get it to work then refactor.

Jon7 Over a year ago

@ryanwkan Good call. Sza's solution will definitely get it to a working state. Then you can refactor to make it more "angular-y". I can't put my finger on the "angular way" to do what you want to do, but I feel like there is a way.

Collectives™ on Stack Overflow

AngularJS/ JS How do I access variables outside my function?

2 Answers 2

1 Comment

2 Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

1 Comment

2 Comments

Your Answer

Sign up or log in

Post as a guest

Related