Оптимизация js кода, нормально ли?

Question

[igor042018]

https://codepen.io/igor72537253/pen/ZEMGNBy

Подскажите пожалуйста можно ли как-то улучшить, упростить строки в функции openCloseMenu()

Нормальный ли прием с arrayElemOnClick ?

Comments

[VoidVolker]

igor042018 Правила, пункт 3.8

[igor042018]

VoidVolker, извиняюсь, поправил

Answer 1

[wakenbyWork]

Переписал на ES6, думаю если вам интересна оптимизация, то и современный стиль вызовет интерес :)

window.addEventListener('load', () => {
  const toggle = document.querySelector('#js-toggle')
  const body = document.querySelector('#js-body')
  const nav = document.querySelector('#js-nav')
  const menuBg = document.querySelector('#js-menu-bg')
  const toggleNav = document.querySelector('#js-toggle-nav')
  
  const openCloseMenu = () => {
    toggle.classList.toggle('toggle--active')
    body.classList.toggle('body--overflow')
    nav.classList.toggle('header__nav--active')
    menuBg.classList.toggle('menu-bg--active')
  }

  for (const clickEl of [toggle, toggleNav, menuBg]) {
    clickEl.addEventListener('click', openCloseMenu)
  }
})

Нормальный ли прием с arrayElemOnClick ?

Вполне, у вас элементы по которым можно кликнуть, еще имеют смену класса при клике.
Без данного условия, можно создать специальный класс или data-attr, чтобы одним querySelectorAll найти все интерактивные элементы, и через цикл так-же добавить click.