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

Question

[Павел]

https://codepen.io/mrusklon/pen/awaJWm

если вводить что либо в поля должен появится красный фон у текста , но это работает везде кроме последнего поля , если добавлю еще input то по аналогии будет работать везде кроме последнего , почему так?

p.s. может можно как то код оптимизировать , если у меня будет много инпутов , будет чертвоски много кода

Answer 1

[Stalker_RED]

Конечно можно оптимизировать. Не прописывать каждый инпут отдельно а работать сразу со всеми.

https://codepen.io/anon/pen/Moqmpp

везде кроме последнего , почему так?

Потому, что последний вы проверяете p4 !== '' а остальные по другому (по сути срабатывает !== false).

Comments

[Павел]

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

Например:
3 из 4 инпута заполнены -> показывать фон
1 из 4 инпутов заполнен -> показывать фон
негде не заполненно -> не показывать фон

а сейчас правило только ко всем сразу или все заполнены или все не заполнены.

А можете обьяснить по p4 !== '' , остальные по другому это как?
Спасибо!

[Stalker_RED]

mrusklon: А что не так с уровнем знаний, слабо прочитать про each?

Логика поведения меняется правкой одного условия. Вместо this.value.length < 1 написать this.value.length > 0, например.

Вот эта штука что по вашему делает?

if (p1 || p2 || p3 || p4 !== '' && 0 )

Только p4 сравнивается с пустой строкой, остальные приводятся к булевому значению. А условие && 0 вообще всегда выдает false.
https://learn.javascript.ru/logical-ops
https://learn.javascript.ru/types-conversion

[Павел]

Stalker_RED: спасибо