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

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

ただいまの
回答率

89.06%

javascriptのif文の処理に関して

解決済

回答 2

投稿

  • 評価
  • クリップ 0
  • VIEW 919

hamotaga54

score 13

 if文をクラス名で判別して、処理を分けたい

isotopeとInfinite Scrollを使ったページを作成しています。
1回目にリストをクリックした時は、if文の最初の処理を行い、
2回目以降はelseの処理を行いたいと思っています。
判定するclassも、javascriptでクラスを付与しています。
どうしても、1回目をクリックした時に、if文の処理がされません。
2回目以降は、elseの処理は動いています。
色々と試したのですがうまくいきません。
ご教授頂けますとうれしいです。
よろしくお願いいたします。

 該当のソースコード

  <script type="text/javascript">

  $('#filters-2').on( 'click', 'li', function() {
      $("#container").addClass("selected");
  });

  var element = document.getElementById('container');
  $('#filters-2').on( 'click', 'li', function() {
    if( element.classList.contains('selected') == false ){
      $(function() {
        $.ajax({
          url: 'pages/2.html',
          cache: false,
          success: function(html){
            $('#filters-2').on( 'click', 'li', function() {
                $(html).find('#container').each(function(){
                    $('#container').prepend(this).isotope( 'reloadItems' ).isotope({ sortBy: 'original-order' });
                  return false;
                });
            });
          }
        });
      });
    }
    else {
      var $container = $('#container').isotope();
      $('#filters-2').on( 'click', 'li', function() {
        var filterValue = $(this).attr('data-filter');
        $container.isotope({ filter: filterValue });
        return false;
      });
    }
    return false;
  });
  </script>
      <ul class="work_cate button-group" id="filters-2">
        <li class="cate_all" data-filter="*">ALL</li>
        <li class="cate_item1" data-filter=".item-1">ITEM1</li>
        <li class="cate_item2" data-filter=".item-2">ITEM2</li>
        <li class="cate_other" data-filter=".item-other">OTHERS</li>
      </ul>
    <div id="container" class="clearfix infinite-scrolling">
      <div class="element cate_item1" data-symbol="H" data-category="other">
        <p class="number">1</p>
      </div>
      <div class="element cate_item2" data-symbol="H" data-category="other">
        <p class="number">2</p>
      </div>
    </div>
  • 気になる質問をクリップする

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

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

    クリップを取り消します

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

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

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

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

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

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

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

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

    質問の評価を下げる

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

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

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

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

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

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

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

    詳細な説明はこちら

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

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

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

回答 2

checkベストアンサー

+4

isotopeについてはよくわかりませんが

  $('#filters-2').on( 'click', 'li', function() {
$("#container").addClass("selected");
});

で、#filters-2をクリックするたびにつねにselectedを付加しているので
ifで判断している

if( element.classList.contains('selected') == false ){

すなわち

if( !$('#container').hasClass('selected')){

は、常に偽になりますのでelseに流れるでしょうね

流れ的には以下のようにするとよいのではないでしょうか?

$(function(){
  $('#filters-2 li').on('click',function(){
    if( !$('#container').hasClass('selected')){
      console.log(1);
      $("#container").addClass("selected");
    }else{
      console.log(2);
    }
  });
});

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2018/03/07 12:52

    このソースで試してみたらうまくいきました。
    ありがとうございます。
    すごく、数時間悩んでいたので助かりました!

    キャンセル

+4

$('#filters-2').on( 'click', 'li', function() {
    $("#container").addClass("selected"); // ……A
});

var element = document.getElementById('container');
$('#filters-2').on( 'click', 'li', function() { // ……B
  if( element.classList.contains('selected') == false ){
    // ……C
  }
});


という構造のため、#filters-2がクリックされると、A→Bの順で実行されます。そうすると#containerにはselectedというclassが必ずついていますので、Cが実行されることはありません。
Aの処理をCの最後で行えば、想定した動作になるのではないでしょうか。

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2018/03/07 14:27

    ご回答ありがとうございます!
    isotopeが原因かもしれないですが、
    順番を変えても想定どおりになりませんでした。

    キャンセル

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

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

関連した質問

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