Почему не срабатывает данное условие — объясните?

Question

[Shimpanze]

Здравствуйте!

Почему не срабатывает это условие (проверка на определённые теги в цикле)? Ведь я их сгруппировал в скобки.

if (els[i].tagName.toLowerCase() === ('area' || 'base' || 'col' || 'command')) { return; }

Спасибо!

Answer 1

[yanis_kondakov]

У вас сначала выполнится'area' || 'base' || 'col' || 'command', получается у вас всегда проверяется

if (els[i].tagName.toLowerCase() === 'area') { return; }

Чтобы сделать, как вы этого хотите, можно сделать так

const values = ['area', 'base', 'col', 'command'];

if (values.includes(els[i].tagName.toLowerCase())) { 
    return; 
}

Comments

[Shimpanze]

К сожалению, это решение не верно, так как данное условие (через contains) не различает границ слова и найдёт, например, теги как 'a' так и 'area'...

[yanis_kondakov]

Shimpanze, не понимаю о чем вы говорите. Вы задали вопрос совершенно о другом.

[yanis_kondakov]

Answer 2

[profesor08]

'area' это не ноль, не undefined, не null, а значит считается за истину, и операция || срабатывает на первой истине. Тем самым ты сравниваешь свое слово всегда с area.

Тебе надо объединить слова в массив и искать в этом массиве нужное слово.

let tags = ['area', 'base', 'col', 'command'];

if (tags .indexOf(els[i].tagName.toLowerCase()) >= 0) {
  return;
}

Тут производится поиск индекса нужного элемента в массиве, если его нет, то вернется -1, если есть, это значение от 0 до [длинны массива - 1]

Comments

[yanis_kondakov]

'area' это не ноль, не undefined, не null, а значит приводится к true, аналогичное касается других слов. Тем самым ты сравниваешь свое слово с true

Это неправда. Не вводите людей в заблуждение

[profesor08]

yanis_kondakov, ну да, js же.

[yanis_kondakov]

profesor08, а в каком языке при 'логическом или' мы будем сравнивать с true?

[profesor08]

yanis_kondakov, например с Си и тд, выражение в скобках вернет 1, 1 == true. Далее зависит от того к какому типу компилятор приведет строку, и захочет ли это делать, обычно не хочет.

[Shimpanze]

Данное условие не срабатывает...

[Влад Хеллсайт]

Shimpanze, а вы сами понимаете что вам нужно?

[profesor08]

Shimpanze, поменяй условие, >= 0

Answer 3

[Анатолий Медведев]

'area' || 'base' || 'col' || 'command' на самом деле равняется 'area'

что бы сделать то что вы задумывали надо было написать так:

if (els[i].tagName.toLowerCase() === 'area' || 
els[i].tagName.toLowerCase() ===  'base' || 
els[i].tagName.toLowerCase() ===  'col' ||
 els[i].tagName.toLowerCase() ===  'command'
) { return; }

но более оптимально сделать так

let items = new Set(['area', 'base', 'col', 'command']);
if(items.has(els[i].tagName.toLowerCase())) {
 return;
}