Верно ли написан JS (jQuery)?

Question

[myskypesla]

Есть такой код https://jsfiddle.net/beLsd91g/

Логика сейчас работает правильно и делает следующее:

если выбираем любой элемент в колонке, то следующая колонка становится активной и т.д. но если меняем выбор в предыдущей колонке, в следующей колонке слетает выбор, а колонка стоящая за следующей становится неактивной.

Вопрос: верно ли я всё реализовал средствами jQuery или можно по-другому и менее затратным по ресурсам для браузера?

Comments

[lega]

https://jsfiddle.net/lega911/qsfstpz2/

Answer 1

[GreatRash]

Неверно.

1) Это в принципе не ошибка, но вместо метода .click() лучше использовать .on(). Таким образом вы добьётесь единообразия в коде.
2) $(this) нужно сохранить в переменную, чтобы не создавать каждый раз экземпляр jQuery
3) $(this).closest('.col') тоже сохранить в переменную, ибо поиск в DOM-дереве самая затратная операция в скриптах.
4) Использовать цепочку вызовов по полной, а не частично как сейчас.
5) Какой-то лишний код у вас написан, хз для чего он.

В итоге ваш код будет выглядеть так: https://jsfiddle.net/beLsd91g/1/

Comments

[myskypesla]

Спасибо большое за ответ! Постараюсь думать глобальнее! Лишний код Вы про nextAll? Это нужно чтобы при клике на элемент в первой колонке, у меня снимались все активные элементы в других колонках и + 3 и 4 колонка становилась disabled.

Answer 2

[Денис Инешин]

Если вас все устраивает в чем проблема? И с чего вы взяли что это затратно для браузера?) На мелких проектах вы никогда не упретесь в потолок производительности.

Comments

[myskypesla]

А если это интернет магазин посещаемость у него от 100 тыс в день? Клиент требует от нас максимально возможную производительнось, а я JS недавно освоил, вот решил поинтересоваться. Спасибо за ответ.

[Денис Инешин]

myskypesla: при большой посещаемости имеет смысл только улучшать производительность бекенда. На клиентской стороне это начинает играть роль, тогда тогда когда сайт становится большим и сложным приложением.

Answer 3

[WebDev]

Все правильно.

Comments

[myskypesla]

Спасибо за ответ.