Return object in array with specific property

Question

I'm fairly new with writing my own JS functions, and I'm struggling with this one.

I want to run through an array of objects, find an object that matches a particular ID, and then return that object.

So far this is what I have:

var findTeam = function() {
  $scope.extraTeamData.forEach(team) {
     if(team.team_id === $scope.whichTeam) { return team }
  }
    $scope.thisTeam = team;
};

$scope.teamDetails is my array, and the $scope.whichTeam variable holds the correct ID which I am checking against.

Ultimately I want to be able to assign the object that results from the function to the $scope.thisTeam variable, so I can call its properties in the view.

Any help would be appreciated.

Thanks.

Can you tell us what issue(s) you're experiencing with your current implementation? — smaili, May 19 '16 at 18:02
Instead of `return team`, you could just do `$scope.thisTeam = team;` — Matt Burland, May 19 '16 at 18:02
Thanks Matt Burland - that trims it down a bit. I want to write it in the most efficient way possible, and to ultimately result in one variable which holds the desired object. — Paulos3000, May 19 '16 at 18:04
forEach requires a callback. your `forEach(team) {` is syntacticaly wrong. it should be `forEach(function (team) {` or if ES6 then `forEach(team => {`. and there is a missing closing parenteses. — Nina Scholz, May 19 '16 at 18:09

score 2 · Answer 1 · edited May 19 '16 at 18:27

2

You could use Array#some which ends the iteration if found

var findTeam = function() {
    $scope.extraTeamData.some(function (team) {
        if (team.team_id === $scope.whichTeam) { 
            $scope.thisTeam = team;
            return true;
        }
    });
};

edited May 19 '16 at 18:27

Paulos3000

3,355
10
35
68

answered May 19 '16 at 18:03

Nina Scholz

376,160
25
347
392

I put this into my controller in Angular and it broke the functionality altogether... :/ – Paulos3000 May 19 '16 at 18:09
1

I think I may have a problem with my original array. Thanks though - your code has helped me to structure the function :) – Paulos3000 May 19 '16 at 18:14

score 1 · Answer 2 · answered May 19 '16 at 18:03

1

Move your $scope.thisTeam = team; to within the if check.

var findTeam = function() {
  $scope.teamDetails.forEach(team) {
     if(team.team_id === $scope.whichTeam) {
         $scope.thisTeam = team;
     }
  }
};

answered May 19 '16 at 18:03

smaili

1,245
9
18

score 0 · Answer 3 · answered May 19 '16 at 18:03

$scope.team = $scope.teamDetails.filter(function (team) {
  return team.team_id === $scope.whichTeam;
})[0];

You need to use filter method of array. It creates new array elements that match given predicate (function that return boolean value). Then you simply take first value.

You can also use find, but it is not implemented in every browser yet.

It would look something like this:

$scope.team = $scope.teamDetails.find(function (team) {
  return team.team_id === $scope.whichTeam;
});

Return object in array with specific property

3 Answers3