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

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

ただいまの
回答率

90.51%

  • JavaScript

    16490questions

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

  • jQuery

    6726questions

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

  • HTML5

    4030questions

    HTML5 (Hyper Text Markup Language、バージョン 5)は、マークアップ言語であるHTMLの第5版です。

  • CSS3

    2077questions

    CSS(Cascading Style Sheet)の第3版です。CSS3と略されることが多いです。色やデザインを柔軟に変更することが可能になります。

jQueryでクラスを追加したのにクリックイベントが発生しない

解決済

回答 2

投稿

  • 評価
  • クリップ 0
  • VIEW 1,411

owl

score 33

現状

現在box1のクラス名が付いたdivのみが一つあります。このdivをクリックするとbox1のクラス名が削除され、box2のクラス名が追加され、背景色が赤から緑に変更されます。

発生している問題

box2のクラス名を追加する所まではできたのですが、このbox2のクラス名が付いたdivをクリックするとbox2のクラス名が削除され、box3のクラス名が追加され、背景色が緑から黄色に変更させたいのですが、クリックしても無反応です。

試したこと

コンソールに出力させたりしてみたり、もしかしたらjsやjQueryの読み込み順かなと思い、</body>の直前で読み込ませさせたのですがダメでした。

該当コードをjsfiddleにアップしました。
ご活用していただければ幸いです。
https://jsfiddle.net/ululami/x9he9939/
よろしくお願いいたします。

該当のソースコード

$(function(){
  $('.box1').on('click',function(){
    $('.box1').removeClass().addClass('box2');
  })

  $('.box2').on('click',function(){
    $('.box2').removeClass().addClass('box3');
  })
})
div{
  width: 100px;
  height: 100px;
}

.box1{
  background: red;
}

.box2{
  background: green;
}

.box3{
  background: yellow;
}
<div class="box1"></div>

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

jQuery 3.2.1

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

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

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

    クリップを取り消します

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

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

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

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

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

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

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

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

    質問の評価を下げる

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

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

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

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

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

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

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

    詳細な説明はこちら

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

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

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

回答 2

checkベストアンサー

+3

こんなかんじでどうでしょう。

$( function() {
    $( 'body' ).on( 'click', '.box1', function() {
        $( this ).removeClass( 'box1' ).addClass( 'box2' );
    } );
    $( 'body' ).on( 'click', '.box2', function() {
        $( this ).removeClass( 'box2' ).addClass( 'box3' );
    } );
} );

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


追記:

せっかくなので box1 - box7 を繰り返すのも作ってみた。

$( function() {
    $( 'body' ).on( 'click', function( e ) {
        var box = e.target.className.match( /.*?(box)(\d).*?/ ); // ちなみにクラスが box1 box3 とか指定されている場合はうまく動かない
        if ( box ) $( e.target ).removeClass( box[ 1 ] + box[ 2 ] ).addClass( box[ 1 ] + ( box[ 2 ] % 7 + 1 ) )
    } );
} );

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

投稿

編集

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2017/10/04 23:15

    ご回答ありがとうございます。
    後から追加したクラス名にもイベントを実行させるには、
    'click'の後にクラス名を入れる必要があるのですね。
    調べてみた所もとはliveメソッドのものを統合したという事ですので、
    非常に勉強になりました。
    本当にありがとうございます。

    キャンセル

  • 2017/10/05 02:28

    7で割ればよかったんですね。また勉強になりましたm(_ _)m
    こんなに短く書けるとは!自分も頑張ろうと思います。

    キャンセル

+2

$(function(){
  $('.box1').on('click',function(){
    $('.box1').removeClass().addClass('box2');
    $('.box2').on('click',function(){
      $('.box2').removeClass().addClass('box3');
    });
  });
});


これでとりあえず
.box1->.box2->.box3となります。

 追記

$(function(){
  $('.box1').click(function(){
    changeBoxNum(1);
  });
});

function changeBoxNum(n){
  $('.box'+n).off("click");
  $('.box'+n).removeClass().addClass('box'+(n+1));
  $('.box'+(n+1)).click(function(){
    changeBoxNum(n+1);
  });
}


これで無制限に変更できます。

jsfiddle
※このデモでは7までくると1に戻るようになっています。

 追記2

今までの話を踏まえて全体的に作り直してみました。

<!doctype html>
<html>
<head>
<meta charset="utf-8">
<title>boxN</title>
<script src="https://ajax.googleapis.com/ajax/libs/jquery/1.12.4/jquery.min.js"></script>
<style>
.bx {
  width: 100px;
  height: 100px;
  margin: 10px;
}

.wrapper{
  display: -webkit-box;
  display: -moz-box;
}

.box1{
  background: red;
}

.box2{
  background: orange;
}

.box3{
  background: yellow;
}

.box4{
  background: green;
}

.box5{
  background: aqua;
}

.box6{
  background: blue;
}

