Как сократить выражение в условном операторе?

Question

Shimpanze @Shimpanze

JavaScript

Как сократить выражение в условном операторе?

if (
  el.nextSibling.nodeName === "DIV" ||
  el.nextSibling.nodeName === "P" ||
  el.nextSibling.nodeName === "SPAN" ||
  el.nextSibling.nodeName === "I" ||
  el.nextSibling.nodeName === "B" ||
  el.nextSibling.nodeName === "PRE" ||
  el.nextSibling.nodeName === "CODE" ||
  el.nextSibling.nodeName === "IMG"
) {...}

Вопрос задан более трёх лет назад
125 просмотров

Комментировать

Подписаться 1 Простой Комментировать

Решения вопроса 2

2 комментария

Shimpanze @Shimpanze Автор вопроса

У меня не получается сократить вот такое условие, предложенным вами способом:

if (
  ( element.previousSibling !== null && element.previousSibling.nodeName.toLowerCase() !== "h1" ) &&
  ( element.previousSibling !== null && element.previousSibling.nodeName.toLowerCase() !== "h2" ) &&
  ( element.previousSibling !== null && element.previousSibling.nodeName.toLowerCase() !== "h3" ) &&
  ( element.previousSibling !== null && element.previousSibling.nodeName.toLowerCase() !== "h4" ) &&
  ( element.previousSibling !== null && element.previousSibling.nodeName.toLowerCase() !== "h5" ) &&
  ( element.previousSibling !== null && element.previousSibling.nodeName.toLowerCase() !== "h6" ) &&
  ( element.previousSibling !== null && element.previousSibling.nodeName.toLowerCase() !== "p" ) &&
  ( element.previousSibling !== null && element.previousSibling.nodeName.toLowerCase() !== "img" )
) {

Написано более трёх лет назад

0xD34F @0xD34F Куратор тега JavaScript

Shimpanze,

const names = [ 'h1', 'h2', 'h3', ... ];
const el = element.previousSibling;
if (el && !names.includes(el.nodeName.toLowerCase())) {
  ...

Написано более трёх лет назад

7 комментариев

CoyoteSS @CoyoteSS

А представь, у тебя таких проверок должно быть сотни, чего теперь 100 полей вписывать в объект?

Написано более трёх лет назад
dollar @dollar

Roman Kotov, а что смущает? Поиск по хешу (ключу) займет O(1), в то время как поиск по значению O(N).

Написано более трёх лет назад
CoyoteSS @CoyoteSS

dollar, я имел ввиду количество хэшей допустим 100, придётся создавать объект и вписывать поле для каждого?)

Написано более трёх лет назад
dollar @dollar

Roman Kotov, ну да. А как иначе? Для массива вы тоже вписываете значение каждого элемента.

Написано более трёх лет назад
CoyoteSS @CoyoteSS

dollar, а представьте хэшей не 100, а сотни тысяч

Написано более трёх лет назад
dollar @dollar

Roman Kotov, тем более! Ведь поиск по хешу практически моментальный, в отличие от поиска по массиву путем перебора примерно половины значений.

Написано более трёх лет назад
dollar @dollar

Roman Kotov, Ну а как вы будете заполнять сам массив, это уж извините, дело ваше. Разницы между заполнением обычного и "ассоциативного" массива в JS нет. И там, и там нужно перечислять какие-то элементы текстом через запятую.

Написано более трёх лет назад

Пригласить эксперта

Ваш ответ на вопрос

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

Войти через центр авторизации

Похожие вопросы

JavaScript

+2 ещё

Средний
Как отсортировать в ХТМЛ по дате рождения от младшего к старшему, а так же, нумерацию их 1,2,3,4. Чтобы можно было потом добавить еще людей?
- 1 подписчик
- 6 часов назад
- 41 просмотр
1

ответ
JavaScript

+2 ещё

Простой
Почему шрифт в JS prompt() белого цвета?
- 1 подписчик
- 9 часов назад
- 44 просмотра
2

ответа
JavaScript

Простой
Выдает ошибку, сам файл не нулл(а ошибка выдается там где подключается жс), как исправить?
- 1 подписчик
- 11 часов назад
- 44 просмотра
0

ответов
JavaScript

+2 ещё

Простой
Пустая страница, ошибок нет. JS. В чем проблема?
- 1 подписчик
- 11 часов назад
- 158 просмотров
1

ответ
JavaScript

Простой
Какой фреймворк использовать для форм JS?
- 1 подписчик
- 11 часов назад
- 52 просмотра
0

ответов
JavaScript

Простой
Почему change срабатывает раньше чем click?
- 1 подписчик
- 12 часов назад
- 63 просмотра
1

ответ
JavaScript

Простой
Как получить metadata аудио на IOS?
- 1 подписчик
- 18 часов назад
- 21 просмотр
0

ответов
JavaScript

+2 ещё

Простой
Как отобразить массив из элементов?
- 1 подписчик
- вчера
- 100 просмотров
0

ответов
JavaScript

+1 ещё

Простой
Неизвестные скрипты на сайте (plupload и moxie). Для чего?
- 1 подписчик
- вчера
- 116 просмотров
1

ответ
JavaScript

+2 ещё

Простой
Как сделать так, чтобы звезда падала только при скролле страницы?
- 1 подписчик
- вчера
- 90 просмотров
0

ответов
Показать ещё Загружается…

JavaScript разработчик

summer • Ярославль

от 100 000 до 140 000 ₽

Junior JavaScript Developer

КРАФТТЕК • Санкт-Петербург

от 60 000 до 80 000 ₽

JavaScript разработчик

вАйТи

от 5 000 до 25 000 ₽

Разработать интранет сайт (корпоративный портал)

20 апр. 2024, в 07:24

100000 руб./за проект

Исправить адаптивную верстку на Tilda Zero Block

20 апр. 2024, в 06:39

4000 руб./за проект

Доработка аддона для Xenforo v2.2.13

20 апр. 2024, в 06:06

200 руб./за проект

Answer 1 · 2019-07-20 03:52:23

Сложить допустимые значения в массив, посмотреть, присутствует ли в этом массиве проверяемое значение:

const names = [ 'DIV', 'P', 'SPAN', ... ];
if (names.includes(el.nextSibling.nodeName)) {
  ...

Answer 2 · 2019-07-20 15:44:40

Вообще быстрее будет не перебором массива, пусть и встроенной функцией, а поиск по хешу. Да и в целом, думаю, красивее, хотя дело вкуса:

const hashes = [ DIV:1, P:1, SPAN:1, ... ];
if (hashes[el.nextSibling.nodeName]) {
  ...

Хотя это и экономия на спичках. Но если проверок сотни и тысячи, то хеш-таблица однозначно выгоднее.

Как сократить выражение в условном операторе?

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

Минуточку внимания

Войдите на сайт