引数

Question

に基づいて配列から番号を削除するにはどうすればこのような二つの配列を持つ：

私は最初の配列で利用可能な番号に基づいて二番目の配列から数字を削除したい

var arr = ["1", "3" , "4"]; 
var arr2 = ["1", "2", "3", "4", "5", "6", "7", "8", "9"]; 

。

私はこれを試してみましたが、私は、2,4,6,8のように間違った値を取得しています：ここで

theButton.onclick = function removePassedInNumbers(arr){ 
    for(var i = 0; i < arr2.length; i++){ 
    if(arr2.indexOf(arr[i])){ 
     arr2.splice(i, 1); 
    } 
    } 
    document.getElementById('myNumber').innerHTML = arr2; 
} 

<input type="button" id="theButton" onclick="removePassedInNumbers(arr)" value="Click here"/> 
<p id="myNumber">hey</p> 

はフィドルです：私は間違って行くのですhttps://jsfiddle.net/4hLcanc6/

？

Answer 1

これは正しくありません。一部のユーザーが既に回答している場合でも、配列をフィルタリングするためのより良い方法があるとしても、なぜこのループを間違っているのか説明しようとしています。

はあなたがARR2に変数 iあなたの現在の位置に保存している最初の行ここ

for(var i = 0; i < arr2.length; i++) 

から始めましょう。 OK。

if(arr2.indexOf(arr[i])) 

これはいいえです。 arr[i]が存在しない可能性があります。 Infact、arr[3]は未定義で、ループに入る4回目にそれを探しています。 これを修正しましょう：

if(arr.indexOf(arr2[i])) 

今は大丈夫です。 arrの内部を調べて、arr2[i]の値を見つけようとしています。これはループで初めてジャンプしたときに「1」、2回目に「2」、最後に「9」になります。

これで動作します。要約すると：

for(var i = 0; i < arr2.length; i++){  
    if(arr.indexOf(arr2[i])){ 
    arr2.splice(i, 1); 
    } 
} 

これはあなたのコードで何が起こっているのかを理解するのに役立ちます。

Answer 2

あなたはこの

var arr = ["1", "3" , "4"]; 
 
var arr2 = ["1", "2", "3", "4", "5", "6", "7", "8", "9"]; 
 

 
arr.forEach(function(n){ 
 
    var index = arr2.indexOf(n); 
 
    if(index !== -1) arr2.splice(index, 1); 
 
}) 
 

 
console.log(arr2);

注ようなもので、あなたのfunctionを更新することができます。あなたは、元のarr2を変更せずに新しい配列をしたい場合は、あなたがしなければならない、spliceはあなたの配列arr2を変更しますfilterを使用してください。

Answer 3

あなただけのフィルタ（）メソッドを使用することができます

var arr = ["1", "3" , "4"]; 
var arr2 = ["1", "2", "3", "4", "5", "6", "7", "8", "9"]; 
var res = arr2.filter(item => arr.indexOf(item)===-1); 
console.log(res); // [ '2', '5', '6', '7', '8', '9' ] 

またはES5とを：forループ

var res = arr2.filter(function(x) {return arr.indexOf(x)===-1});