using indexOf to see if there is a match in an object in angularJS

Question

i want to loop through an array of application numbers to see if there is a match using indexOf.

currently my array looks like..

array:

"applications":[  
         "123456",
         "224515",
         "658454",
         "784123"
      ]

my controller:

$scope.hasApplicationNumber = function (appNo) {
        var applicationsArray = applications;
        return applicationsArray.indexOf(appNo);
      }

html:

<span ng-if="hasApplicationNumber(784123)">Its a Match!</span>

indexOf doesn't return a boolean.

Ben
– Ben

2017-05-09 19:50:01 +00:00
Commented May 9, 2017 at 19:50 — Ben
– Ben, Commented May 9, 2017 at 19:50

Rohìt Jíndal · Accepted Answer · 2017-05-10 08:54:25Z

1

DEMO

var myApp = angular.module('myApp',[]);

myApp.controller('MyCtrl', function($scope) {
    $scope.applications = [  
         "123456",
         "224515",
         "658454",
         "784123"
      ];

    $scope.hasApplicationNumber = function (appNo) {
        var res = $scope.applications.filter(item => { return item.indexOf(appNo) != -1; });
        return (res.length) ? true : false;
      }  
});

<div ng-app="myApp" ng-controller="MyCtrl">
  <span ng-if="hasApplicationNumber(784123)">Its a Match!</span>
</div>

answered May 10, 2017 at 8:54

Rohìt Jíndal

27.4k16 gold badges80 silver badges133 bronze badges

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

Comments

Bergur · Accepted Answer · 2017-05-09 20:27:40Z

0

Am I misunderstanding something? Why not just use Array.prototype.includes?

Disclaimer: This uses ES6 arrow syntax,

$scope.hasApplicationNumber = function (appNo) {
    return applicationsArray.map(no => Number(no)).includes(appNo);
}

JS Fiddle Link: https://jsfiddle.net/3p4mq9qL/

Edit: Fixed if you want to look by number (when the array is string)

edited May 9, 2017 at 20:27

answered May 9, 2017 at 20:03

Bergur

4,07714 silver badges21 bronze badges

1 Comment

couzzi Over a year ago

Worth noting that this uses ES6. A beginner might not know that, and get frustrated trying to use ES6 without Babel or similar.

Pankaj Parkar · Accepted Answer · 2017-05-10 09:40:04Z

0

indexOf method returns a index of matched number. So you should be checking if its greater than -1. Over here you were checking number instead of string(array member), so before checking you have to convert all array to Number and check otherwise check appNo to string then find it in an array.

$scope.hasApplicationNumber = function (appNo) {
    var applicationsArray = applications;
    //return applicationsArray.indexOf(appNo.toString()) > -1;
    //OR
    //return !!~applicationsArray.map(Number).indexOf(appNo); //then check number
    //OR
    return !!~applicationsArray.indexOf(appNo.toString());
}

edited May 10, 2017 at 9:40

answered May 9, 2017 at 19:48

Pankaj Parkar

136k23 gold badges241 silver badges307 bronze badges

3 Comments

Joe Over a year ago

If you want to confuse people you can use return !!~applicationsArray.indexOf(appNo);

Phix Over a year ago

Bang bang squiggle

Pankaj Parkar Over a year ago

@Joe cool man. That's fantastic, I made an update to answer :)

Collectives™ on Stack Overflow

using indexOf to see if there is a match in an object in angularJS

3 Answers 3

Comments

1 Comment

3 Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

Comments

1 Comment

3 Comments

Your Answer

Sign up or log in

Post as a guest

Related