Какие недостатки в этом коротком скрипте на jQuery?

Question

[elailasou]

Скрипт для плавной навигации по ссылкам-якорям:

$('a[href^="#"]:not(a[href="#"])').click(function(){
    var link = $(this).attr('href').substr(1);
    var anchor = $('a[name='+link+']');
    var top = anchor.offset().top;
    $("body").animate({scrollTop: top}, 500);
    return false;
});

Answer 1

[raven]

1. Лучше использовать on вместо click.
2. Лучше подписаться на click на body. Если ссылок будет много, то и подписок будет много.
3. Не люблю $(this). Используйте $(e.target) (или currentTarget).
4. Событие лучше отменить с помощью e.preventDefault().
5. По name вы anchor ищете: его хорошо бы в кавычки обернуть. И ещё anchor это не обязательно ссылка.

$(document.body, 'a[href^="#"]:not(a[href="#"])', function(e) {
var link = $(e.target).attr('href').substr(1); // или currentTarget
var anchor = $('[name="'+link+'"]');
var top = anchor.offset().top;
$("body").animate({scrollTop: top}, 500);
e.preventDefault();
})

Comments

[elailasou]

А почему "Лучше использовать on вместо click"? Разница?

[raven]

On лучше использовать потому что вторым параметром можно передать селектор элементов - при использовании click у вас не будет такой возможности.

[raven]

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

[Анна Бакурова]

@raven O_o а чем тут ещё может быть $(this)

[Евгений Петров]

Лучше подписаться на click на body

Слушать лучше всего в document.

@junk1114 вы до сих пор для ссылок используете name?.. Вообще давно для прокрутки используется id, а использование name есть плохой тон. А уж поиск по id многократно быстрее.