Есть ли более чистый способ сортировки списка?

Question

Я работаю с ES6 и React. Я просматриваю объект ответа из службы Java Rest, и мне стало ясно, что, возможно, существует более чистый способ разбора объекта на два объекта. Этот wokrs, он просто выглядит неуклюжим.

  let draftList = []; 
      let readyForApprovalList = []; 
      for (let i = 0; i < action.allStatusesSurveysList.length; i++){ 
       if (action.allStatusesSurveysList[i].status === statusTypes.DRAFT){ 
        draftList.push(action.allStatusesSurveysList[i]); 
       } else if (action.allStatusesSurveysList[i].status === statusTypes.READY_FOR_APPROVAL){ 
        readyForApprovalList.push(action.allStatusesSurveysList[i]); 
       } 
      } 

Answer 1

Сочетание Array.prototype.filter() и arrow filter functions будет довольно чистое решение.

Обратите внимание, что хотя это решение является приятным и читаемым, оно не является наиболее эффективным, так как требуется как минимум две итерации массива. Тем не менее, он вполне может быть достаточно эффективным для подавляющего большинства случаев использования и по этой причине является хорошим компромиссом.

const draftList = action.allStatusesSurveysList 
    .filter(i => i.status === statusTypes.DRAFT) 
const readyForApprovalList = action.allStatusesSurveysList 
    .filter(i => i.status === statusTypes.READY_FOR_APPROVAL) 

---

Можно даже извлечь функцию состояния фильтра дополнительно:

const byStatus = status => item => item.status === status 

const draftList = action.allStatusesSurveysList 
    .filter(byStatus(statusTypes.DRAFT)) 
const readyForApprovalList = action.allStatusesSurveysList 
    .filter(byStatus(statusTypes.READY_FOR_APPROVAL)) 

Answer 2

forEach вероятно, будет работать лучше здесь - в то время как вы можете использовать filter, потребуется один проход для каждого типа категории.

action.allStatusesSurveysList.forEach(function(item) { 
    switch(item.status) { 
     case statusTypes.DRAFT: 
      draftList.push(item); 
      break; 
     case statusTypes.READY_FOR_APPROVAL: 
      readyForApprovalList.push(item); 
      break; 
     default: 
      // "unknown status" list?? 
    } 
}); 

Это перебрать массив и имеет аккуратный EXTRA использования item вместо многократного ввода action.allStatusesSurveysList[i]. Кроме того, использование оператора switch лучше, чем if..else if..