Weird hiccup удаление дубликатов из массива

Question

Я бегу в странный сбой. У меня есть немного кода, в котором я запускаю массив массивов, захватывая кучу названий городов и объединяя их все вместе. Мне нужно удалить дубликаты из готового списка. Это должно быть довольно просто. Используйте подсчет, чтобы выяснить, в каком городе более одного экземпляра, а затем сплайсируйте их. Мой возвращенный массив не выходит правильно, хотя и я не уверен, почему. Может ли кто-нибудь определить, что я делаю неправильно?

const input = [ 
{ 
    name: "ACH2000", 
    year: 2005, 
    cities: ['Chicago', 'New York', 'Ames', 'Columbus'], 
    ages: [12, 32, 2, 51] 
}, 
{ 
    name: "FXG3000", 
    year: 2008, 
    cities: ['Chicago', 'Joliet', 'Plymouth', 'Dallas'], 
    ages: [12, 32, 2, 51] 
}, 
{ 
    name: "GTG1234", 
    year: 2012, 
    cities: ['Indy', 'Tampa', 'Houston', 'Dallas'], 
    ages: [12, 32, 2, 51] 
} 
]; 

function getUniqueCities(data){ 
    let citiesInArray = data.map(function(item){ return item.cities }); 
    let concatCities = [].concat.apply([], citiesInArray); 
    let count = {}; 
    for(let i = 0; i< concatCities.length; i++) { 
     let num = concatCities[i]; 
     count[num] = count[num] ? count[num]+1 : 1; 
     if(count[num] > 1){ 
     console.log('bad',num); 
     concatCities.splice(num, 1); 
     } else { 
     console.log('good',num); 
     } 
    } 
    console.log(count); 
    console.log(concatCities); 
} 

getUniqueCities(input); 

Answer 1

вы можете попробовать что-то вроде этого

var input = [ 
    { 
     name: "ACH2000", 
     year: 2005, 
     cities: ['Chicago', 'New York', 'Ames', 'Columbus'], 
     ages: [12, 32, 2, 51] 
    }, 
    { 
     name: "FXG3000", 
     year: 2008, 
     cities: ['Chicago', 'Joliet', 'Plymouth', 'Dallas'], 
     ages: [12, 32, 2, 51] 
    }, 
    { 
     name: "GTG1234", 
     year: 2012, 
     cities: ['Indy', 'Tampa', 'Houston', 'Dallas'], 
     ages: [12, 32, 2, 51] 
    } 
]; 

var citiesStats = {}; 

input.forEach(data => 
    data.cities.forEach(city => { 
     if (!citiesStats[city]) { 
      citiesStats[city] = 0; 
     } 
     ++citiesStats[city]; 
    }) 
); 

var cities = Object.keys(citiesStats); 

// ["Chicago", "New York", "Ames", "Columbus", "Joliet", "Plymouth", "Dallas", "Indy", "Tampa", "Houston"] 
console.log(cities); 
// {"Chicago":2,"New York":1,"Ames":1,"Columbus":1,"Joliet":1,"Plymouth":1,"Dallas":2,"Indy":1,"Tampa":1,"Houston":1} 
console.log(citiesStats); 

Answer 2

Как nnnnnn предложил сплайсинг внутри цикла Мессинг индексы в массиве.

Если вы можете использовать Set, вот решение:

Array.from(new Set(concatCities)) 

Вот ссылка на fiddle.