🎄teratailクリスマスプレゼントキャンペーン2024🎄』開催中!

\teratail特別グッズやAmazonギフトカード最大2,000円分が当たる!/

詳細はこちら
for

for文は、様々なプログラミング言語で使われている制御構造です。for文に定義している条件から外れるまで、for文内の命令文を繰り返し実行します。

スコープ

スコープとは、プログラム内で変数名など、参照可能な有効範囲のことを指します。

JavaScript

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

コードレビュー

コードレビューは、ソフトウェア開発の一工程で、 ソースコードの検査を行い、開発工程で見過ごされた誤りを検出する事で、 ソフトウェア品質を高めるためのものです。

Q&A

1回答

768閲覧

for文で生成した複数のidにイベントがつけられない。

daikiii

総合スコア3

for

for文は、様々なプログラミング言語で使われている制御構造です。for文に定義している条件から外れるまで、for文内の命令文を繰り返し実行します。

スコープ

スコープとは、プログラム内で変数名など、参照可能な有効範囲のことを指します。

JavaScript

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

コードレビュー

コードレビューは、ソフトウェア開発の一工程で、 ソースコードの検査を行い、開発工程で見過ごされた誤りを検出する事で、 ソフトウェア品質を高めるためのものです。

0グッド

2クリップ

投稿2020/12/07 08:19

js

1コード for(let i = 0; i < quizeSet.length; i++){ 2 3 for(var g = 0; g < 3; g++){ 4 h = `<li id="${i}-${g}-1">${quizeSet[i].answers[g]}</li>`; 5 6       let id = `document.getElementById("${i}-${g}-1")` 7   console.log(id); 8 9 } 10 11 document.write(h); 12 }; 13 14 15 16  id.addeventlisner("click",() => { 17 window.alert("ok") 18 }); 19

コードの一部分しかのせていませんが、
上のようにfor文で作った30個のidにaddeventlisnerを付けたいのですが、つけることができません。
consoleで確認してみるとidは30こ取得できているようです。

質問としましては
idの取得の方法(もっといい方法があれば)と、複数のidに対するイベントの付け方を教えていただきたいです。

また、 let id = document.getElementById("${i}-${g}-1")
をfor文の外でやろうとすると、iとjが定義されてませんとエラーが出てしまいます。
for文にはスコープがないと書かれていたので、なぜこのようなエラーが出でしまうのかわかりません。

何か解決策がありましたら、教えていただきたいです

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

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

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

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

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

fumu7

2020/12/07 08:48

[ヒントというか、問題点の指摘] 「consoleで確認してみるとidは30こ取得できているようです」というのは、 ```ここに言語を入力 console.log(id); ``` というコードが30回実行されて、consoleに30行出力された事を意味しているのだと思います。 しかし、イベントを付ける対象の li要素を生成する以下のコードは、"quizeSet.length"回しか実行されません。 ```ここに言語を入力 document.write(h); ``` また、イベントを付ける下記のコードは、1回しか実行されません。(forループの外に書かれているので、繰り返し実行されることはありません) ```ここに言語を入力 id.addeventlisner("click",() => { window.alert("ok") }); ``` このコメントがプログラムの修正に役立つのではないかと思います。
guest

回答1

0

document.writeを使っているうちはなにをやってもまともなソースはかけません

投稿2020/12/07 08:30

yambejp

総合スコア116661

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

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

yambejp

2020/12/07 08:32

動的にHTML要素をつくるならdocument自体にaddEventListener してください e.targetに対してclosestで条件をしぼって反応させればいいでしょう
yambejp

2020/12/07 08:55

ざっくりこんな感じ <script> 'use strict'; const quizeSet=["a","b","c"]; window.addEventListener('DOMContentLoaded',()=>{ quizeSet.map(x=>Object.assign(document.createElement('li'),{className:"fuga",id:`test-${x}`,textContent:'test'})) .forEach(x=>document.querySelector('#hoge').appendChild(x)); }); document.addEventListener('click',e=>{ let y; if(y=e.target.closest('.fuga')){ console.log(y.id); } }) </script> <ul id="hoge"> </ul>
daikiii

2020/12/07 14:19

回答ありがとうございます 試してみます!
guest

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

まだベストアンサーが選ばれていません

会員登録して回答してみよう

アカウントをお持ちの方は

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

ただいまの回答率
85.36%

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

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

質問する

関連した質問