Javascript - liczenie wielokrotnych wystąpień w tablicy

Question

Witam.

Na stackoverflow znalazłem taki kod: 

uniqueCount = ["a","b","c","d","d","e","a","b","c","f","g","h","h","h","e","a"];

var  count = {}; 

uniqueCount.forEach(function(i) { count[i] = (count[i]||0)+1;  });

// count = {"a":3,"b":2,"c":2,"d":2,"e":2,"f":1,"g":1,"h":3}

Ten kod liczy ile jest wystąpień tej samej wartości tablicy. Jak zrobić, 'count' zawierało same wartości, które się duplikują? 

Pozdrawiam.

Answer 1

Na pewno można skrócić ten kod, ale aby obiekt count zawierał tylko duplikaty (czyli, moim zdaniem wartości występujące dokładnie 2 razy) - możesz zrobić to w następujący sposób (poprzez usunięcie wartości, które nie są duplikatami - wtedy zostaną tylko duplikaty):

var notDupes = [];

for ( var key in count ) {
    if ( count.hasOwnProperty( key ) ) {
       if ( count[ key ] !== 2 ) { // jesli wartosc klucza jest inna niz 2 (czyli nie jest duplikatem), to dodaj do tablicy nie zawierajacej duplikaty
          notDupes.push( key );
       }
    }
}

for ( var i = 0, len = notDupes.length; i < len; i++ ) {
   delete count[ notDupes[ i ] ]; // usuwasz z obiektu 'count' klucze, ktore nie zawieraja duplikatow - czyli zostana same duplikaty
}

console.log( count ); // Object {b: 2, c: 2, d: 2, e: 2}

 

Comments

No strasznie go skróciłeś. xD W logice nie różni się tak samo za dużo, bo tak samo iterujesz do liczenia i usuwania później.

uniqueCount = ["a","b","c","d","d","e","a","b","c","f","g","h","h","h","e","a"];
var  count = {}; 
uniqueCount.forEach(function(i) { count[i] = (count[i]||0)+1;  });

Object.keys(count).forEach(e => {
	if (count[e] === 1) delete count[e];
});

// count = {"a":3,"b":2,"c":2,"d":2,"e":2,"h":3}

 

---

Na pewno można skrócić ten kod

Miałem na myśli moją propozycję kodu :) 

[edit]

Twoja propozycja zostawia też liczbę wystąpień równą 3 - a czy aby słowo "duplikat" nie odnosi się do dokładnie dwóch wystąpień czegoś?

http://sjp.pl/duplikat

Nie, że czepiam się szczegółów, ale skrypt powinien precyzyjnie wykonywać swoje zadanie :)

Chyba wystarczy "u Ciebie" dodać if (count[e] === 1 || count[e] === 3) albo po prostu dać warunek if (count[e] !== 2) , czyli usunąć wszystko co nie jest duplikatem (tak jak ja podałem w swojej propozycji).