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

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

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

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

HTML

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

Q&A

解決済

2回答

3771閲覧

配列をaddEventListenerで制御できない

AGadget

総合スコア60

JavaScript

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

HTML

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

0グッド

0クリップ

投稿2019/02/16 16:54

先日"HTML文書をJavaScriptで開閉させたい"という題に対して、いくつかの回答を頂きました。
回答を参考に色々なことを試しているのですが、その1つに、どうにも解決方法が分からないものがありましたので質問させて頂きます。

まずは以下のHTMLをご覧ください。

HTML

1<body> 2 <h1>メインタイトル</h1> 3 <section> 4 <div class="section_head"> 5 <h2>サブタイトル1</h2> 6 <p>サブタイトル1についての備考</p> 7 </div> 8 <div class="section_body"> 9 <p>サブタイトル1に関する説明</p> 10 : 11 : 12 : 13 <p>サブタイトル1に関する説明</p> 14 </div> 15 </section> 16 <section> 17 <div class="section_head"> 18 <h2>サブタイトル2</h2> 19 <p>サブタイトル2についての備考</p> 20 </div> 21 <div class="section_body"> 22 <p>サブタイトル2に関する説明</p> 23 : 24 : 25 : 26 <p>サブタイトル2に関する説明</p> 27 </div> 28 </section> 29</body>

やりたい事としては以下の通りです。

  1. JavaScriptが動く環境ならばページ読み込み時に<div class="section_body">タグを非表示にする
  2. <div class="section_head">タグをクリックする度に<div class="section_body">タグが表示状態と非表示状態を繰り返す

そのために以下のようなJavaScriptを書いたのですがaddEventListener処理部分がエラーで動きません。
これが<div class="section_body">タグを非表示にするところからエラーなら「多分、配列関連のことをちゃんと理解してないから動かないんだろうなぁ」と思えるのですが、そこだけは正常に処理されているために尚のこと原因が掴めずにいます。
とりあえずJavaScript入門書片手に、forEachとかdocument.querySelectorAllですとか片っ端から色々試してはいるのですが全く解決する気配がありません。

直接の解決方法もそうですが、そもそも、どこか、どのように間違っているのかすら分かっていない状態です。
長くなりましたが、ご回答のほう、よろしくお願い申し上げます。

JavaScript

1onload = () => { 2 let i; 3 let section_head = document.getElementsByClassName("section_head"); 4 let section_body = document.getElementsByClassName("section_body"); 5 section_head = Array.from(section_head); 6 section_body = Array.from(section_body); 7 //<div class="section_body">を非表示に 8 for(i = 0; i < section_body.length; i++){ 9 section_body[i].style.display = "none"; 10 } 11 //クリックする度開閉 12 for(i = 0; i < section_head.length; i++){ 13 section_head[i].addEventListener("click", () => { 14 if(section_body[i].style.display === "none"){ 15 section_body[i].style.display = "flex"; 16 }else{ 17 section_body[i].style.display = "none"; 18 } 19 }); 20 } 21}

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

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

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

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

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

guest

回答2

0

ブロックスコープ

問題は関数スコープで変数iが共有されている事にあり、for ループ完了時の変数 i が全ての listener 関数から参照されています
kei344 さんの回答では、「ブロックスコープでスコープを分割する(独立させる)」ことで対応しました。

別の解決手段

アプローチを変えると、他にも解決法があります。

event.currentTarget

最も、この場合は event.currentTarget でも参照できるので、細かいテクニックは不要かもしれません。

別の問題

本題ではありませんが、質問文のコードにはいくつか問題があります。

  • addEventListener の第三引数は後方互換性の為に省略しない方が良い
  • listener関数をループ回数分生成してしまっている(メモリの無駄遣い)

JavaScript

1function handleClick (event) { // listener関数を変数に格納しておく 2 var style = event.curentTarget.style; 3 4 style.display = style.display === 'none' ? '': 'none'; 5} 6 7for (let i = 0, len = section_head.length; i < len; i++){ 8 section_head[i].addEventListener("click", handleClick, false); // 格納した変数からlistener関数を指定する 9}

イベントバブリングを利用して親ノードにイベント定義すれば、イベント定義回数を1回に節約できますが、今の段階ではハードルが高そうなので、今後の課題で良いと思います。

Re: AGadget さん

投稿2019/02/17 03:50

編集2019/02/17 03:54
think49

総合スコア18156

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

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

AGadget

2019/02/17 09:49

ご回答ありがとうございます。 まだ完全には理解できていないのですが、いくつかテストをしてみたことで体感的にスコープの仕様が分かってきたように思います。 解決策がfor文内で let = i する以外にも、あるということですので色々試してみようと思います。
guest

0

ベストアンサー

iをforのところでletすればよいのでは。(addEventListenerに登録した関数内のiのスコープを狭くするため)

js

1// for(i = 0; i < section_head.length; i++){ 2// ↓ 3 for(let i = 0; i < section_head.length; i++){

投稿2019/02/16 17:23

kei344

総合スコア69364

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

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

AGadget

2019/02/17 00:37

早速のご回答ありがとうございます。 試してみたところを正しく動作致しました。 本当に助かりました、ありがとうございます。 もし、よろしければ追加でお答え頂きたいのですが、どうして変数iのスコープ範囲を狭くする必要があるのでしょうか? 文頭において宣言された変数iの中身はundefinedです。 そして<div class="section_body">タグを非表示にするfor文において、0が代入され繰り返し処理をするなかで1ずつ値が大きくなることで正常に動作しました(そのはず……)。 そして問題となった項目の開閉処理を行うfor文において再び、i = 0となって初期化されました。 初心者目には、これで問題なく動くように見えるのですが何が問題なのでしょうか。 逆に「変数iのスコープ範囲が狭すぎて、変数iが使えていないよ」というのであれば分かるのですが、スコープ範囲が広いのが不都合というのが、よく分からないでおります。
kei344

2019/02/17 06:10

提示されたコードのスコープはonloadに代入している関数全体になります。forもclickに使用している関数も含みます。 これがどういうことかというと、clickが押されるのは少なくともforが回りきった後になり、そのときの「i」はsection_head.lengthと同じになっているので、クリック内での「section_body[i]」は常に無い状態になります。( 配列は section_body[ section_body.length - 1 ] までしかないので) forでletを使用すると、forブロック内がスコープになります。なので、forが回るたびに別スコープになるためclickに使用している関数内の変数に影響がなくなります。 詳しくはthink49さんの回答にもあるように「繰り返し構文 ブロックスコープ JavaScript」などで検索してみてください。letを使った解決方法以外にもいくつか方法があるため、そのあたりも調べてみてください。
AGadget

2019/02/17 09:56

まだ理解できているとは言い難いのですが色々と試してみたことで感覚的にスコープの動きが分かってきました。 スコープもそうですがfor文の仕様が思っていたものと、かなり異なっているようですので、そちらも更に詳しく調べてみます。
guest

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

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

ただいまの回答率
85.50%

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

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

質問する

関連した質問