JavaScriptを減らす方法Accumulatorオブジェクトを作成する

Question

var numbers = [5, 3, 8, 6, 9, 1, 0, 2, 2]; 
var oddEvenCounts = numbers.reduce(function(counts, number) { 
    if (isOdd(number)) { 
    counts[odd]++; 
    } else { 
    counts[even]++; 
    } 
}, {}); 

私はこのコードのバグを探しています（まだreduceメソッドを学習しています）） - どこが間違っていますか？

Answer 1

コメントと作業コード：

var numbers = [5, 3, 8, 6, 9, 1, 0, 2, 2]; 
 

 
function isOdd(n) { 
 
    return !!(n % 2); 
 
} 
 

 
var oddEvenCounts = numbers.reduce(function(counts, number) { 
 
    if (isOdd(number)) { 
 
    counts.odd++; // use dot notation or ['odd'] 
 
    } else { 
 
    counts.even++; // use dot notation or ['even'] 
 
    } 
 
    
 
    return counts; // return the accumulator 
 
}, { odd: 0, even: 0 }); // set the initial values of odd and even 
 

 
console.log(oddEvenCounts);

あなたは括弧表記と三項演算子を使って、コードを少し短くすることができます：

var numbers = [5, 3, 8, 6, 9, 1, 0, 2, 2]; 
 

 
function isOdd(n) { 
 
    return !!(n % 2); 
 
} 
 

 
var oddEvenCounts = numbers.reduce(function(counts, number) { 
 
    counts[isOdd(number) ? 'odd' : 'even']++; 
 
    
 
    return counts; 
 
}, { odd: 0, even: 0 }); 
 

 
console.log(oddEvenCounts);

Answer 2

はあなたのアキュムレータを返し

：

var numbers = [5, 3, 8, 6, 9, 1, 0, 2, 2]; 
var oddEvenCounts = numbers.reduce(function(counts, number) { 
    if (isOdd(number)) { 
    counts[odd]++; 
    } else { 
    counts[even]++; 
    } 
    return counts; 
}, {});