2つのアレイを同時に比較し、偶数だけを記録する

Question

2つのアレイを同時に比較し、偶数の値だけを記録しようとしています。これは私が思いついたコードですが、正しくログしますが、これは正しい方法ではないと確信しています。このコードを改善するにはどうすればよいですか？

「ARR1とARR2を取るだけでもものをログ」

const arr1 = [5,8,2,1,5,7,3,4,5,8,1,2,4,8,3,1,4,5]; 
const arr2 = [15,26,74,12,3,6,9,1,2,5]; 

for (var i=0; i < arr1.length; i++) { 
    if ((arr1[i] % 2) === 0) { 
     console.log(arr1[i]); 
    } 
} 

for (var i=0; i < arr2.length; i++) { 
    if ((arr2[i] % 2) ===0) { 
     console.log (arr2[i]); 
    } 
} 

Answer 1

連結方式のあなたの配列をして、単一のアレイ上でコードを実行します。

const arr1 = [5, 8, 2, 1, 5, 7, 3, 4, 5, 8, 1, 2, 4, 8, 3, 1, 4, 5]; 
 
const arr2 = [15, 26, 74, 12, 3, 6, 9, 1, 2, 5]; 
 

 
const mergedArray = arr1.concat(arr2); 
 

 
for (var i = 0; i < mergedArray.length; i++) { 
 
    if ((mergedArray[i] % 2) === 0) { 
 
    console.log(mergedArray[i]); 
 
    } 
 
}

Answer 2

また、単に一つのループで両方の配列を横断し、あなたがより多くのスペースを追加することができない場合の両方で偶数かどうかを確認できます。 2つのforループを実行する方が効率的です。最初のループは0 to min(arr1.length, arr2.length)になり、2番目のループはmin(arr1.length, arr2.length) to max(arr2.length,arr1.length)になります（どちらが先に大きいかを確認する必要があります）。また、正しく実行されるコードの効率を向上させることに関連する質問についてはyou should go hereと考えています。

const arr1 = [5, 8, 2, 1, 5, 7, 3, 4, 5, 8, 1, 2, 4, 8, 3, 1, 4, 5]; 
 
const arr2 = [15, 26, 74, 12, 3, 6, 9, 1, 2, 5]; 
 
const mx = Math.max(arr1.length, arr2.length); 
 
for (var i = 0; i < mx; i++) { 
 
    if (i < arr1.length && (arr1[i] % 2 === 0)) { 
 
    console.log(arr1[i]); 
 
    } 
 
    if (i < arr2.length && (arr2[i] % 2 === 0)) { 
 
    console.log(arr2[i]); 
 
    } 
 
}