Можно ли упростить данный код?

Question

[nastya_zholudeva]

Прилагаю схематический код

if (isAllTypes || array.contains('akt')) {
       selected.push({type: 'akt'})
  }
          
if (isDTypes) {
    selected.push({type: 'd'})
}
          
 if (isKType) {
   selected.push({type: 'k'})
}

Comments

[Алексей Ярков]

И в чем сложность?

[nastya_zholudeva]

Алексей Ярков, похожий код, может его как-то отрефакторить можно?

[Алексей Ярков]

nastya_zholudeva, зачем? Что не устраивает?

[Алексей Ярков]

nastya_zholudeva, ну можно так
isKType && selected.push({type: 'k'});
Но опять же - зачем?

[nastya_zholudeva]

Алексей Ярков, хорошо, спасибо)

Answer 1

[Алексей Ярков]

Можно, например, так
isKType && selected.push({type: 'k'});
Но опять же - зачем?

Answer 2

[Дмитрий Лузанов]

+1 вариант

switch (true) {
  case isAllTypes || array.contains("akt"): selected.push({ type: "akt" });
  case isDTypes: selected.push({ type: "d" });
  case isKType: selected.push({ type: "k" });
}

Answer 3

[Alexey]

Да на мой взгляд тут и так все просто, опять же можно без {} сделать.

Можно через условный (тернарный) оператор - почитать
что то вроде такого:
isKType ? selected.push({type: 'k'}) : false

Я бы просто накатал:

if (isAllTypes || array.contains('akt')) selected.push({type: 'akt'})
if (isDTypes) selected.push({type: 'd'})
if (isKType) selected.push({type: 'k'})