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

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

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

ECMAScriptとは、JavaScript類の標準を定めるために作られたスクリプト言語です。

JavaScript

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

Q&A

解決済

3回答

1486閲覧

【ES6】複数イベントリスナが登録されるせいで、削除時にイベントが複数回発生するのをどうにかしたい

decteckaaaaaaaa

総合スコア18

ECMAScript

ECMAScriptとは、JavaScript類の標準を定めるために作られたスクリプト言語です。

JavaScript

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

0グッド

1クリップ

投稿2020/03/30 08:31

編集2020/04/11 03:26

標題の通り、クリックしたら動的に増える(ユーザー動作によって増える)要素に対して、増加時にイベントを追加しているのですが、
その追加したものを削除する際にイベントが複数回発生してしまいます。

原因はわかっているのですが、解決方法がわからず
何卒よろしくお願い致します

ためしてみたこと

・デリゲート
・removeEventlistener
┗ 増加する要素のラッパーに対して、クリックイベントを付与してみたり
addEventする前に、イベント自体をremoveしてみたがうまくいかず

コードについて

具体的なコードを提示できないのですが、以下のような作りになっています

function 削除関数() {
・生成関数で生成された要素数に応じて、for文でaddEventListener('click');
・クリックされた要素のindex番号を取得、対象の要素を削除
}

function 生成関数() {
・ユーザー動作に応じて、動的に要素を増やす関数
・削除関数を登録しておく(動的に追加された要素のイベントを有効にするため)
⬆︎ 生成用関数の中で、削除関数が記述されているのは、なんだかよくない気もしています。
}

原因(まとめ)

原因としては、登録時に複数イベントが登録されてしまっていること
→ for文で回しているため、例えば3つ要素がある時は以下のようになっている
「1番目の要素(3イベント)、2番目の要素(2イベント)、3番目の要素(1イベント)」

お手数おかけしますが、何卒よろしくお願い致します。

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

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

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

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

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

guest

回答3

0

イベントのバブリングで解決できます。

具体的なコードを提示できない

CODEPEN こんな感じ文書断片を作って、イベントの挙動をテストすれば開示できますよ。

投稿2020/03/30 09:40

AkitoshiManabe

総合スコア5434

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

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

decteckaaaaaaaa

2020/03/30 13:04

ありがとうございます!こちらもありですね!
guest

0

ベストアンサー

増加時にイベントを追加している

親要素にイベントをつけて、セレクタで合致する場合だけ
反応するようにつくればよいでしょう

投稿2020/03/30 08:45

yambejp

総合スコア116724

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

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

yambejp

2020/03/30 08:56

あとは具体的な挙動がわかるようなサンプルを提示ください
decteckaaaaaaaa

2020/03/30 10:07

ご回答いただきありがとうございます。 具体的なコード公開できずすみません。 > 親要素にイベントをつけて、セレクタで合致する場合だけ > 反応するようにつくればよいでしょう → デリゲートという事でしょうか?  自分でも色々調べてみて実際やってみてもうまくいかず...
yambejp

2020/03/30 10:30

たとえばこんな感じ <script> window.addEventListener('DOMContentLoaded', ()=>{ document.querySelector('#add').addEventListener('click',()=>{ var del=Object.assign(document.createElement('input'),{type:"button",value:"del",className:"del"}); document.querySelector('#hoge').appendChild(del); }); document.querySelector('#hoge').addEventListener('click',e=>{ var n=e.target; if(n.classList.contains('del')){ n.parentNode.removeChild(n); } }); }); </script> <input type="button" id="add" value="add"> <div id="hoge"> </div>
decteckaaaaaaaa

2020/03/30 13:02

ありがとうございます! なるほど... 少し関数に固執しすぎていたのも原因でした。 こちらで解決しました
guest

0

コードがご提示いただけないので隔靴掻痒の感がありますが。

→ for文で回しているため、例えば3つ要素がある時は以下のようになっている
「1番目の要素(3イベント)、2番目の要素(2イベント)、3番目の要素(1イベント)」

for文で回さなければいいのではないかと思いました。
つまり、生成された要素に生成されたときだけイベントを設定すればいいのでは。

投稿2020/03/30 08:55

Lhankor_Mhy

総合スコア36960

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

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

decteckaaaaaaaa

2020/03/30 10:05

ご回答いただきありがとうございます! > for文で回さなければいいのではないかと思いました。 > つまり、生成された要素に生成されたときだけイベントを設定すればいいのでは。 動的に増える且つ削除する際に、何番目の要素かindex検索しなければならないため、 for文で回していました。 動的に増えるイベントを一つ一つにつけるとなると...どういった方法があるのでしょうか?
Lhankor_Mhy

2020/03/30 10:11 編集

具体的なコードがないとわかりにくいかもしれませんが、参照を持っておけばいいと思います。 つまり、要素が生成された時点では、生成された要素=消すべき要素の参照を持っているのですから、クロージャなどで参照を持ったままイベントを設定すればいいのではないでしょうか?
decteckaaaaaaaa

2020/03/30 13:04 編集

なるほど... 今回は別方法で実装になりましたが調べてみます!
guest

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

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

ただいまの回答率
85.35%

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

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

質問する

関連した質問