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

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

ただいまの
回答率

88.61%

【jQuery】タブで二つの要素を切替表示

解決済

回答 4

投稿

  • 評価
  • クリップ 2
  • VIEW 2,589

k.r.

score 30

前提・実現したいこと

jQueryで、if構文、.hide().show()メソッドを使用し、タブ表示切替を実装したいですが、
想定の動きをしません。
.addclass()でcssの切替や、if構文以外の方法で実装も視野に入れてはいるのですが、
理解促進のため下記でなぜ動かないのか知りたいです。
JS自体始めたばかりです。

発生している問題・エラーメッセージ

1回目の$('abc').clickでは想定通りいくのですが、
2回目の表示タブを戻す際の$('.Jsyllabary').clickが動作しません。

該当のソースコード

<ul class="list__Jsyllabary">
 <li>あいう<li/>
</ul>
<ul class="list__abc">
 <li>ABC<li/>
</ul>
<div>
 <span class="Jsyllabary"></span> | <span class="abc">A</span></p>
</div>
var Jsyllabary = true;
$(function(){
    if ( Jsyllabary === true ){
        $('.list__abc').hide();
        $('.abc').click(function(){
            $('.list__abc').show();
            $('.list__Jsyllabary').hide();
            Jsyllabary = false;
    });
    } else {
        $('.Jsyllabary').click(function(){
            $('.list__Jsyllabary').show();
            $('.list__abc').hide();
            Jsyllabary = true;
    });
    }
});

補足情報(言語/FW/ツール等のバージョンなど)

jQuery 3.0.0
HTML5

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

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

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

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 過去に投稿した質問と同じ内容の質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

回答 4

+4

理解促進のためのコードとのことですので、動作するコードの提示は避けます。

$(function(){...})の中のコードは、ページがロードされた時に実行されます。ページの作り方にもよりますが、大抵は1度だけしか実行されません。
ですので、else文のコードはおそらく実行されないでしょう。
分かりにくいようでしたら、開発者ツールなどでステップ実行してみてはいかがでしょう?

コードをステップ実行する - 開発ツール | MDN

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

checkベストアンサー

+3

flag(true||false)で、処理を分けたいのでしょうか?

でしたら、jqueryオブジェクトは、まとめた方が良いかもですね
以下に例文示しますので、ご参考にしてみてください。

質問者さんのソースへのコメント

var Jsyllabary = true; 
/* 
ここだと、グローバル変数宣言なので、
オブジェクトの中にいれて、スコープを持たせた方が、安全性が高いと言えます
*/
$(function(){
/*
$(function(){...}}は、DOM解析(HTMLそのもの読み込み)後に
実行したい処理を書く場所って感じです。利用する時は、どのタイミングで使いたいのか、
意識してみると良いです。
※これは、jquery(instance)へバインドされるので、スコープも変わりますから、注意です。
 たとえば、thisとかとか...(^^;
*/
    if ( Jsyllabary === true ){
        $('.list__abc').hide();
        $('.abc').click(function(){
            $('.list__abc').show();
            $('.list__Jsyllabary').hide();
/*
$('.list__abc')
↓
var $abcList = $('.list__abc');
イベントは複数回呼ばれるので、毎回jqueryセレクタでオブジェクトを
生成するよりは、一度、変数代入した方が良いです.
*/
            Jsyllabary = false;
    });
    } else {
        $('.Jsyllabary').click(function(){
            $('.list__Jsyllabary').show();
            $('.list__abc').hide();
            Jsyllabary = true;
    });
    }
});

以下、参考に!

<ul class="listItems list__Jsyllabary">
 <li>あいう<li/>
</ul>
<ul class="listItems list__abc">
 <li>ABC<li/>
</ul>
<div>
 <span class="listTab Jsyllabary"></span> | <span class="listTab abc">A</span></p>
</div>


まず、HTMLが編集できるようであれば、上記のようにすると良いと思います。
classでjqueryオブジェクトを生成する場合、そのclassを持っている要素(DOM)をすべて、
まとめてくれます。
それを利用していきましょう
※単一(まとまった要素の中でもそいつだけ)が欲しいって時の対処も合わせて書くので、参考にw

足したのは、それぞれの対象にclass[listItems, listTab]を一つ足しました。

var JS_TAB_ACTION = {
    set : function(){
        this.tabClass = 'listTab';
        this.itemClass= 'listItems';
        this.Jsyllabary = false;
        this.setup()
    },
    setup : function(){
        var _this = this;
        var $tabs = $('.' + this.tabClass);
        var $items = $('.' + this.itemClass);
        $tabs.on('click', function(e){
            var $this = $(this);
            var index = $tabs.index(this);
            var $target = $items.eq(index);
            /*これで単一要素が取れるます。
            $('.class').eq(index)でラップされた要素から、
            indexにマッチするやつだけ、returnします。*/
            $items.hide();
            $target.show();
            _this.Jsyllabary = (index === 1) ? true : false;
        });
    }
};
$(function(){
   JS_TAB_ACTION.set();
});

上記で質問者さんの望む挙動ができると思います。
ただ、留意があります。
私の書いたソースでは、制限があり、
1つめにタブの表示順(タグの書かれている順番)と表示したいアイテムの順番がリンクしている必要があります。
個別にイベントを張って、処理を分ける必要があれば、申しつけてください。
頑張って書いてみますw

やっていることは、至ってJavaScriptのMDNで推挙している方法で、
オブジェクトを作り、処理をその中へ入れて、呼び出し式で、実行するようにしている
これが、上記の概念・抽象です。

具体的な説明をすると、
var JS_TAB_ACTIONで、オブジェクトを作成
set(){...},setup(){...}をその中に入れる...イメージはハッシュ(連想配列)が良いと思います。
画面loadが終わったら、set()を呼び出し、イベントを発付する
こんなところです。

MDN:(サイト:https://developer.mozilla.org/ja/docs/Web/JavaScript)

分からなかったら、分かるまで、付き合いますから、どんどん聞いて下さいな(*・ω・*)

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

+3

if文の中にイベントリスナーを入れてるので、上記の条件だとelse中の文が実行されず、従って二つ目のイベントリスナーが登録されません。
これで動きます。

$(function(){
    var Jsyllabary = true;
    $('.list__abc').hide();
//初期処理にわざわざhide()を使うより、CSSでdisplay:none;にしておく方がいいと思います
    $('.abc').click(function(){
        $('.list__abc').show();
        $('.list__Jsyllabary').hide();
        Jsyllabary = false;
    });
    $('.Jsyllabary').click(function(){
        $('.list__Jsyllabary').show();
        $('.list__abc').hide();
        Jsyllabary = true;
    });
});

投稿

編集

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

+1

function抜きでやってみたらいかがでしょう?

$(document).ready(function(){
  $('.list__Jsyllabary').show();
  $('.list__abc').hide();
 $('.abc').click(function(){
          $('.list__abc').show();
          $('.list__Jsyllabary').hide();
  });
 $('.Jsyllabary').click(function(){
          $('.list__Jsyllabary').show();
          $('.list__abc').hide();
  });
});


やはり、[ $(document).ready(function() ] が必要ではないでしょうか。

投稿

編集

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

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

  • ただいまの回答率 88.61%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる

同じタグがついた質問を見る