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

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

ただいまの
回答率

90.34%

  • JavaScript

    22141questions

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

  • jQuery

    8775questions

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

jQueryの「||」の使い方がよくわかりません。

解決済

回答 4

投稿 編集

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

退会済みユーザー

下のjqueryについて「★★★★」が機能しないのは、なぜでしょうか?

この星の部分は、

1つでも子にチェックがあるときに、親のエリアがクリックされたならば、そのときは子の要素を全て非表示にしたい。

という機能です。

現状はなぜか「その子は消えて、もう片方の子が表示されてしまう」という状態です。

どうやら、「||」が機能せずdisplay:none;が効かずに、.toggleだけが効いている状態のようです。

▼動くサンプル
https://jsfiddle.net/1mfv6qho/

▼そのコード

<ul class="c-checklist">

  <li><input id="c-apple" type="checkbox" name="chk_fluits2">
    <p>2つとも表示切替</p>
    <ul>
    <li><input id="c-apple-1" class="check-c check-1" type="checkbox" name="chk_fluits" value="c-apple-1" target="c-apple-1"><label for="c-apple-1" class="check_label">apple1を表示切替</label></li>
    <li><input id="c-apple-2" class="check-c check-2" type="checkbox" name="chk_fluits" value="c-apple-2" target="c-apple-2"><label for="c-apple-2" class="check_label">apple2を表示切替</label></li>
    </ul>
  </li>

  <li><input id="c-grape" type="checkbox" name="chk_fluits2">
    <p>2つとも表示切替</p>
    <ul>
    <li><input id="c-grape-1" class="check-c check-1" type="checkbox" name="chk_fluits" value="c-grape-1" target="c-grape-1"><label for="c-grape-1" class="check_label">grape1を表示切替</label></li>
    <li><input id="c-grape-2" class="check-c check-2" type="checkbox" name="chk_fluits" value="c-grape-2" target="c-grape-2"><label for="c-grape-2" class="check_label">grape2を表示切替</label></li>                
    </ul>
  </li>

</ul>

<div class="respnce">
  <span class="c-apple-1 apple">apple1</span>
  <span class="c-apple-2 apple">apple2</span>
  <span class="c-grape-1 grape">grape1</span>
  <span class="c-grape-2 grape">grape2</span>
</div>
////////////////
////親子チェックの連動
(function($){
    $('input[name="chk_fluits"]').on('change',function(){
        var p=$(this).closest('ul').closest('li').find('input[name="chk_fluits2"]');
        var c=$(this).closest('ul').find('input[name="chk_fluits"]');
        if(c.filter(':checked').length==2) p.prop('checked',true);
        if(c.filter(':checked').length==0) p.prop('checked',false);
    }).trigger('change');
    $('input[name="chk_fluits2"]').on('change',function(){
        $(this).siblings().find('input[name="chk_fluits"]').prop('checked',$(this).prop('checked'));
    }).trigger('change');
    $('.c-checklist input[type=checkbox],.check_label').on('click',function(e){
        e.stopPropagation();
    });
    $('.c-checklist>li').on('click',function(e){
        $(this).find('[type=checkbox]').prop('checked',$(this).find('[type=checkbox]:checked').length==0?true:false);
    });
})(jQuery);

////////////////
////項目表示切替
(function($){

  //親チェック時
    $('.c-checklist > li').click(function(){
        var idp = $(this).find('input[name="chk_fluits2"]').attr('id');
        var id1 = $('.' + idp + '-1');
        var id2 = $('.' + idp + '-2');

        //子にチェックがなければ子を表示切替
        if ($(id1).prop('checked',false) || $(id2).prop('checked',false)){
            $(id1).toggle()
            .each(function(){
            if ($(this).css('display') == 'initial')
                $(this).parent().prepend(this)
            });
            $(id2).toggle()
            .each(function(){
            if ($(this).css('display') == 'initial')
                $(this).parent().prepend(this)
            });
        } else {
        //子にチェックがあれば子を非表示★★★★
            $(id1).css('display', 'none');
            $(id2).css('display', 'none');    
        }

    });

  //子チェック時            
    $('input[name="chk_fluits"]').change(function(event) {
        event.preventDefault();
        var idc = $(this).attr('id');
        $('.' + idc)
            .toggle()
            .each(function(){
            if ($(this).css('display') == 'initial')
                $(this).parent().prepend(this)
            });
    });

})(jQuery);

