私はこの違いについて疑問に思ってフィルタマップ

Question

対JavaScriptの反復：両方のコードは、コードが入っている場合は、単純にtrueを返し、同じことを行う必要があります

function doesCodeExist(arrvalidcodes, code) { 

    for (var i = 0; i < arrvalidcodes.lenght; i++) { 
    if (arrvalidcodes[i] === code) { 
     return true; 
    } 
    } 
    return false; 
} 

：

function doesCodeExist(arrvalidcodes, code) { 

    var hit = arrvalidcodes.filter(function(item){ 
    return (code === item) 
    }).map(function(item){ 
    return true; 
    }); 
    if (hit[0] === true) { 
    return true; 
    } else { 
    return false; 
    } 
} 

と、この指定された配列またはそれ以外の場合はfalseを返します。

これは最も読みやすく、効率的な方法です。

Answer 1

あなたはArray#indexOf

indexOf()方法は、それが存在しない場合、所与の素子がアレイ状に見つかった、または-1可能な最初のインデックスを返す使用することができます。

function doesCodeExist(arrvalidcodes, code) { 
    return arrvalidcodes.indexOf(code) !== -1; 
} 

またはES6と、それは様々な無用の配列を割り当て、読み取り可能ではありませんので、あなたの最初のコードが悪化しているもちろんArray#includes

Answer 2

を使用しています。

あなたはセマンティックES5の配列方法をしたい場合は、someを使用します。もちろん

function doesCodeExist(arrvalidcodes, code) { 
    return arrvalidcodes.some(function(item){ 
    return code === item; 
    }); 
} 

を、ちょうど厳密な比較のために、あなたはindexOfを使用する必要があります、またはあなたがSameValueZeroのcomparisson使用includesをしたいです。