Избежать дублирование кода. Объеденить условия.
Я прям новичок, в этой сфере и прошу помощи у ассов. Необходимо убрать дублирование кода и объединить условия. Как лучше это сделать и какая польза от этого?:help:
setEventListeners() { this._popupItem.addEventListener('mousedown', (evt) => { if (evt.target.classList.contains('popup__opened')) { this.close(); } else if (evt.target.classList.contains('popup__button-close')) { this.close(); } }) } |
setEventListeners() {
this._popupItem.addEventListener('mousedown', (evt) => {
const { classList } = evt.target;
if (classList.contains('popup__opened') || classList.contains('popup__button-close')) {
this.close();
}
})
}
Польза - не писать одно и то же. Ещё так можно, но обычно это на случай если проверок одинаковых больше двух:
setEventListeners() {
this._popupItem.addEventListener('mousedown', (evt) => {
if (['popup__opened', 'popup__button-close'].some(name => evt.target.classList.contains(name)) {
this.close();
}
})
}
|
:thanks:
|
| Часовой пояс GMT +3, время: 15:17. |