how to get all values in for loop angular js

Question

when I alert this I only get the last value of id. ican't findout where I'm going wrong

  $scope.addToList = function (products,qty) {
    if ($scope.order.length > 0) {
        for (var i = 0; i < $scope.order.length; i++) {
            $scope.Thx=$scope.order[i].id;
            }}};

   alert($scope.Thx);

What do you think $scope.Thx=$scope.order[i].id; is doing? Inside the loop it is resetting the value of $scope.Thx on every loop. Thus, when you alert after the loop, it will be the very last value. — rmlan
– rmlan, Commented May 22, 2018 at 11:58
$scope.Thx should be an array. Then append values to it with $scope.Thx.push( ... ) — Aleksey Solovey
– Aleksey Solovey, Commented May 22, 2018 at 11:58

Satpal · Accepted Answer · 2018-05-22 12:05:11Z

3

Declare the $scope.Thx as an array first.

$scope.Thx = [];

$scope.addToList = function (products, qty) {
    if ($scope.order.length > 0) {
        for (var i = 0; i < $scope.order.length; i++) {
            $scope.Thx.push($scope.order[i].id);
        }
    }
};

edited May 22, 2018 at 12:05

Satpal

134k13 gold badges168 silver badges171 bronze badges

answered May 22, 2018 at 12:04

Sravya Nagumalli

7481 gold badge6 silver badges8 bronze badges

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

Comments

Harsh Jaswal · Accepted Answer · 2018-05-23 04:43:49Z

This line will overwrite the $scope.Thx for each iteration inside the loop.

$scope.Thx=$scope.order[i].id;

Now there are two ways to get the desired result. 1. If you want the separate alert for each item in the array then:

$scope.addToList = function (products,qty) {
if ($scope.order.length > 0) {
    for (var i = 0; i < $scope.order.length; i++) {
        $scope.Thx=$scope.order[i].id;
        alert($scope.Thx);
        }}};

2. If you want single alert for all the items. declare $scope.Thx as an array first.

$scope.Thx = [];
$scope.addToList = function (products, qty) {
if ($scope.order.length > 0) {
    for (var i = 0; i < $scope.order.length; i++) {
        $scope.Thx.push($scope.order[i].id);
    }
}
};
var tempVar="";
for(var i=0; i<$scope.Thx.length;i++)
{
    tempVar += $scope.Thx[0]+"\n";
}
alert(tempVar);

Collectives™ on Stack Overflow

how to get all values in for loop angular js

2 Answers 2

Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related