Получить самый высокий статус из списка

Question

Я работаю над куском javascript, который устанавливает значение цвета, основанное на статусе healthcheck. Значения цвета строки привязываются к CSS, с которыми я работаю, а статусы строк привязываются к проверкам работоспособности, указанным в этом скрипте.

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

Текущий код работает, чтобы установить индивидуальные значения цвета, но не в целом. Есть ли «элегантный» способ установить общий, не используя много вложенных операторов if?

function getStatus(checks) { 
    var overall = ''; 
    for (j in checks) { 
    checks[j]['color'] = ''; 
     switch (checks[j].status) { 
      case 'OK': 
       checks[j]['color'] = 'success'; 
       break; 
      case 'INFO': 
       checks[j]['color'] = 'info'; 
       break; 
      case 'WARN': 
       checks[j]['color'] = 'warning'; 
       break; 
      case 'ERROR': 
       checks[j]['color'] = 'danger'; 
       break; 
      case 'FATAL': 
       checks[j]['color'] = 'danger'; 
       break; 
     } 
    } 
    return overall; 
} 

я мог бы сделать что-то вроде этого:

case 'WARN': 
    if (overall === "OK" || overall === "INFO") { 
     overall = 'warning'; 
    } 

... но это становится грязным, особенно при проверке «ERROR», которая должна быть больше, чем 3 значения, но меньше, чем один. Есть ли более эффективный способ?

Answer 1

Ну там нет другой выбор? Но возможны некоторые улучшения. Добавить еще один для цикла + переключатель. Если есть ошибка где-то, что вам не нужно, чтобы продолжить цикл, и вы можете вернуть «опасность»

overall = 'success' 

for (j in checks) { 
    switch (checks[j].status) { 

     case 'OK': 
      break; 
     case 'INFO': 
      if (overall === 'success') 
       overall = 'info' 
      break; 
     case 'WARN': 
      if (overall === 'success' || overall === 'warning') 
       overall = 'warning' 
      break; 
     case 'ERROR': 
     case 'FATAL': 
      return 'danger' 
    } 
} 

Answer 2

Используйте числа вместо строк, а затем (если я понял, что вы пытаетесь сделать) это просто случай возврата самого высокого статуса, который вы получаете. Вы можете использовать ассоциативный массив для хранения строки представления, кода ошибки и цвета для каждого состояния в отдельных объектах и ​​переместить окружающую (с тем преимуществом, что вы можете бен от этого переключателя заявления):

var statuses = { 
    ok:  { code: 1, status: 'OK', color: 'success'}, 
    info: { code: 2, status: 'INFO', color: 'info'}, 
    warn: { code: 3, status: 'WARN', color: 'warning'}, 
    error: { code: 4, status: 'ERROR', color: 'danger'}, 
    fatal: { code: 5, status: 'FATAL', color: 'danger'}, 
} 

// example checks 
var checks = [statuses['ok'], statuses['warn'], statuses['fatal']]; 

function getStatus(checks) { 

    var overall = statuses['ok']; 

    for (check in checks) { 
    if (check.code > overall.code) { 
     overall = check; 
    } 
    } 
    return overall; 
} 

Answer 3

После ответа net.uk.sweet, вы можете определить enum для хранения значений статуса

medical.status = { 
    INFO : 0, 
    OK : 1, 
    WARNING : 2, 
    DANGER : 3, 
    FATAL : 4 
} 

А в остальной части кода, вместо того, чтобы давать строковые значения checks[j], вы должны назначить одну из этих констант:

checks[j].status = medical.status.DANGER 

в n вы можете использовать оператор < так же, как в ответе net.uk.sweet