Javascript - Find duplicates in an array

Asked May 29 '17 at 16:39

Active May 29 '17 at 16:50

Viewed 358 times

I want to find duplicates in a particular array, for that I have the following:

    this.drivers.filter((driver)=>{
      if(driver.userId == driverId){
      //this is used to updates the user location in real-time (lat, long), that's why I am removing the old location and pushing the new location
      this.removeDriver(driver.userId)
      this.pushNewDriver(lat, long, driverId);
}
else{
   //if the driver does not exists already in the array, then push it
        this.pushNewDriver(lat, long, driverId);
}
})



removeDriver(userId){
      this.drivers = this.drivers.filter((user) => {
      return user.userId !== userId;
    });
}

 pushNewDriver(lat, long, userId) {
let currentLontLat = { "lat": lat, "long": long };

 this.drivers.push({ "userId": userId, "currentLocation": currentLontLat });

}

What I want to do more specifically, is to update the info of a particular driver (e.g driver with Id : sajk2299o) however, with the code above, sometimes it works as expected (remove the driver from the array and push its new location) and sometimes it doesn't remove the old location and adds automatically the new location (somehow it does not respect driver.userId == driverId and go to else directly).

I believe that I am missing something here, but I can't figure it out.

edited May 29 '17 at 16:50

asked May 29 '17 at 16:39

Folky.H

1,164
4
15
37

1

Possible duplicate of [Easiest way to find duplicate values in a JavaScript array](https://stackoverflow.com/questions/840781/easiest-way-to-find-duplicate-values-in-a-javascript-array) – Andrea Carraro May 29 '17 at 16:42
can I see the removeDriver and pushNewDriver methods? – João Paulo Macedo May 29 '17 at 16:42
@banzomaikaka I've added it – Folky.H May 29 '17 at 16:48
Does this make any sense: https://jsfiddle.net/r6pzftsw/1/ ? – João Paulo Macedo May 29 '17 at 17:10
@banzomaikaka nope, mostly, the condition `driver.userId == driverId` does not always meet that's why it goes to `else` and it pushes a new element in the array – Folky.H May 29 '17 at 17:29
Your removeDriver() function doesn't have any arguments – Chris May 29 '17 at 16:49
modified, just a typo mistake – Folky.H May 29 '17 at 16:50

Javascript - Find duplicates in an array

0 Answers0