push.apply() not increasing array size

Question

I have this piece of code

if ($scope.newLeverAdded) {
    console.log("executed this");
    console.log($scope.tradeDataObj, "data is here");
    console.log($scope.group_levers, "this 2 here");
    $scope.group_levers.levers.push.apply($scope.group_levers.levers, $scope.tradeDataObj.levers[0]);

    console.log($scope.group_levers, "mid prrinting");
    console.log("after this");
}
console.log($scope.group_levers, "final prrinting");

All i am trying is to increase my Array size by adding new element to it.. SO that now the array contains 2 element. But the final console still returns a single element in array.

Why is it so?

2nd argument to `apply` need to be an array. Is `$scope.tradeDataObj.levers[0]` is an array? — , Oct 04 '15 at 05:16
Why are you using `apply`? Is there a specific reason for that? — , Oct 04 '15 at 05:19
No..there is no specific reason. All I need is to preserve previous array data and push new data..so that I can send complete JSON back to server. I would really appreciate if some one could illustrate when is the need for .apply() withpush — Saurabh Tiwari, Oct 04 '15 at 07:14

score 1 · Answer 1 · answered Oct 04 '15 at 05:14

1

Without being able to run your code, here's a couple of things you can try:

it could be that your code is not entering the condition that you have specified.
why not just test with a $scope.group_levers.levers.push('new element') instead of the apply

answered Oct 04 '15 at 05:14

filype

8,034
10
40
66

Wierdly..when i tried .push($scope.tradeDataObj.levers[0]). The IDE throws syntax error sayong expected identifier – Saurabh Tiwari Oct 04 '15 at 05:24
I am assuming `$scope.group_levers.levers` is an array. – filype Oct 04 '15 at 05:26
Ya that surely an array...Thats why it was weird to see such response from IDE. Visual Studio, if that matters – Saurabh Tiwari Oct 04 '15 at 05:40

score 1 · Accepted Answer · edited May 23 '17 at 11:44

1

I think you need:

$scope.group_levers.levers.push.apply($scope.group_levers.levers, $scope.tradeDataObj.levers);

The second argument of apply() should be an array.

I assume that you are trying to push multiple items like this and this

edited May 23 '17 at 11:44

Community

1
1

answered Oct 04 '15 at 05:20

Mohayemin

3,841
4
25
54

Mohayemin..thanx for pointing to right direction. I guessed, if the original source is array, only object should be added to it not the array again. Hence was doing levers[0]. Is this specifically the case fr apply – Saurabh Tiwari Oct 04 '15 at 05:37

push.apply() not increasing array size

2 Answers2