有效地检查对象数组循环中的许多条件

Question

我有一个对象数组，一个属性是它们的状态，我需要计算以下状态：如果所有状态都失败，则返回“失败”，如果所有状态都被取消，则返回“已取消”，依此类推，但如果数组有一个失败，或者一个取消返回“部分完成”，我做到了，但我循环了很多，我觉得存在一种更好更有效的方法来做同样的事情，这是我的代码：function getComputedStatus(list) {      if (list.every(status => status.statusName == 'Failed'))          return 'Failed';      if (list.every(status => status.statusName == 'Canceled'))          return 'Canceled';      if (list.every(status => status.statusName == 'DidNotRun'))          return 'DidNotRun';      if (list.some(status => status.statusName == 'InProgress' ||          status.statusName == 'Pending'))          return 'InProgress';      if (list.some(status => status.statusName === 'Failed'   ||                              status.statusName === 'Canceled' ||                              status.statusName === 'DidNotRun'))         return 'PartiallyCompleted';      return 'Completed';}

Answer 1

所以，看起来你的逻辑实际上是这样的：

1. 您有一组状态对象。

2. 如果任何状态为“InProgress”或“Pending”，则返回“InProgress”

3. 如果所有状态都相同且为“失败”、“取消”或“DidNotRun”，则返回该值。

4. 如果任何状态为“失败”、“取消”或“DidNotRun”，则返回“部分完成”

5. 否则，返回“完成”

因此，如果您能弄清楚如何通过一次遍历数组而不是 5 次部分遍历数组来完成所有这些检查，将会获得巨大的执行效率。

这是一种方法：

const doneStatuses = new Set(["Failed", "Canceled", "DidNotRun", "Completed", "InProgress"]);

function getComputedStatus(list) {

    const allStatuses = new Set();

    // collect all status values in a Set object

    for (let status of list) {

        allStatuses.add(status.statusName);

    }

    // if all the statusName values were the same and they were  

    // "Failed", "Canceled", "DidNotRun", "InProgess" or "Completed"

    // then return that specific status

    if (allStatuses.size === 1) {

        let theStatus = Array.from(allStatuses)[0];

        if (doneStatuses.has(theStatus)) {

            return theStatus;

        }

    }

    // if any status was "InProgress" or "Pending", return "InProgress"

    if (allStatuses.has("InProgress") || allStatuses.has("Pending")) {

        return "InProgress";

    }

    // If any status was "Failed", "Canceled" or "DidNotRun" (but not all)

    // then return "PartiallyCompleted"

    if (allStatuses.has("Failed") || allStatuses.has("Canceled") || allStatuses.has("DidNotRun")) {

        return "PartiallyCompleted";

    }

    // otherwise, return "Completed"

    return "Completed";

}