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

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

ただいまの
回答率

90.61%

  • JavaScript

    15898questions

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

  • jQuery

    6530questions

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

addClassとremoveClassのタイミングがよくわかりません

解決済

回答 2

投稿 編集

  • 評価
  • クリップ 0
  • VIEW 268
退会済みユーザー

退会済みユーザー

目的

2つチェックしたときにclass.pair_onが付与されdisplay:block;し、他の2つがチェックされたらそれを外してdisplay:none;にする。

という機能を作りたいです。

しかしaddClassとremoveClassのタイミングがよくわかりません。

現状

現状はremoveClassを書いているのに.pair_onが外れてくれないためdisplay:none;にならず、下図のようにdisplay:block;(水色の中の項目)が増え続けてしまいます。

イメージ説明

どのように書けば目的のように水色の項目を1つにできるでしょうか?

▼上図のサンプル
https://jsfiddle.net/tLv577fn/

▼そのコード

<div class="check-wrap">
        <div>
            <p>2つチェックしてください。</p>
        </div>
        <ul id="fluits_list">
            <li><input id="fluits_apple" class="check_fluits" name="check_fluits" value="apple" type="checkbox">
                <label for="fluits_apple" class="check_label">
                <span class="i apple"></span>
                </label>
            </li>
            <li><input id="fluits_banana" class="check_fluits" name="check_fluits" value="banana" type="checkbox">
                <label for="fluits_banana" class="check_label">
                <span class="i banana"></span>
                </label>
            </li>
            <li><input id="fluits_kiwi" class="check_fluits" name="check_fluits" value="kiwi" type="checkbox">
                <label for="fluits_kiwi" class="check_label">
                <span class="i kiwi"></span>
                </label>
            </li>   
        </ul>
</div>

<div class="output-wrap">
    <p>選択中のフツールはこちらです。</p>
        <div>
      <p><span id="output_name1"></span><span id="output_name2"></span></p>      
        </div>        
</div>

<div class="pair-wrap">
  <div id="pair_none" class="fluits-widget">相性が良いのは何かな?</div>
  <div id="pair_applebanana" class="fluits-widget pair_on">りんごの後のバナナは相性ばつぐん!</div>
  <div id="pair_bananaapple" class="fluits-widget">バナナの後のリンゴはまぁまぁの相性。</div>  
  <div id="pair_applekiwi" class="fluits-widget">りんごの後のキウイはぼちぼちの相性。</div>
  <div id="pair_kiwiapple" class="fluits-widget">キウイの後のリンゴはなかなかの相性。</div>  
  <div id="pair_kiwibanana" class="fluits-widget">キウイの後のバナナはほどほどの相性。</div>
  <div id="pair_bananakiwi" class="fluits-widget">バナナの後のキウイは相性サイコー!</div>
  <div id="pair_applekiwi" class="fluits-widget">りんごの後のキウイは相性サイテー!</div>
  <div id="pair_kiwiapple" class="fluits-widget">キウイの後のリンゴは相性ぴったり。</div>   
</div>
$(function(){

/**************************
/*ピンクと黄緑の部分は完璧
**************************/
  $('input[name=check_fluits]').change(function(){
    var clone=$(this).siblings().find('.i').clone();
    if($('input[name=check_fluits]:checked').length>2) {
      $(this).prop("checked",false);
      alert("チェックは2つまで");
      return false;
    }
    if($(this).prop('checked')){
      $('#output_name1,#output_name2').filter(function(){
        return $(this).html()=="";
      }).eq(0).html(clone);
    }else{
      console.log($.map(clone.prop('classList'),function(x){return "."+x;}).join(""));
      $('#output_name1,#output_name2').has($.map(clone.prop('classList'),function(x){return "."+x;}).join("")).html("");
    }
  });
  $('input[name=check_fluits]').prop('checked',false).filter('[value=apple],[value=banana]').prop('checked',true).trigger("change");

/**************************
/*水色の部分がちょっとあやしい
**************************/ 
    $('input[name=check_fluits]').change(function(){

        //チェックがないときは「#paie_none」だけを表示
        if($('#fluits_list :checked').length == 0){  
          $('#pair_none').fadeIn();
        }

        var pair1 = $('#output_name1 .i').get(0).className.split(" ")[1];
        var pair2 = $('#output_name2 .i').get(0).className.split(" ")[1];
        var pair3 = ('pair_' + pair1 + pair2);
        if($('#fluits_list :checked').length == 2){
        $('#' + pair3).addClass('pair_on');

        //こうすれば水色の相性についての表示を1つにできると思ったのですが…
        $('.fluits-widget:first-of-type').removeClass('pair_on');

        }
    });

});
/*******************
/*チェックエリア(ピンク)
********************/
.check-wrap {
  background: #ffcccc;
}
#fluits_list {
    list-style: none;
    padding: 0;
}
.apple:before {
  content:'りんご';
}
.banana:before {
  content:'バナナ';
}
.kiwi:before {
  content:'キウイ';
}
.grape:before {
  content:'ぶどう';
}

