回答編集履歴

objがundefinedで動かなかったので修正

2019/01/14 14:14

投稿

スコア21158

test CHANGED Viewed

@@ -196,9 +196,9 @@
     for (let it of array) {
-      if (it.rev > ((obj[it.id] || {}).rev || 0)) {
+      if (it.rev > ((result[it.id] || {}).rev || 0)) {
-        obj[it.id] = it;
+        result[it.id] = it;
       }

やっぱり破壊的処理の場合はreduce不要だった

2019/01/14 14:14

投稿

スコア21158

test CHANGED Viewed

@@ -190,13 +190,21 @@
   const result = {};
+  // for...ofかforEachかは正直好み
-  arrayList.forEach(array => array.reduce((obj, it) => {
+  for (let array of arrayList) {
+    for (let it of array) {
-    if (it.rev > ((obj[it.id] || {}).rev || 0)) obj[it.id] = it;
+      if (it.rev > ((obj[it.id] || {}).rev || 0)) {
-    return obj;
+        obj[it.id] = it;
-  }, result));
+      }
+    }
+  }
   return Object.values(result);

Keyさんを見ながらひらめいたコード追記

2019/01/14 14:12

投稿

スコア21158

test CHANGED Viewed

@@ -169,3 +169,57 @@
 // 3: {id: "c1", rev: 2, val: 30}
 ```
+---
+keiさんのコードをヒントにいじったのがこちら
+スマートになりつつの速度確保もそれなりに良い感じに仕上がってると思います。
+```JavaScript
+// 別に2個の配列の引数に限定せず、3個でも4個でも受け付けるようになったけどいいよね？
+const mergeArrayBasedIDByRevision = (...arrayList) => {
+  const result = {};
+  arrayList.forEach(array => array.reduce((obj, it) => {
+    if (it.rev > ((obj[it.id] || {}).rev || 0)) obj[it.id] = it;
+    return obj;
+  }, result));
+  return Object.values(result);
+}
+const ar1 = [{id:'a1', rev:1, val:10}, {id:'a2', rev:2, val:15}, {id:'b1', rev:1, val:20}]
+const ar2 = [{id:'a1', rev:2, val:11}, {id:'a2', rev:1, val:12}, {id:'c1', rev:2, val:30}]
+console.log(mergeArrayBasedIDByRevision(ar1, ar2));
+// (4) [{…}, {…}, {…}, {…}]
+// 0: {id: "a1", rev: 2, val: 11}
+// 1: {id: "a2", rev: 2, val: 15}
+// 2: {id: "b1", rev: 1, val: 20}
+// 3: {id: "c1", rev: 2, val: 30}
+```