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

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

ただいまの
回答率

88.92%

jQuery `Uncaught TypeError: Cannot read property 'addClass' of undefined` エラーが出る

解決済

回答 2

投稿 編集

  • 評価
  • クリップ 1
  • VIEW 3,346

hibikikudo

score 230

前提・実現したいこと

「one」チェックボックスを一つでもチェックすると、「all」チェックボックスが外れ、「one」のチェックが全て外れると「all」がチェックされる。
上記にリストの表示切り替えが連動する。

 追記

動くサンプル
https://jsfiddle.net/uetr0pfw/

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

期待する動きは実現できたが、下記のエラーが出てしまいます。 

エラーメッセージ
Uncaught TypeError: Cannot read property 'addClass' of undefined

該当のソースコード

<input type="checkbox" class="all"><label>all</label>
<input type="checkbox" class="one" data-target=".c1"><label>one 1</label>
<input type="checkbox" class="one" data-target=".c2"><label>one 2</label>
<ul id="list">
<li class="c1">target-one</li>
<li class="c2">target-two</li>
</ul>
var checkOne = $('.one');
var checkAll = $('.all');

checkAll.on('click, change', function() {
    if (checkAll.prop('checked')) {
      $('li').addClass('is-selected');
      $('li').prop('data-target').addClass('is-selected');
    }
  });

checkOne.on('click', function() {
    if (checkOne.filter(':checked').length) {
      checkAll.prop('checked', false).prop('disabled', false).change();
    } 
    else {
      checkAll.prop('checked', true).prop('disabled', true).change();
    }
  });

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

jQuery v3.2.1

よろしくお願いいたします。

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

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

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

    クリップを取り消します

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

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

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

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

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

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

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

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

    質問の評価を下げる

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

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

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

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

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

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

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

    詳細な説明はこちら

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

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

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

回答 2

checkベストアンサー

+1

セレクタが不正になっているようです。
下記でどうでしょうか?

$('li').prop('data-target').addClass('is-selected');

$($('li').data('target')).addClass('is-selected');

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2017/03/31 15:06

    ありがとうございます。単純な書き間違いで失礼いたしました。動くサンプルを作成しましたので、よろしければこちらをご覧になった上でアドバイスいただけないでしょうか。
    https://jsfiddle.net/uetr0pfw/

    キャンセル

  • 2017/03/31 15:11

    To: hibikikudoさん
    質問文に実際のコードを追記されてはいかがでしょうか。(決して元のコードを消さないでください)
    また、回答者の指摘した内容と同じ間違いがリンク先のコードにありますので、それの問題では?

    キャンセル

  • 2017/03/31 15:15

    ありがとうございます。元の質問文にもサンプルを追記しました。

    キャンセル

  • 2017/04/01 23:02

    リンク先のコードを拝見すると、
    同様の間違いになっています。

    .data()に.addClassをつなげるのではなく、
    .data()で取得した値をセレクターとして利用する必要があるので、
    下記のようにする必要があります。
    (段階的に、分解した処理にしています。)
    ーーー
    var selector = $('#faq-list').data('target');
    $(selector).addClass('is-selected');
    ーーー

    キャンセル

0

disabledの必要性がよくわかりませんが

$(function(){
  $('.all').on('click, change', function() {
    $('li').toggleClass('is-selected',$(this).prop('checked'));
  });
  $('.one').on('click, change', function() {
    $('li').filter($(this).data('target')).toggleClass('is-selected',$(this).prop('checked'));
    $('.all').prop('checked',$('.one:checked').length==0);
  }).trigger('change');
});

 sample

例示のhtmlから想像されるjavascriptを書いておきます

<style>
li.is-selected{color:lime;}
</style>
<script src="http://ajax.googleapis.com/ajax/libs/jquery/3.2.1/jquery.min.js"></script>
<script>
$(function(){
  $('.js-check-all').on('click, change', function() {
    $(this).prop('disabled',true);
    $('.js-checkbox').prop('checked',false).trigger('change');
  });
  $('.js-checkbox').on('click, change', function() {
    var flg=$(this).prop('checked');
    $('li').filter($(this).data('target')).toggleClass('is-selected',flg).prop('checked',flg);
    var flg=$('.js-checkbox:checked').length==0;
    $('.js-check-all').prop('checked',flg).prop('disabled',flg);
  }).trigger('change');
});
</script>
<ul>
<li><input type="checkbox" name="faq-all" disabled="disabled" checked class="js-check-all" /><label for="checkboxAll" class="form--default__checkbox"><i></i>all</label></li>
<li class="faq01"><input type="checkbox" name="faq-nutrition"  id="checkbox01" class="js-checkbox" data-target=".faq01"/><label for="checkbox01" class="form--default__checkbox"><i></i>1</label></li>
<li class="faq02"><input type="checkbox" name="faq-campaign" class="js-checkbox" id="checkbox02" data-target=".faq02"/><label for="checkbox02" class="form--default__checkbox"><i></i>2</label></li>
</ul>

投稿

編集

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2017/03/31 15:05

    ありがとうございます。上記試してみましたが、ダメでした。動くサンプルを作成しましたので、よろしければこちらをご覧になった上でアドバイスいただけますでしょうか。
    https://jsfiddle.net/uetr0pfw/

    キャンセル

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

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

関連した質問

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