////////////////
////読み込み時
(function($){    
$(document).ready(function () {
    $('input[name="chk_fluits"]').each(function(){
        if ($(this).prop("checked") == true) {
          $( '.' + $(this).attr('id') ).css('display', 'initial');        
        } else {
          $( '.' + $(this).attr('id') ).css('display', 'none');
        }
    });
});
})(jQuery);
  • 気になる質問をクリップする

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

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

    クリップを取り消します

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

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

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

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

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

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

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

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

    質問の評価を下げる

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

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

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

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

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

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

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

    詳細な説明はこちら

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

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

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

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

  • defghi1977

    2018/04/07 11:05

    知識不足が問題ではないと以前申し上げたはず. あなたの学習態度に深刻な瑕疵があるがために, かように時間を掛けても一向に進展が見られないのです. ここが改まらない限り, いくら質問を連ねたところで周りを不幸にするだけです. 今一度, 基礎の基礎の基礎の基礎からやり直した上で相手の立場に立った質問がお出来になられるよう祈っております.

    キャンセル

  • 退会済みユーザー

    退会済みユーザー

    2018/04/07 11:08

    知識不足が問題だと書いた私の先のコメントへのご返信ですね。余計だったので編集してしまったすみません。また丁寧なお返事を書いてくださってありがとうございます。

    キャンセル

  • 退会済みユーザー

    2018/04/08 11:52

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

回答 4

+9

Returns: jQuery
.prop( propertyName, value )

とありますから、$(id1).prop('checked',false) の戻り値は常にjQueryオブジェクトです。
また、

偽とみなされる (falsy) 値 (つまり、false、0、""、null、undefined および NaN) として定義されていない値はすべて、真とみなされる値になります。
Truthy - 用語集 | MDN

とありますから、jQueryオブジェクトは常に真とみなされ、

expr1 を true と見ることができる場合は、expr1 を返します。そうでない場合は、expr2 を返します。
論理演算子 - JavaScript | MDN

とありますから、$(id1).prop('checked',false) || $(idT).prop('checked',false)は常に$(id1)と評価され、常に真とみなされます。

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2018/04/08 13:24

    色々と勘違いしていたみたいですね。次回から「 || 」を使う時に気を付けたいと思います。どうもありがとうございます。

    キャンセル

+9

ツギハギコードが進みすぎてコーディングの一貫性が失われていっているので、その辺りを整理して意味あるコードにスクラップ&ビルドするくらいの気持ちでかえていっては?
ちょこちょこ枝葉をいじっていっても本筋部分は進みません。
自己リファクタリングは技術力を見つめなおし、次の段階に進むキッカケになります。その方が着実に理解でき、しっかりとした土台の技術力が身につきます。
結果、完成が早まります。これまでの感じだと3歩進んで5歩下がってることもあるように見受けられます。

投稿

編集

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2018/04/07 18:06

    同じプロジェクトに参画している他社が雇ったコーダが書いたこういうコードをメンテナンスさせられて死んだことが、ありまぁす。

    キャンセル

  • 2018/04/07 18:09

    まあ今回は自身で書かれたものなので本来は自身が一番分かってなきゃいけないものですけどね。
    理解が曖昧なまま、半分も理解してない状態で来られているようなのでコード量だけ増えて何も進んでない状態から脱せてないわけですしね。

    キャンセル

checkベストアンサー

+2

結局、難しいこと考えずに
チェック状態によって表示/非表示を切り替えるだけなので

//親チェック時
$('.c-checklist > li').click(function(){
    const idp = $(this).find('input[name="chk_fluits2"]').attr('id')
    const id1 = idp + '-1'
    const id2 = idp + '-2'

    $.each([id1,id2], (ix, cl) => {
        $('.'+cl).toggle($('#'+cl).prop('checked'))
    })
})

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2018/04/08 13:23

    どうもありがとうございます。お蔭さまでできました。

    キャンセル

+1

こんにちは
jqueryコード長いですね。もっと短くできると思いますよ。
例えば、
$(this)でチェックボックスのtrue,falseをチェックする → css付与で表示非表示のコントロール
親のチェックボックスのtrue,falseをチェックする → 子のチェックボックスをtrue or false にする → css付与で表示非表示のコントロールでいけるかと思います。

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2018/04/07 10:48 編集

    こんにちは。お世話になります。はい。長いですよね。いろんな方の教えを参考に(1番上のは丸パクリですが)、なんとか自分で書いてみたコードなので、なんだか子供みたいにかわいいです。笑

    なるほど。表示切替の機能というのは、クラス付与でこなすのが基本なのでしょうか。考えてまた取り組んでみます。アドバイスありがとうございます!

    キャンセル

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

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

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

  • JavaScript

    22141questions

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

  • jQuery

    8775questions

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