Избежать дублирование кода. Объеденить условия.

[Evgeniya S]

Я прям новичок, в этой сфере и прошу помощи у ассов. Необходимо убрать дублирование кода и объединить условия. Как лучше это сделать и какая польза от этого?

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();
}
})
}

[Aetae]

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();
    }
  })
}

[Evgeniya S]