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

Question

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

この星の部分は、

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

という機能です。

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

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

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

▼そのコード

<ul class="c-checklist">

  <li><input id="c-apple" type="checkbox" name="chk_fluits2">
    <p>２つとも表示切替</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>２つとも表示切替</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);

Comments

[defghi1977]

ちなみに「||」の意味は判っとるん？

[退会済みユーザー]

「かつ」のつもりでした。なので【 if ($(id1).prop('checked',false) || $(id2).prop('checked',false))】の部分で、【もしid1にチェックがなくて、かつid2にもチェックがなければ】という条件を付けているつもりでした。（id1id2というのは２つの子チェックのことです。）　そして、その条件について「else」をした「display:none;」が効かないのが不思議で、質問に至っている次第でございます。

[defghi1977]

JavaScriptを使い始めて7ヶ月も経っているのに「||」と「&&」の区別すら付いていないとは…

[退会済みユーザー]

ありがとうございます。「かつ」は「&」でしたね。

[defghi1977]

もうJavaScriptの習得そのものを諦めたら如何です？もっと別の例えばCSSに集中するとか

[退会済みユーザー]

アドバイスありがとうございます！参考にさせて頂きます♪

[defghi1977]

「&&」と「＆」の区別もついておられんようで

[退会済みユーザー]

あ、「&」って私が書いたのね。書き間違いでした。「&&」でした。ご指摘ありがとうございます。でも「&」いっこだけというのが存在すること知らなかったのでそれも感謝です。調べてみます。

[defghi1977]

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

[退会済みユーザー]

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

Answer 1

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

Comments

[退会済みユーザー]

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

[m.ts10806]

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

Answer 2

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)と評価され、常に真とみなされます。

Comments

[退会済みユーザー]

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

Answer 3

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

//親チェック時
$('.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'))
    })
})

Comments

[退会済みユーザー]

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

Answer 4

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

Comments

[退会済みユーザー]

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

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