Array.find(value) return value 'is not a function'

Question

I'm trying to use JavaScript's find() function on an AngularJS array. That's legal, right...?

This very simple code is causing me some problems. It's saying that the return value from $scope.names.find(name1) is not a function.

TypeError: Name1 is not a function

if($scope.names.find(name1) !== name1) {
   $scope.names.push(name1);
}

I have also tried...

if($scope.names.find(name1) === undefined) {
   $scope.names.push(name1);
}

and

if(!$scope.names.find(name1)) {
   $scope.names.push(name1);
}

This if is in a loop. If name is not in the array, then add it. If it is already in the array, don't add it.

Here is the loop:

angular.forEach($scope.names1, function (name1) {
    angular.forEach($scope.names2, function (name2) {
        if (name1 === name2) {
            $scope.names.push(name1);
        }
    });
    if ($scope.names.find(name1) === name1) {
        $scope.names.push(name1);
    }
});

I don't know what the error is referring to exactly. I must be misusing find().

As you can see in Docs find takes callback function as argument, use if (!$scope.names.find(function (name1) { $scope.names.push(name1); }) } — Tushar
– Tushar, Commented Oct 7, 2015 at 16:08
TypeError: name1 is not a function because .find takes a callback function and executes it on array items, and returns the value of an item that satisfies the callback function. developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/… — pawel
– pawel, Commented Oct 7, 2015 at 16:10
@DaveNewton The error is self-explanatory, it says that the passed parameter is not a function, that means parameter should be a function — Tushar
– Tushar, Commented Oct 7, 2015 at 16:10

Vineet · Accepted Answer · 2015-10-07 16:13:11Z

5

You need to use indexOf. Your snippet would be

if($scope.names.indexOf(name1) < 0) { //Value not exists in your array
   $scope.names.push(name1);
}

edited Oct 7, 2015 at 16:13

answered Oct 7, 2015 at 16:10

Vineet

4,7153 gold badges25 silver badges42 bronze badges

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

2 Comments

sdgluck Over a year ago

An index of zero means the element is at the zeroth position and therefore does exist in the array. You can instead do $scope.names.indexOf(name1) === -1.

pawel Over a year ago

Or $scope.names.includes(name1).

jbrown · Accepted Answer · 2015-10-08 00:40:59Z

0

Alternatively, consider using underscore.js There are a lot of useful functions including _.find()

answered Oct 8, 2015 at 0:40

jbrown

3,0351 gold badge18 silver badges22 bronze badges

Collectives™ on Stack Overflow

Array.find(value) return value 'is not a function'

2 Answers 2

2 Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

2 Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related