なぜ私の配列の最後の要素はスプライシングではなく、未定義になっていますか？

Question

私は小さな運動に取り組んでいる

：

=> ['kept','kept'] 

しかし、その代わりに、私はこのを得続ける：

function keep(array, keeper) { 

//This will return an array of undefined's and 'kept's 
// =>[ 'kept', undefined, 'kept', undefined, undefined ] 

    matchingNumbers = array.map(function matching(element){ 
    if (element === keeper) { 
     return element; 
    } 
    }); 


//Eliminate all undefined's from the matchingNumbers array 

    matchingLength = matchingNumbers.length; 
    for (var i = 1; i < matchingLength; i++) { 
    if(matchingNumbers[i] === undefined) { 
     (matchingNumbers.splice(i, 1)); 
    } 
    } 
    return matchingNumbers; 
} 

keep(['kept', 'thirty', 'kept', 2, 1], 'kept') 

問題は、私はこのをしたい ということです

私は、for-loopでmatchingNumbersのすべての未定義をスプライスしようとしています。だからなぜlaがあるのですか？残りは未定義ですか？

Answer 1

関数がreturnステートメントを実行しない場合、デフォルトでundefinedが返されます。 array.map()は結果の配列に関数の戻り値を格納し、これにはundefinedの値が含まれます。

あなたが代わりに array.mapの array.filterを使用する必要があります

：

matchingNumbers = array.filter(function matching(element){ 
    return element === keeper; 
}); 

Answer 2

0の代わりにiを1に開始していると思います。また、@Waliで述べたように、配列の長さを反復しながら変更しています。それを解決するために、逆の順序で配列を経る：サイドノートとして

for (var i = array.length; i > 0; i--) { 
    ... 
} 

、あなたがマッピングおよびスプライシングを扱うのではなく、Array.filterを使用して欲しいものを達成することができます。