web-dev-qa-db-ja.com

jquery add remove class

アンカーが4つあります。アンカーをクリックすると、現在のクラスをアンカーに追加し、同時に他の3つのアンカーからクラスを削除します。これが私のコードです。私は何が間違っているのですか?

if ($("ul#thumb a").hasClass("current") {
    $("ul#thumb a").removeClass("current");
    $(this).addClass("current");
});

私のhtmlは次のようになります:

<ul id="thumbs">
    <li>
        <!-- intro page navi button -->
        <a id="t0" class="active" name="t0">The Company</a>

        <ul class="navi">
            <li><a style="display:none"></a></li>
            <li><a id="t1" name="t1">The Brief</a></li>
            <li><a id="t2" name="t2">The Solution</a></li>
            <li><a id="t3" name="t3">The Result</a></li>
        </ul>

    </li>
</ul>
15
mtwallet

イベント委任を使用することにより、すべてのクリックイベントに対して1つのハンドラーのみをバインドします。アンカー(.currentアンカー以外)がクリックされると、そのクラスの.currentアンカーが削除され、クリックされたアンカーが新しい現在のアンカーになります。

$("#thumbs").on("click", "a:not(.current)", function ( event ) {
    $(".current", event.delegateTarget).removeClass("current");
    $(this).addClass("current");
});
29
Sampson

これはあなたを助けると思います

$("a").click(function() {
  $('a').removeClass('current');
  var id = $(this).attr('id');
  $('#'+id).addClass('current');
});
0
Raghvendra