オブジェクトの2つの配列を比較し、2つ目のオブジェクトの値を更新します。非常に遅い

Question

以下は私の作業コードの例ですが、問題は私が100kを超えるレコードがあるときです。このスクリプトは実行に非常に時間がかかっています。

注：私はオブジェクトの2つの配列を持っており、最初のものに基づいて2番目の配列のオブジェクト名を設定する必要があります。 IDは2つの間で一致し、順序はランダムになります。

const _ = require('lodash'); 

let res1 = [ 
    {id:1, name:'Sandra'}, 
    {id:2, name:'Bill'}, 
    {id:3, name:'Peter'}, 
    {id:4, name:'Jill'} 
]; 

let res2 = [ 
    {id:2, name:'John'}, 
    {id:4, name:'Bobby'} 
]; 

_.forEach(res1, function(data1) { 
    _.forEach(res2, function(data2) { 
    if (data1.id === data2.id) { 
     data2.name = data1.name; 
    } 
    }); 
}); 

// res2 = [{id:2, name:'bill'},{id:4, name:'Jill'}]; 

Answer 1

res1のidでマップを作成し、第二の配列を反復処理よりも、あなたがres1mapを見つける各IDの名前を置き換えます。

const res1 = [ 
 
    {id:1, name:'Sandra'}, 
 
    {id:2, name:'Bill'}, 
 
    {id:3, name:'Peter'}, 
 
    {id:4, name:'Jill'} 
 
]; 
 

 
const res2 = [ 
 
    {id:2, name:'John'}, 
 
    {id:4, name:'Bobby'} 
 
]; 
 

 
const res1Map = _.keyBy(res1, 'id'); 
 

 
res2.forEach((o) => res1Map[o.id] && (o.name = res1Map[o.id].name)); 
 

 
console.log(res2);

<script src="https://cdnjs.cloudflare.com/ajax/libs/lodash.js/4.17.4/lodash.min.js"></script>

Answer 2

forEachとfindを使用して簡単な解決策：

let res1 = [ 
 
    {id:1, name:'Sandra'}, 
 
    {id:2, name:'Bill'}, 
 
    {id:3, name:'Peter'}, 
 
    {id:4, name:'Jill'} 
 
]; 
 

 
let res2 = [ 
 
    {id:2, name:'John'}, 
 
    {id:4, name:'Bobby'} 
 
]; 
 

 
res2.forEach((item) => { 
 
    item.name = res1.find((person) => person.id === item.id).name 
 
}); 
 

 
console.log(res2);