簡潔に書きたいのですが...

質問

シングルページのホームページで、
表示されているエリアに来たら、メインメニューの該当する箇所にclass名'visit'を付けたり、消したりしてるのですが、もっとスマートな書き方はないでしょうか？

これでも意図した動きにはなるのですが、少し勉強させて下さい。
よろしくお願いします。

------------------------------------------------------------------------------------------
$(function() {
function checkPosition() {
var posy = $(window).scrollTop();
var winh = $(window).height();

if (posy <= winh) {
$('.go-1 a').addClass('visit');
} else {
$('.go-1 a').removeClass('visit');
}

if (posy > winh && posy <= winh*2) {
$('.go-2 a').addClass('visit');
} else {
$('.go-2 a').removeClass('visit');
}

if (posy > winh*2 && posy <= winh*3) {
$('.go-3 a').addClass('visit');
} else {
$('.go-3 a').removeClass('visit');
}

if (posy > winh*3 && posy <= winh*4) {
$('.go-4 a').addClass('visit');
} else {
$('.go-4 a').removeClass('visit');
}

if (posy > winh*4 && posy <= winh*5) {
$('.go-5 a').addClass('visit');
} else {
$('.go-5 a').removeClass('visit');
}
}

$(window).scroll( checkPosition );
});

回答No.1

めとろいと（@naktak） · ベストアンサー

JavaScriptやjQueryはすぐ忘れるし全然わからないんですが、
明らかに繰り返しまくってる部分を

for (var i = 0; i < 5; i++) {
if (((i > 0 && posy > winh * i) || i == 0) && posy <= winh * (i + 1)) {
$('.go-' + (i + 1) + ' a').addClass('visit');
} else {
$('.go-' + (i + 1) + ' a').removeClass('visit');
}
}
とかって出来ないんですか？

お礼

求めてた完璧のものです。
勉強になります。

ありがとうございました。