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

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

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

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

jQuery

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

HTML

HTMLとは、ウェブ上の文書を記述・作成するためのマークアップ言語のことです。文章の中に記述することで、文書の論理構造などを設定することができます。ハイパーリンクを設定できるハイパーテキストであり、画像・リスト・表などのデータファイルをリンクする情報に結びつけて情報を整理します。現在あるネットワーク上のほとんどのウェブページはHTMLで作成されています。

Q&A

解決済

3回答

7280閲覧

イベントハンドラが二重登録されてしまう

k499778

総合スコア599

JavaScript

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

jQuery

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

HTML

HTMLとは、ウェブ上の文書を記述・作成するためのマークアップ言語のことです。文章の中に記述することで、文書の論理構造などを設定することができます。ハイパーリンクを設定できるハイパーテキストであり、画像・リスト・表などのデータファイルをリンクする情報に結びつけて情報を整理します。現在あるネットワーク上のほとんどのウェブページはHTMLで作成されています。

0グッド

0クリップ

投稿2017/08/22 23:23

編集2017/08/23 14:16

現在HTML,javascriptを使って印刷機能を実装しています。

質問があるのですが、
結論から言うと、
印刷ボタンを押下する度にダイアログが開く回数が増えてしまいます。どういった対応をすればいいでしょうか?

またイベントの複数登録をさせないためにonメソッド→oneメソッドに変えると印刷ボタンが1度しか機能しないブラウザが出てきます。

現状のコードは以下です。

javascript

1$("body .btnPrint").on("click",function(){ 2 window.print(); 3 return false; 4});

デベロッパーツールで見るとイベントがボタンを押下する度に登録されていたので、
oneメソッドに変えたところ、
・chrome(60.0.3112.101)では直り
・Firefox(46.0.1)では1度印刷ダイアログを開くと2度とボタンを押しても開かなくなりました。デグレました。

このような状態で一概にoneメソッドで直るとは言えず、どうすればいいか困っています。
もし何かわかる方がいらっしゃれば教えていただければと思います。


追記
すいません。一部誤った情報を流してしまっていました。
上記コードの外側でonメソッドが使われていました。
以下のような構造になっていました。申し訳ありません。

javascript

1$(window).on("load resize",function(){ 2if(flg){ 3flg = false; 4settimeout(function(){ 5 : 6 : 7 8$("body .btnPrint").on("click",function(){ 9 window.print(); 10 return false; 11 }); 12},600); 13} 14}); 15

flgがどういう意図で使われているのかわかりませんが、このような構造になっており、
settimeoutの中にいろいろなメソッドがある状態です。

またHTMLは以下の通りです。

HTML

1<a class="btnPrint" href="#"><span>印刷</span></a>

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

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

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

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

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

maisumakun

2017/08/22 23:25

「$("body .btnPrint").on」以下の外側はどのようなコードになっていますでしょうか。
k499778

2017/08/22 23:30 編集

確認してみます。さらに外側にonメソッドがあるのではないかという心配でしょうか?それに関してはありませんでした。
s8_chu

2017/08/22 23:30

HTMLを追記していただけませんか?
k499778

2017/08/23 14:30

HTML追記いたしました
guest

回答3

0

ベストアンサー

イベントがボタンを押下する度に登録されていた

まずこれをする必要があるのかってところですが、
onしている前後に別の処理があって、そこはボタンを押すたびに実行したいとかでしょうか。

だとしても、リスナー付与の処理を毎回する必要はないと思いますので、
それをoneに変えたりって対応は問題の本質からズレている気がします。
「nullでエラーになるから、nullチェック入れました」みたいな。

さらに外側にonメソッドがあるのではないかという心配でしょうか?

とコメントにありますが、もし本当にボタンを押すたびに
window.print()するonが走っているのであれば、
必ずどこかでそのイベントリスナーが登録されているはずです。

普通に1回だけ登録するのではダメなんですか?

JavaScript

1jQuery(function($) { 2 $("body .btnPrint").on("click",function(){ 3 window.print(); 4 return false; 5 }); 6});

投稿2017/08/23 01:05

root_jp

総合スコア4666

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

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

k499778

2017/08/24 15:13

回答ありがとうございます。 登録されていましたね。冷静になって見るきっかけを作ってくださいました。 そのように対応することで対処できました。ありがとうございました。
guest

0

追加のコードを確認しましたが、windowがリサイズされる度にプリントダイアログを開くイベントが定義されている、といった状態になっています。
同箇所のみ別のwindow.loadで定義するようにしてみてはどうでしょうか?

javascript

1$(window).on("load",function(){ 2 $("body .btnPrint").on("click",function(){ 3 window.print(); 4 return false; 5 }); 6});

投稿2017/08/23 23:56

退会済みユーザー

退会済みユーザー

総合スコア0

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

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

k499778

2017/08/24 15:12

回答ありがとうございます。 resizeも気になりましたが、どうもonが二重になっていたのがそもそもよくなかったようです。 気にかけてくださってありがとうございました。
guest

0

まずセレクタの指定方法があまり良くないように感じます。
$('body').find('.btnPrint').on と記述するか、もしくはスコープをbtnPrintクラスの親クラス等にした方がわかりやすいのではないかと思います。
また、イベントがボタンを押すたびに増えていってしまうようですので、以下で解決しないでしょうか。
$('body').find('.btnPrint').off('click').on('click', function(){
一度offでclickイベントを削除し、その上で再度clickイベントを登録しています。
一度試してみてください。

投稿2017/08/23 00:13

Begi

総合スコア56

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

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

k499778

2017/08/24 15:25

回答ありがとうございます。 そのような書き方もあるんですね。今後参考にしたいと思います。
guest

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

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

ただいまの回答率
85.50%

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

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

質問する

関連した質問