Задать вопрос
nastya_zholudeva
@nastya_zholudeva
javascript

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

Прилагаю схематический код
if (isAllTypes || array.contains('akt')) {
       selected.push({type: 'akt'})
  }
          
if (isDTypes) {
    selected.push({type: 'd'})
}
          
 if (isKType) {
   selected.push({type: 'k'})
}
  • Вопрос задан
  • 126 просмотров
5 комментариев
Подписаться 1 Простой 5 комментариев
Решения вопроса 3
yarkov
@yarkov Куратор тега JavaScript
Помог ответ? Отметь решением.
Можно, например, так
isKType && selected.push({type: 'k'});
Но опять же - зачем?
Ответ написан
Комментировать
Комментировать
dmitry_luzanov
@dmitry_luzanov
+1 вариант
switch (true) {
  case isAllTypes || array.contains("akt"): selected.push({ type: "akt" });
  case isDTypes: selected.push({ type: "d" });
  case isKType: selected.push({ type: "k" });
}
Ответ написан
Комментировать
Комментировать
alexeynobody
@alexeynobody
Да на мой взгляд тут и так все просто, опять же можно без {} сделать.

Можно через условный (тернарный) оператор - почитать
что то вроде такого:
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'})
Ответ написан
Комментировать
Комментировать
Пригласить эксперта
Ваш ответ на вопрос

Войдите, чтобы написать ответ

Войти через центр авторизации
Похожие вопросы
Вакансии с Хабр Карьеры
JavaScript разработчик
SummerWeb • Ярославль
от 100 000 до 140 000 ₽
JavaScript разработчик
вАйТи
от 5 000 до 25 000 ₽
JavaScript Fullstack
OnClass
от 200 000 до 600 000 ₽
Ещё вакансии
Заказы с Хабр Фриланса
Интегрировать МИС Медцентра с ЕЦП в части получения Направлений
24 апр. 2024, в 08:24
100000 руб./за проект
Модель управления электродвигателем в Matlab, Simulink
24 апр. 2024, в 06:52
5000 руб./за проект
Анимация логотипа
24 апр. 2024, в 00:08
20000 руб./за проект
Ещё заказы