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

Question

[Antonio09]

https://jsfiddle.net/mwoq0u48/

Comments

[Ярослав Иванов]

https://jsfiddle.net/bLad4w5q/

Answer 1

[Игорь]

Например не дублировать код, использовать краткую форму onReady и не ставить столько пустых строк. К тому же не зачем выносить в функцию то, что используется лишь раз

$(function() {
  $(window).resize(onRes);
  
  onRes();
  
  function onRes() {
    if ($(window).width() <= 1050) {
      $('.popup__list').on('click', 'a[href=""]', function(e) {
        e.preventDefault();
        $(this).next('.popup__submenu').slideToggle();
      });
    }
  }
});

Comments

[Antonio09]

спасибо
но как то подглючивает (когда открываю подменю)
после ресайза именно лагает
тоесть на мелких экран открыл меню
потом возвращаю к большу - и лагает
а оно вообще не должно показываться на декстопе(подменю)

https://jsfiddle.net/43ksqxa0/

не подскажете как подправить?

[Игорь]

Я не понимаю цель этого скрипта вообще.

Answer 2

[Андрей Пике]

$(document).ready(function() {
...
});

Устарело. Нормально:

$(() => {
...
});

$(window).resize(function() {
...
});

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

e.preventDefault();

можно заменить на

return false;

в конце функции. Но это просто как рекомендация.

Comments

[Андрей Пике]

$('.popup__list').on('click', ...);
=>
$('.popup__list').click(...);

Лаконичнее.

[Игорь]

Андрей Пике, может и лаконичнее, но динамически добавленные ссылки обрабатываться не будут

[Андрей Пике]

Ну, я тут не вижу никакой динамики.
Решение по ситуации.

[Андрей Пике]

По задаче, точнее.

[Игорь]

Андрей Пике, Естественно не видите, он же только вырезку кода дал. Просто потом будет вопрос тип "ПАМАГИТИ!!! У меня ничего не работает!"

[Андрей Пике]

IMHO, невозможно прогнозировать завтра и делать слишком сложно.
Не окупится для бизнеса.
С небольшим намёком на будущее - нормально что-то предусмотреть.
Но не очень.
Бизнес не поймёт.

[Андрей Пике]

(я с американцами последние годы работал, они сложно понимают понятия "на будущее" и "предусмотреть")