Cannot read property 'push' of undefined Javascript [duplicate]

Question

Hi it seams like i can't push my array ? code:

  $scope.arrResult = [];
  dpd.timesheets.get( function (result) {
    console.log(result);

    for (i = 0, n = result.length; i < n; i++) {
      var item = result[i];
      $scope.arrResult[item.week].push(item);
    }
    console.log($scope.arrResult);

  });

i get this console error

Uncaught TypeError: Cannot read property 'push' of undefined

if i set $scope.arrResult[item.week].push(item); to $scope.arrResult[item.week] = item; it works with no error but i need / want to push, whats wrong ?

AmmarCSE · Accepted Answer · 2015-05-20 13:07:06Z

12

This is because

$scope.arrResult[item.week]

itself is not an array.

push() is of Array.prototype

To see what I mean, try

$scope.arrResult[item.week] = [];

or

$scope.arrResult[item.week] = new Array();

and then try push()

edited May 20, 2015 at 13:07

answered May 20, 2015 at 13:02

AmmarCSE

30.7k6 gold badges48 silver badges53 bronze badges

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

13 Comments

Stweet Over a year ago

I see, mmm.. but how do i do this the correct way then, if i set = [] before push, i just get one object in the array, and i want all :)

AmmarCSE Over a year ago

@Stweet, what do you mean 'all'. Maybe don't even need an array?

Stweet Over a year ago

result contaions alot of objects, and i want this objects sortet in $scope.arrResult[week] eg. $scope.arrResult[21] contains all objects of week 21

AmmarCSE Over a year ago

@Stweet, Ah! Use concat stackoverflow.com/questions/4156101/…

AmmarCSE Over a year ago

@Stweet So, if item is an array, then do $scope.arrResult[item.week].concat(item);

|

Collectives™ on Stack Overflow

Cannot read property 'push' of undefined Javascript [duplicate]

1 Answer 1

13 Comments

Linked

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

13 Comments

Linked

Related