Оптимизировать, улучшить код (Показывать div по клику на ссылку, каждая ссылка соответствует div, a1 = div1; a2 = div2 и так дал.)?

Question

[Alex Polyakh]

https://jsfiddle.net/anatooly/u9240xja/

Comments

[IceJOKER]

немножко лучше - https://jsfiddle.net/u9240xja/1/

[Alex Polyakh]

IceJOKER, спасибо Вам!

[mib]

Вовсе не обязательно использовать ява-скрипт
https://jsfiddle.net/dyjLvb9s

[Bowen]

IceJOKER: e.preventDefault(); это лишнее. Зачем когда все делает return false ?

[IceJOKER]

Bowen: "затаму" что так как-бы правильнее. но можно конечно убрать(либо return, либо его)

Answer 1

[Денис]

Тут, разве что, можно чуть сократить код и закешировать элементы ".cats", чтобы при каждом клике не выполнять их поиск в документе.

$(function() {
    var cats = $('.cats');
    $('.js-show-some-cat').on('click', function(e) {
        var current = cats.filter('.cat-' + $(this).data('cat-id')).show();
        cats.not(current).hide();
        return false; // или e.preventDefaul();, что в данном случае не принципиально
    });
});

Comments

[Владислав Поляков]

Мой проще. У меня динамически подгружаются , и не нужно ничего кэшировать .

[Денис]

Владислав Поляков вы это серьёзно? То есть, вы считаете, что лучше изменять DOM по ходу пьесы и подгружать изображения динамически, особенно на слабой скорости подключения инета?

[Владислав Поляков]

Денис: Вот именно , зачем загружать сразу все ненужные изображения ? Сразу , чтоб увеличить загрузку сайта ? Вдруг пользователь просмотрит только 2 изображения, а все остальные 98 , ему будут не нужны , а он тупо ждал 5 минут загрузки всех фото.

[Денис]

Владислав Поляков мде... похоже, что вести диалог дальше - бессмысленно.

[Alex Polyakh]

Денис, скажите пожалуйста если у меня три ссылки, которые открывают разные div. На сколько будет плохим код, если на каждую ссылку навесить событие отдельно для показа определенного блока? Спасибо

[Денис]

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

Answer 2

[Владислав Поляков]

Держи