計數元件的occurances在一個陣列中的JavaScript

Question

我有一個這樣的陣列，我想count每個元件的頻率

var arr = [1, 5, 7, -1]; 
function countArray (arr){ 
    var map = []; 
    for (var i =0; i < arr.length ; i++){ 
     map[arr[i]]++; 
    } 
    console.log(map); 
} 

的console.log打印[1: NaN, 5: NaN, 7: NaN, -1: NaN]

我預期它打印[1: 1, 5: 1, 7: 1, -1: 1]

我不明白爲什麼我得到NaN

Answer 1

好，如果這是第一次發生，那麼你會嘗試增加一個未定義的。因此，首先檢查它是否未定義，然後分配0或增加它。你應該使用一個對象而不是數組。該對象有一個key和value而Array是一組有序的元素，通過它們的數字索引進行訪問。

我在編輯這個來解釋你的可以使用Array對象。如果索引的值大於數組中的最大索引，則將調整其大小。不過，我認爲一個對象更好用，因爲將來如果使用其他類型的鍵（不是數字），那麼代碼將不需要任何更改。

var arr = [1, 5, 7, -1]; 
function countArray (arr) { 
    var map = {}; 
    for (var i =0; i < arr.length ; i++){ 
     map[ arr[i] ] = typeof map[ arr[i] ] === 'undefined' ? 1 : ++map[ arr[i] ]; 
    } 
    console.log(map); 
} 

Answer 2

您應該檢查數組是否已經包含密鑰，您可以使用「in」來做到這一點。

var arr = [1, 5, 7, -1, -1]; 
function countArray (arr){ 
    var map = []; 
    for (var i =0; i < arr.length ; i++){ 
     if (arr[i] in map) map[arr[i]]++; 
     else map[arr[i]] = 1; 
    } 
    console.log(map); 
} 
countArray (arr) 

輸出：

[1：1,5：1,7：1，-1：2]

Answer 3

你的功能是好的，但可以用做陣列它。降低一個班輪：

var arr = [1, 5, 7, -1, 1,1]; 
 

 
var r = arr.reduce((ac, x) => { if (!ac[x]) ac[x] = 0; return ac[x]++, ac},{}); 
 

 
document.write(JSON.stringify(r))

選擇任何你找到更易讀..

Answer 4

更explanatory version (JSBin)與降低：

var arr = [1, 5, 7, -1];  

var count = arr.reduce(function(obj, next) { 
    obj[next] = ++obj[next] || 1; 
    return obj; 
}, {});