javascript/angularjs looping through an array show only one object

Question

in angularjs modal i am trying to display all classes that have level 4 but i got only one. please how should i alter my could in order to get all the class

$scope.openClass = function (classes) {
    $log.info("classes",classes);
    var modalInstance = $modal.open({
        templateUrl: 'classes.html',
        controller: 'ModalClassInstanceCtrl',
        resolve: {
            info: function () {
                var info = {};
                for (var i=0;i<classes.length;i++){                     
                    if (classes[i].level==4){                           
                        info['name']= classes[i].name;
                        $log.info("classinfo",info);                            
                    }
                }                   
                $log.info(info);
                return info;
            }
    }
});

the $log in if condition show me the write classes they are two in my case but the $log after the for loop show me only one

First of all, you should add var in your for loop. Without it, it assumes that iis a global variable. And if using anyway in the script anothervariable named i, you'll have some unexpected results. — ADreNaLiNe-DJ
– ADreNaLiNe-DJ, Commented Jul 6, 2016 at 11:12
And it constantly overwrites info['name'] in this code, so you'll only get 1 result. — Matheno
– Matheno, Commented Jul 6, 2016 at 11:15
i have added var to my for but how can i prevent it from overwrites , i tried to use break; but it just break after the object and never continue — george
– george, Commented Jul 6, 2016 at 11:22
Ofcourse it will stop when you call it to stop. Probably do something like info[]['name'] or info['name'].push(classes[i].name — Matheno
– Matheno, Commented Jul 6, 2016 at 11:23

developer033 · Accepted Answer · 2016-07-06 11:46:30Z

1

You could use the filter method, as the following:

var filteredArray = classes.filter(function(value) {
   return value.level == 4;
));

After it, if you want to pluck only the name of those items you can do the following:

var names = filteredArray.map(function(obj) {
  return obj.name;
}

If you are going to do this often, you could write a function that abstracts away the map, this way:

function pluck(array, key) {
  return array.map(function(item) {
    return item[key]; 
  }); 
}

edited Jul 6, 2016 at 11:46

answered Jul 6, 2016 at 11:32

developer033

25k8 gold badges85 silver badges113 bronze badges

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

1 Comment

george Over a year ago

thanks developer033 the filteredArray is what i want thanks a lot

Andrew Medvedsky · Accepted Answer · 2016-07-06 11:27:02Z

0

I think, you overwrite you param "name"

info['name']= classes[i].name;

If you want collection, you should

var info = [];
info.push(classes[i].name)

answered Jul 6, 2016 at 11:27

Andrew Medvedsky

832 silver badges9 bronze badges

Collectives™ on Stack Overflow

javascript/angularjs looping through an array show only one object

2 Answers 2

1 Comment

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

1 Comment

Comments

Your Answer

Sign up or log in

Post as a guest

Related