.box7{
  background: purple;
}
</style>
<script>
$(function(){
  // 初期化 全ての.bxにイベント付与
  // click.chn -> changeBoxNumに関するclickイベントであることを明記 意味は無いかも。。(^ ^;
  $('.bx').one('click.chn',function(e){
    var clsNames = $(e.target).attr('class').split(' ');
    var n = 1;
    clsNames.forEach(function(val,ind,arr){
      if(val.indexOf('box') != -1) n = parseInt(val.split('box')[1]);
    });
    changeBoxNum(n,e.target);
  });
});

function changeBoxNum(n,et){
    if(n<7){
    $(et).removeClass('box'+n).addClass('box'+(n+1));
    $(et).one('click.chn',function(e){
      changeBoxNum(n+1,e.target);
    });
  }else{
    $(et).removeClass('box7').addClass('box1');
    $(et).one('click.chn',function(e){
      changeBoxNum(1,e.target);
    });
  }
}
</script>
</head>
<body>
<div class="wrapper">
  <div class="bx box1"></div>
  <div class="bx box2"></div>
  <div class="bx box3"></div>
  <div class="bx box4"></div>
  <div class="bx box5"></div>
  <div class="bx box6"></div>
  <div class="bx box7"></div>
</div>
<div class="wrapper">
  <div class="bx box1"></div>
  <div class="bx box2"></div>
  <div class="bx box3"></div>
  <div class="bx box4"></div>
  <div class="bx box5"></div>
  <div class="bx box6"></div>
  <div class="bx box7"></div>
</div>
<div class="wrapper">
  <div class="bx box1"></div>
  <div class="bx box2"></div>
  <div class="bx box3"></div>
  <div class="bx box4"></div>
  <div class="bx box5"></div>
  <div class="bx box6"></div>
  <div class="bx box7"></div>
</div>
</body>
</html>


jsfiddle

これで互いに影響を及ぼすことは無いはずです。

投稿

編集

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2017/10/04 22:39

    追記のコードについて、divに重複してイベントが設定されていくのは想定されている挙動でしょうか?

    キャンセル

  • 2017/10/04 22:41

    ご指摘ありがとうございます。直してみます。

    キャンセル

  • 2017/10/04 22:56

    `$('.box'+n).off("click");`を加えたのですがこれだけで大丈夫でしょうか?

    これを書き加えるくらいならkei344様のコードのようにあらかじめbodyに有限個分つけた方がいいですね(^ ^;

    キャンセル

  • 2017/10/04 23:14 編集

    .one() にすると少しすっきりしそうな気がします。

    【.one() | jQuery API Documentation】
    https://api.jquery.com/one/

    .off() を使うなら、名前空間を使っておけば「全クリックイベント」を削除せずに消せると思います。.removeClass() についても、もし最初に他のクラスが存在する場合、それごと消してしまう恐れがあります。(そんな要件書いていないのでどっちでもいいんですが)

    【jQuery.fn.onに名前空間をつけることができる - Qiita】
    https://qiita.com/yuku_t/items/89044987d45ec58ea7c5

    関数化するのは良いですね。

    キャンセル

  • 2017/10/04 23:18

    ご回答ありがとうございます。
    box2のクリックイベントをbox1のクリックイベントの中に入れると、
    ちゃんと動作するんですね。
    私のやり方だと、生成されたクラス名には反応しないということなのですね。
    また新たに関数を作って実行させるやり方も非常に勉強になりました。
    本当にありがとうございます。

    キャンセル

  • 2017/10/05 00:07 編集

    kei344様

    いろいろと教えていただきありがとうございました!とても勉強になりました。m(_ _)m
    教えていただいたことを元に改めてコードを書いてみました。少し不安かも(^ ^;

    owl様

    以下、私なりの今回の総括を書いてみます。すでに理解していらっしゃるかもしれませんが(^ ^;

    今回の問題の根本は「`$('.box2').on('click',func);`が処理される時に`div`要素に'.box2'がついていないため、この時点で`div`要素にイベントがつかなかった」ということです。

    その場合の解決策がそれぞれ(ちょっと不正確かもしれませんが)
    私 -> '.box2'がつく時にイベントを付ける
    kei344様 -> そもそも'body'にイベントを付ける
    という方法でした。

    イベントはクラスにつくのではなく、要素につくのだということが鍵だったわけですね。

    キャンセル

  • 2017/10/05 15:12

    わざわざわかりやすくまとめていただきありがとうございます。
    とても参考になりました。
    まだまだ自分にはjQueryのことがわかっていないということが痛感しました。
    本当にありがとうございます。

    キャンセル

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

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

関連した質問

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

  • JavaScript

    16490questions

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

  • jQuery

    6726questions

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

  • HTML5

    4030questions

    HTML5 (Hyper Text Markup Language、バージョン 5)は、マークアップ言語であるHTMLの第5版です。

  • CSS3

    2077questions

    CSS(Cascading Style Sheet)の第3版です。CSS3と略されることが多いです。色やデザインを柔軟に変更することが可能になります。