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

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

新規登録して質問してみよう
ただいま回答率
85.48%
jQuery

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

Q&A

解決済

2回答

1710閲覧

jQuery のイベント発生が1つずつ増えて行ってしまいます

bouyomisan

総合スコア87

jQuery

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

0グッド

0クリップ

投稿2017/06/12 05:33

フォームで入力した後 enter キーを押したら次のフォームにfocusする機能を作っています。ここではテストとして「表示したよ!」と alertさせています。
この状態で focus した後 enterキーを押したら、1度目は「表示したよ!」とalertされるだけなのですが、2回目からは「表示したよ!」が二回表示され、3度目は三回表示されます。どうしてこんなことになったのでしょうか、教えてください。

html

1<div class="col-sm-6 col-sm-offset-3"> 2 <h2 class="sample">Sign up</h2> 3 4 <%= form_for(resource, as: resource_name, url: registration_path(resource_name)) do |f| %> 5 <%= devise_error_messages! %> 6 7 <div class="field"> 8 <%= f.label :name %> 9 <span id="user_name_count"></span> 10 <span id="user_name_error"></span><br/> 11 <%= f.text_field :name, class: "form-control" %> 12 </div> 13 14 <div class="field"> 15 <%= f.label :email %> 16 <span id="user_email_error"></span><br /> 17 <%= f.email_field :email, autofocus: true, class: "form-control", placeholder: "メールアドレスを入力" %> 18 </div> 19 20 <br/> 21 <span id="user_password_error"> </span> 22 23 <div class="field"> 24 <%= f.label :password %> 25 <% if @minimum_password_length %> 26 <em>(<%= @minimum_password_length %> characters minimum)</em> 27 <% end %><br /> 28 <%= f.password_field :password, autocomplete: "off", class: "form-control" %> 29 </div> 30 31 <div class="field"> 32 <%= f.label :password_confirmation %><br /> 33 <%= f.password_field :password_confirmation, autocomplete: "off", class: "form-control" %> 34 </div> 35 36 <div class="actions"> 37 <%= f.submit "Sign up", class: "btn" %> 38 </div> 39 <% end %> 40</div>

javascript

1$(function(){ 2 $("input").focus(function(){ 3 var target_num = target_form.index(this); 4 var target_nums = (target_form.length - 1); 5 6 target_form.on("keypress", function(e){ 7 if(e.which == 13){ 8 alert("表示したよ!"); 9 return e.which !== 13; 10 } 11 }); 12 }); 13});

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

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

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

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

guest

回答2

0

ベストアンサー

focusイベントの中に.on('keypress')を入れてしまっているので、focusが動くたびにイベントが追加されていきます。

今回の場合、inputfocusしなければその中でkeypressイベントも発生しないので、イベントバブリングを使えばもっとシンプルに書ける気もします。

javascript

1$(function(){ 2 target_form.on("keypress", 'input', function(e){ 3 if(e.which == 13){ 4 alert("表示したよ!"); 5 return false; 6 } 7 }); 8});

投稿2017/06/12 05:47

maisumakun

総合スコア145183

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

0

jQuery.on()は追加式なのでイベントの重複登録に注意しよう - Qiita

↑ということらしいので、「target_form.on("keypress", function(e){」などを何回も実行しているのが原因ですね。

投稿2017/06/12 05:44

tkturbo

総合スコア5572

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

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

ただいまの回答率
85.48%

質問をまとめることで
思考を整理して素早く解決

テンプレート機能で
簡単に質問をまとめる

質問する

関連した質問