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

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

所以，看起来你的逻辑实际上是这样的：您有一组状态对象。如果任何状态为“InProgress”或“Pending”，则返回“InProgress”如果所有状态都相同且为“失败”、“取消”或“DidNotRun”，则返回该值。如果任何状态为“失败”、“取消”或“DidNotRun”，则返回“部分完成”否则，返回“完成”因此，如果您能弄清楚如何通过一次遍历数组而不是 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";}