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

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

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

JavaScriptは、プログラミング言語のひとつです。ネットスケープコミュニケーションズで開発されました。 開発当初はLiveScriptと呼ばれていましたが、業務提携していたサン・マイクロシステムズが開発したJavaが脚光を浴びていたことから、JavaScriptと改名されました。 動きのあるWebページを作ることを目的に開発されたもので、主要なWebブラウザのほとんどに搭載されています。

jQuery

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

Q&A

解決済

1回答

2838閲覧

【jQuery】もっと良い記述の方法があれば知りたいです。サイドバーに現在地のクラス「current」を追加

umauman

総合スコア57

JavaScript

JavaScriptは、プログラミング言語のひとつです。ネットスケープコミュニケーションズで開発されました。 開発当初はLiveScriptと呼ばれていましたが、業務提携していたサン・マイクロシステムズが開発したJavaが脚光を浴びていたことから、JavaScriptと改名されました。 動きのあるWebページを作ることを目的に開発されたもので、主要なWebブラウザのほとんどに搭載されています。

jQuery

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

0グッド

0クリップ

投稿2017/04/19 15:45

###実現したいこと
アドレスバーの最後のパスと、サイドナビゲーションの最後のパスが一致した場合に
サイドナビゲーションの<li>に現在地を示す「class="current"」を付けたく思っております。

色々調べて記述をし、意図通りの動作はするようになったのですが、もし、もっとシンプル、もしくは作法的なことで良い記述方法がありましたら向学のためにご教授いただけないでしょうか?

###HTML

html

1<ul id="side"> 2 <li><a href="/xxx/aaa/">サイドナビ</a></li> 3 <li><a href="/xxx/bbb/">サイドナビ</a></li> 4 <li><a href="/xxx/ccc/">サイドナビ</a></li> 5</ul>

###下記で上手くいきましたが、より良い記述の方法があれば勉強したく思っています。
アドレスバーとサイドバーのアドレスを取得して、「/」で分割して配列にし、最後のパスを取得、この2つがマッチしたら「class="current"」をつける、ということをしているつもりです。

javascript

1var pathname = location.href.split('/'); 2var pathnameLast = pathname[pathname.length - 2]; 3$('#side a').each(function(){ 4 var $href = $(this).attr('href').split('/'); 5 var $hrefLast = $href[$href.length - 2]; 6 if(pathnameLast.match($hrefLast)) { 7 $(this).parent('li').addClass('current'); 8 } 9});

学習する中で、なかなか要領よく記述するコツが掴めず、アドバイスいただけますと嬉しく思います。

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

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

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

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

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

guest

回答1

0

ベストアンサー

下記コードのいずれかは如何ですか。

JavaScript

1jQuery('a[href="' + location.pathname + '"]').addClass('current'); 2jQuery('a[href="' + location.pathname.replace(/[^/]*$/, '') + '"]').addClass('current');

正確には、liの方にクラスを付与したかったので、「.parent('li')」を加えてうまくいきました。

親のli要素にマッチさせたいのなら、:has() を併用すれば jQuery#parent を利用しなくて済みます。

JavaScript

1jQuery('li:has(a[href="' + location.pathname + '"])').addClass('current'); 2jQuery('li:has(a[href="' + location.pathname.replace(/[^/]*$/, '') + '"])').addClass('current');

Re: umauman さん

投稿2017/04/19 15:58

編集2017/04/19 16:26
think49

総合スコア18156

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

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

umauman

2017/04/19 16:20

ご回答ありがとうございます。 記述いただいたいずれのコードでもうまくいきました。 正確には、liの方にクラスを付与したかったので、「.parent('li')」を加えてうまくいきました。 下記で動作しました。 $('a[href="' + location.pathname + '"]').parent('li').addClass('current'); $('a[href="' + location.pathname.replace(/[^/]*$/, '') + '"]').parent('li').addClass('current'); 1行で記述できるとは…。こんな風にさらっと書けるようになりたいです。 本当にありがとうございました。
think49

2017/04/19 16:27

親のli要素にマッチさせたいのなら、:has() を併用すれば jQuery#parent を利用しなくて済みます。 親記事を編集しました。
umauman

2017/04/19 16:56

「:has」でもうまくいきました。このような記述もできるのですね。 大変勉強になりました! 何度も本当にありがとうございました。
guest

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

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

ただいまの回答率
85.50%

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

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

質問する

関連した質問