/*******************
/*出力エリア(黄緑)
********************/
.output-wrap {
  background: #d8ffb2;
}

/*******************
/*愛称エリア(水色)
********************/
.pair-wrap {
  background: #c4fdff;
}
.fluits-widget {
    display: none;    
}
.pair_on {
  display: block;
}


長くなってしまってすみません。どなたかご教示頂けませんでしょうか(>_<)
よろしくお願いいたします。

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

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

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

    クリップを取り消します

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

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

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

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

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

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

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

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

    質問の評価を下げる

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

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

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

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

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

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

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

    詳細な説明はこちら

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

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

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

質問への追記・修正、ベストアンサー選択の依頼

  • 退会済みユーザー

    2018/04/21 02:32

    複数のユーザーから「やってほしいことだけを記載した丸投げの質問」という意見がありました
    「質問を編集する」ボタンから編集を行い、調査したこと・試したことを記入していただくと、回答が得られやすくなります。

回答 2

checkベストアンサー

+2

    $('input[name=check_fluits]').change(function(){
                // この部分追加しました。
                $('.fluits-widget').each(function(index,dom){
            $(dom).removeClass('pair_on');
        });

          //チェックがないときは「#paie_none」だけを表示
        if($('#fluits_list :checked').length == 0){  
              $('#pair_none').fadeIn();
        }

        var pair1 = $('#output_name1 .i').get(0).className.split(" ")[1];
        var pair2 = $('#output_name2 .i').get(0).className.split(" ")[1];
        var pair3 = ('pair_' + pair1 + pair2);
        if($('#fluits_list :checked').length == 2){
            $('#' + pair3).addClass('pair_on');
            //こうすれば水色の相性についての表示を1つにできると思ったのですが…
            //$('.fluits-widget:first-of-type').removeClass('pair_on');
                //↑これだと.fluits-widget最初のものしか見なくなってしまいます。
                //さらにここに追加すると、常に2つチェックされている必要があります。
        }
    });

こうでしょうかね?

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2018/04/20 23:19

    こんばんは。ご回答ありがとうございます。さらにコメントまでつけて頂き感謝です。書く位置はなかなか難しいですね。

    ご提示のようにしてほぼ理想の形になりました。どうもありがとうございます!

    …あと、夜分に大変恐縮なのですが、、

    //チェックがないときは「#paie_none」だけを表示

    の部分については、どのように書けばいいのでしょうか?

    これ、質問文に書いていなかったのですが、この機能にも不備がありました。すみません。(ご提示の形ですと、チェックがあるときにもこれが表示されてしまいまして、改めてお聞かせいただけませんでしょうか?)

    お手すきの折に教えて頂ければ幸いでございます。

    キャンセル

  • 2018/04/20 23:33

    if($('#fluits_list input:checked').length == 0) $('#pair_none').fadeIn();
    1行にまとめていますが、$("#fluits_list :checked")ではなく、inputを付けて明確にしたらいいかと思います。

    キャンセル

  • 2018/04/20 23:48

    たびたびありがとうございます。

    …すみません。できないようすです。(>_<)
    https://jsfiddle.net/tLv577fn/9/

    キャンセル

  • 2018/04/20 23:54

    inputの後ろにスペースがあるからじゃないでしょうか?

    キャンセル

  • 2018/04/21 00:04 編集

    たびたび失礼しました。どうもありがとうございます。

    キャンセル

0

    $('input[name=check_fluits]').change(function(){

          //↓追加
        $('.fluits-widget').removeClass('pair_on');

          //チェックがないときは「#paie_none」だけを表示
            if($('#fluits_list :checked').length == 0){  
              $('#pair_none').fadeIn();
        }

        var pair1 = $('#output_name1 .i').get(0).className.split(" ")[1];
        var pair2 = $('#output_name2 .i').get(0).className.split(" ")[1];
        var pair3 = ('pair_' + pair1 + pair2);
        if($('#fluits_list :checked').length == 2){
            $('#' + pair3).addClass('pair_on');

            //こうすれば水色の相性についての表示を1つにできると思ったのですが…
            $('.fluits-widget:first-of-type').removeClass('pair_on');

        }
    });

こうですか?

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

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

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

関連した質問

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

  • JavaScript

    15898questions

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

  • jQuery

    6530questions

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