简化javascript函数中的嵌套循环

Question

我在JS中有这个函数

function getMap(objectList) {
const objectMap = new Map();
IDS.foreach(id => {
const attribute = objectList.find(object => object.getId() === id);
if (attribute) {
objectMap.set(id, attribute);
} else {
objectMap.set(id, null);
}
}

这是一个嵌套循环，因为find在for循环中。这怎么能简化呢?如果不能简化嵌套循环，其他部分可以简化吗?

Answer 1

假设对象id是唯一的，看起来您真正要做的就是事先在每个对象上调用getId。如果您愿意，可以使用条件运算符代替if/else。

function getMap(objectList) {
const objectsById = new Map(
objectList.map(object => [object.getId(), object])
);
const objectMap = new Map();
for (const id of IDS) {
objectMap.set(id, objectsById.get(id) || null);
}
}

Answer 2

您可以为每个ID创建一个包含null条目的数组，然后是您在objectList中实际具有值的条目，并将该数组传递给Map构造函数:

function getMap(objectList) {
return new Map([
...IDs.map(id => [id, null]),
...objectList.map(object => [object.getId(), object])
]);
}

Answer 3

使用本地代码进行简单的回调

const result = (IDS || []).map(function(id, idx, arr) {
const pos = (objectList || []).findIndex(object => object.getId() === id);
const output = [];
output[id] = (pos >= 0 ? objectList[pos] : null);
return output;
});

希望这有助于…; D