質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

新規登録して質問してみよう
ただいま回答率
85.50%
jQuery

jQueryは、JavaScriptライブラリのひとつです。 簡単な記述で、JavaScriptコードを実行できるように設計されています。 2006年1月に、ジョン・レシグが発表しました。 jQueryは独特の記述法を用いており、機能のほとんどは「$関数」や「jQueryオブジェクト」のメソッドとして定義されています。

Q&A

解決済

3回答

3546閲覧

jQueryのaddClass()とremoveClass()の使い方について

azuapricot

総合スコア2341

jQuery

jQueryは、JavaScriptライブラリのひとつです。 簡単な記述で、JavaScriptコードを実行できるように設計されています。 2006年1月に、ジョン・レシグが発表しました。 jQueryは独特の記述法を用いており、機能のほとんどは「$関数」や「jQueryオブジェクト」のメソッドとして定義されています。

0グッド

0クリップ

投稿2019/03/07 07:36

また独学でjQueryを学習していて、疑問に思った点がありましたので質問させていただきます。

まずは現在のコードです。

HTML

1<ul> 2 <li><a href="#" class="dispMaxCount <?= $displayCount == 20 ? 'active' : '' ?>">20</a></li> 3 <li><a href="#" class="dispMaxCount <?= $displayCount == 100 ? 'active' : '' ?>">100</a></li> 4 <li><a href="#" class="dispMaxCount <?= $displayCount == 200 ? 'active' : '' ?>">200</a></li> 5 <li><button class="btn_secondary changeCount">表示件数変更</button></li> 6</ul> 7 8<script> 9 $(function(){ 10 //最大表示件数の変更 11 $('.dispMaxCount').on('click', function(){ 12 $(this).parent().parent().find('.active').removeClass('active'); 13 $(this).addClass('active'); 14 }); 15 }); 16</script>

UI
イメージ説明


表示件数が変えられるようなボタンを作っていたのですが、
なかなかうまくいかず、現在は強制的に親要素のさらに親要素までさかのぼり、
一度activeクラスを全消しし、
最後にクリックした要素にactiveクラスを付与しています。

・・・一応上記のコードでも問題なく動くのですが、
クラスを全消ししたあとで、再度現在のクラスにactiveクラスをつけるというこのコードはいかがなものなのかと思いました。
(また固定概念が邪魔をしていて、もしかしたらこれが一番効率がよいのかもしれませんが・・・)

もう一点、 「.parent().parent()」と記述しているのがかっこ悪く見えたのですが、このように書かずとも済むものなのか、この記述方法は至って一般的なのかをお聞きしたいです。

お手隙の方がいらっしゃいましたらご教示いただけますと幸いです。

気になる質問をクリップする

クリップした質問は、後からいつでもMYページで確認できます。

またクリップした質問に回答があった際、通知やメールを受け取ることができます。

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

guest

回答3

0

ベストアンサー

.activeを追加したり削除したりするのは親要素ではなく<a>タグですよね。

jQuery

1 $(function(){ 2 //最大表示件数の変更 3 $('.dispMaxCount').on('click', function(){ 4 $(this).addClass('active') 5 .parent().siblings().find('.active').removeClass('active'); 6 }); 7 });

HTMLの構造にもよりますが、クリックされた要素(またはその親要素)に.activeを付与してそれ以外は消すというやり方は一般的だと思います。
.siblings()は自身以外の兄弟要素を取得できるので便利です。
また、場合によってはclassToggle()を使って差し替えることもありますが、提示されたようなHTML構造でしたらやはり上記のようなやり方がいいのではと思います。
あと補足ですが、もし$(this)を繰り返し使う場合は、一旦

jQuery

1var $this = $(this);

のように変数にし、それを使う方が処理が速くなります。

投稿2019/03/07 07:55

編集2019/03/07 08:36
cerfweb

総合スコア1899

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

azuapricot

2019/03/08 01:18

すごい!ありがとうございます~! やりたかったことが全部叶えられて感謝感激です・・・! .siblings()も知らなかったので勉強させていただきました! $(this)も次回からは変数に格納しようと思います!ありがとうございました!
guest

0

疑似クラスの付け外しについては知識不足故明確な回答ができないのですが、
parentについてはこんな感じでいかがでしょうか。

jQuery

1$(this).closest('ul').find('.active').removeClass('active');

Qiita - parent連打やめろ

投稿2019/03/07 07:45

madoka9393

総合スコア992

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

0

下記でどうでしょうか。

jQuery

1$('.dispMaxCount').filter('.active').removeClass('active');

投稿2019/03/07 07:41

編集2019/03/07 07:47
n_takapyon

総合スコア443

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

15分調べてもわからないことは
teratailで質問しよう!

ただいまの回答率
85.50%

質問をまとめることで
思考を整理して素早く解決

テンプレート機能で
簡単に質問をまとめる

質問する

関連した質問