JavaScript. Check if value is not in object array

Question

I have 2 list. One bigger and one smaller. I have added infinite scroll to keep adding items from bigger list to smaller one.

// Bigger list (data from DB)
var data; 
// Smaller list 
$scope.projects = [];

$scope.$watch('projectSearch', function (val) {
    if (typeof val !== 'undefined' && val !== "") {
        //console.log("SEARCH: " + val);
        for (var a = 0; a < data.length; a++) {
            if (data[a].name.toLowerCase().indexOf(val) > -1) {
                console.log("FOUND: " + data[a].name);
                if($scope.projects.indexOf(data[a].name) === -1) {
                    console.log("PUSHED " + data[a].name);
                    $scope.projects.push( data[a]);

                } else {
                    console.log("ALREADY IN " + data[a].name);
                }
            }
        }
    }  
});

Search form:

<input id="search" ng-model="projectSearch" placeholder="Search ..."/>

The problem is it keeps pushing items to new list. It does not check this if statement correctly

if($scope.projects.indexOf(data[a].name) === -1) {
    console.log("PUSHED " + data[a].name);
    $scope.projects.push( data[a]);
}

You are not pushing the name. $scope.projects.push(data[a].name); — thefourtheye
– thefourtheye, Commented Jun 6, 2016 at 8:09

pwolaq · Accepted Answer · 2016-06-06 08:13:56Z

3

you should check for data[a], not its property name

if($scope.projects.indexOf(data[a]) === -1) {
    console.log("PUSHED " + data[a].name);
    $scope.projects.push( data[a]);
}

also, you could remove data[a] after copy to make next search faster

answered Jun 6, 2016 at 8:13

pwolaq

6,41221 silver badges45 bronze badges

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

Comments

asdf_enel_hak · Accepted Answer · 2016-06-06 08:11:06Z

1

$scope.projects should be type of string assuming data[a].name is string then your indexOf function properly

answered Jun 6, 2016 at 8:11

asdf_enel_hak

7,6585 gold badges47 silver badges91 bronze badges

1 Comment

Laurynas Over a year ago

I push only objects which contain characters from input field. I do not check if it's string.

Collectives™ on Stack Overflow

JavaScript. Check if value is not in object array

2 Answers 2

Comments

1 Comment

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

Comments

1 Comment

Your Answer

Sign up or log in

Post as a guest

Related