Конфликт скриптов

[Unnamed75]

Как исправить конфликт двух скриптов?

jQuery(document).ready(function($) {
		var url=document.location.href;
		$.each($(".header-menu a"),function(){
		if(this.href==url){$(this).addClass('menu-active');};
		});
})(jQuery);

Хочу продублировать этот скрипт, пишу вот так:

jQuery(document).ready(function($) {
		var link=document.location.href;
		$.each($(".sushi a"),function(){
		if(this.href==link){$(this).addClass('sushi-active');};
		});
})(jQuery);

Но он почему то не работает, где я допустил ошибку?

[danik.js]

$.each($(".header-menu a, .sushi a") ...

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

[Vlasenko Fedor]

$.each($(".header-menu a, .sushi a") ...

$("a[href='" + document.location.href + "']").addClass('sushi-active');

Зачем выбирать, а затем в цикле перебирать.
Может стоить сразу выбрать нужные

[danik.js]

Poznakomlus, это не тоже самое. Как правило, такой подход не будет работать.

[Vlasenko Fedor]

Как правило

Какое правило? Поясни
Вообще выборки типа

$('.cls')

- считаются не оптимизированными. Так пишут те, кто не понимает как происходит поиск по коллекции DOM элементов. Правильнее задавать хотя-бы элемент среди которых производиться поиск. К примеру

$('li.cls')

Работа с ссылками здесь вообще, отдельная история. Не стоит забывать что массив всех ссылок есть уже в

document.links

[danik.js]

Начнем с того что задача подсветки активных пунктов решается на уровне CMS. Даже зассанная джумла это умеет делать. Автор видимо либо юзает херню какую-то, а не CMS, либо не в курсе как сконфигурировать.

Какое правило? Поясни

Все просто. location.href - абсолютный урл, включающий схему.
А в атрибут href как правило пишут относительный урл:

<a href="/">Home</a>
<a href="/about">About</a>

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

считаются не оптимизированными

Кем? Тобой? Ссылку плиз. Погонял в хроме - вариант без тега работает в два раза быстрей. В любом случае на разницу можно только дрочить. Тут другое:

Так пишут те, кто не понимает как происходит поиск по коллекции DOM элементов

Так пишут те, кто знает, что привязываться к тегам - плохо.

[danik.js]

Poznakomlus, ты меня огорчаешь

[Vlasenko Fedor]

Все просто. location.href - абсолютный урл, включающий схему.
А в атрибут href как правило пишут относительный урл:

Какое это отношение имеет к вопросу Т. С.
Если у него линк var url=document.location.href; и его так устраивает

Кем? Тобой?

тут и смотреть не надо, видно количество проведенных итераций при разных подходах
Про скорость здесь комментировать не буду.

привязываться к тегам - плохо

Не видишь здесь паранои в области привязки

$('li.cls')

и

$('.cls')

с каких делов ты li будешь менять? какие затраты будут на замену? насколько информативно $('li.cls') и $('.cls')?
что будет если добавят класс в элементе котором не надо обрабатывать. Таких причин и условий может быть ...

[рони]

if(url.indexOf(this.href)!=-1){}

[danik.js]

Какое это отношение имеет к вопросу Т. С.

Наверное такое что твой код попросту не будет работать? И он вообще почти ни на одном сайте не будет работать. Или ты не считаешь это проблемой?

тут и смотреть не надо, видно количество проведенных итераций при разных подходах

Где видно? Куда смотреть? Как при большем числе итераций код отрабатывает быстрей?

Про скорость здесь комментировать не буду.

Какой ты загадочный. Ну раз начал гнуть свою линию, давай продолжай.

Или я правильно понимаю, что нужно использовать только такие селекторы, которые "считаются оптимизированными", даже если они работают медленней и менее практичны?

Твои доводы по li неубедительны и не подтверждаются в реальных проектах. Зайди на гитхаб и посмотри код популярных js-скриптов.