How can i prevent duplicate Record in Array

Question

I need to prevent inserting duplicate Rec in $scope.EmployeeList Array For that i wrote if ($scope.EmployeeList.indexOf(EmpDetails) == -1) but its not filtering my rec

 $scope.EmployeeList = [];
        var amtArray = [];
        $scope.G_Total = "Total Amount is";
        $scope.SaveDb = function (Isvalid) {
            var EmpDetails = {
                'EmpName': $scope.EmpName,
                'Email': $scope.Email,
                'Cost': $scope.cost

            }
            if ($scope.EmployeeList.indexOf(EmpDetails) == -1) {
                $scope.EmployeeList.push(EmpDetails);
                console.log($scope.EmployeeList);
            }
            else
                alert('Duplicate Value....');

Faly · Accepted Answer · 2017-11-16 06:36:28Z

1

Don't use indexOf (it checks for strict equality), try findIndex:

if ($scope.EmployeeList.findIndex(function(e) { return e.EmpName === EmpDetails.EmpName; }) === -1) {
    $scope.EmployeeList.push(EmpDetails);
    console.log($scope.EmployeeList);
}

edited Nov 16, 2017 at 6:36

answered Nov 16, 2017 at 6:26

Faly

13.4k2 gold badges21 silver badges39 bronze badges

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

5 Comments

Md Ghousemohi Over a year ago

Thank soo much for ur Answer

Md Ghousemohi Over a year ago

but its not find casesensitive Like Abc vs abc

Faly Over a year ago

That's not automatic, you have to turns name into lowercase: return e.EmpName.toLowerCase() === EmpDetails.EmpName.toLowerCase()

Md Ghousemohi Over a year ago

u mean return e.EmpName.toLowerCase === EmpDetails.EmpName.toLowerCase;

Faly Over a year ago

yes, return e.EmpName.toLowerCase() === EmpDetails.EmpName.toLoweCase();

Shaishab Roy · Accepted Answer · 2017-11-16 07:04:39Z

1

You can also use Array.prototype.some to find duplicate value.The some() method tests whether at least one element in the array passes the test implemented by the provided function.

var empDetails = {name: 'abc', email: '[email protected]'};

let duplicate = $scope.EmployeeList.some(function (emp) {
  return emp.name === empDetails.name && emp.email === empDetails.email;
});

if (!duplicate) {
   $scope.EmployeeList.push(EmpDetails);
}
else
   alert('Duplicate Value....');

answered Nov 16, 2017 at 7:04

Shaishab Roy

16.9k7 gold badges56 silver badges69 bronze badges

Collectives™ on Stack Overflow

How can i prevent duplicate Record in Array

2 Answers 2

5 Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

5 Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related