Как улучшить код?

Question

[sloupone]

Собственно, что делает код: подключает Masonry, если элемент с нужным селектором вообще есть на странице и ширина окна больше 768 px .

Код под спойлером

(function membersListInit() {
  const grid = '.members-list__list';
  const options = {
    columnWidth: '.members-list__grid-sizer',
    itemSelector: '.members-list__item-container',
    horizontalOrder: true,
  };


  const elem = document.querySelector(grid) && !document.querySelector('.members-list_teaser');
  let isInit = false;
  let msnry;


  window.addEventListener('load', () => {
    if (window.innerWidth > 768 && elem) {
      msnry = new Masonry(grid, options);
      isInit = true;
    }
  });


  window.addEventListener('resize', () => {
    if (window.innerWidth >= 768 && elem) {
      if (!isInit) {
        msnry = new Masonry(grid, options);
        isInit = true;
      }
    } else if (isInit) {
      msnry.destroy();
      isInit = false;
    }
  });
}());

Есть ощущение, что я делаю что-то не так. Закидайте камнями.

Answer 1

[Stalker_RED]

window.addEventListener('load',   masonryOn768);
window.addEventListener('resize', masonryOn768);

function masonryOn768() {
    if (window.innerWidth >= 768 && elem) {
      if (!isInit) {
        msnry = new Masonry(grid, options);
        isInit = true;
      }
    } else if (isInit) {
      msnry.destroy();
      isInit = false;
    }
  }

Так лучше?
И непонятно что такое elem.

Comments

[Владимир]

Вот что-то такое я себе представлял, но не смог самостоятельно к этому прийти. Спасибо.
elem — проверка, есть ли нужный элемент на странице (иначе Masonry ругается в консоле, если передать ему элемент, которого нет на текущей странице).
А насколько нормально выглядит isInit?

[Stalker_RED]

Владимир, ее можно выбросить заменив на if (msnry)

[Владимир]

Stalker_RED, не совсем понял, как это должно работать. Я передаю в Масонори grid. До этого я выясняю, есть ли этот grid вообще, и результат записываю в elem. Если его нет, msnry вообще можно не создавать.
(Если что, вопрос мой, случайно задал с другого аккаунта.)

[Stalker_RED]

Владимир, у вас там есть

let msnry;

и в этой переменной со старта undefined, а после инициализации
msnry = new Masonry()ссылка на masonry.

Флаг isInit не нужен, можно просто проверить if (msnry).

[Владимир]

Stalker_RED, а, думал, что речь про elem.

[Stalker_RED]

Владимир, а я отвечал на вопрос

А насколько нормально выглядит isInit?

[Владимир]

Stalker_RED, да, разобрался. Спасибо за ответы