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

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

ただいまの
回答率

88.05%

アラート関数の表示される値がliタグの番号とずれてしまう

解決済

回答 3

投稿

  • 評価
  • クリップ 0
  • VIEW 366

score 73

jsの勉強をしていますがリスト項目のクリックした項目のliタグ中のテキストと数字を表示させるようにしたいのですがうまくいきません(m_ _m)
わかる方がいらっしゃれば教えていただきますようよろしくお願いいたします(m_ _m)

エラー内容

エラー表示はなし

期待する動作

html内のliタグのテキストと変数iの値がアラートで正しく表示されるようにしたい

試したこと

li.addEventListner以降の部分を

function(){

}())


で囲んでみましたがうまくいきませんでした(m_ _m)

該当のソースコード

<!DOCTYPE html>
<html lang="ja">
<head>
  <meta charset="UTF-8">
  <meta name="viewport" content="width=device-width, initial-scale=1.0">
  <meta http-equiv="X-UA-Compatible" content="ie=edge">
  <title>Document</title>
</head>
<body>
  <ul>
    <li>one</li>
    <li>two</li>
    <li>three</li>
    <li>four</li>
    <li>five</li>
  </ul>

  <script type="text/javascript">
    var items = document.querySelectorAll('li')

    for(var i = 0; i < 5; i++){
      var li = items[i]

      // (アラート表示で常にfive:5が表示される)
      li.addEventListener('click', function () {
        alert(li.textContent + ':' + i)
      })
    }
  </script>

</body>
</html>
  • 気になる質問をクリップする

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

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

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 過去に投稿した質問と同じ内容の質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

回答 3

checkベストアンサー

+2

function () { alert(li.textContent + ':' + i )}と関数を作った時点では、liiの値はまだ参照されません。この関数が実行されるときに初めて、liiの値が参照されます。

実際にこの関数が実行されるのはfor文が終わったあと、<li>がクリックされたときです。そのときのliiの値は何でしょうか?varで宣言された変数はfunction scopeですから、for文を通してliiという変数はそれぞれ1個しかありません。for文はi=5、li=5つ目の<li> になって終了しました。そのため、どれをクリックしてもfive:5がアラート表示されます。

これはES6というJSのバージョンで追加されたconstletを用いることで解決できます。constletで宣言された変数はblock scopeであり、そのブロック限定の変数となります。例えば下のようにブロック内で宣言された変数はその中でしか使えません。

if (true) {
  let x = 3
}
console.log(x)   // Uncaught ReferenceError: x is not defined


同様に、while文やfor文内でlet/constで宣言すると、反復するごとに新しい変数が作られます。

kei344さんの回答のように、for(let i = 0; ...) { }letを用いて宣言するとiという変数はループが反復するごとに作られます。つまり、全体を通してi=0、i=1、i=2、i=3、i=4という別々の変数が作られます。最初の関数function () { alert(li.textContent + ':' + i )}はそれぞれ別のiを指すようになるため、クリックした<li>に対応するiが表示されるようになります。

liについても同様に、letあるいはconstで宣言することによって、反復するごとに新しい変数liが作られるようになるので、意図した動作になります。

Demo

参考
クロージャ - MDN

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2019/04/02 23:05

    とても詳しい解説ありがとうございます!!
    おかげさまでもやもやがスッキリしました!!

    キャンセル

+2

// for(var i = 0; i < 5; i++)
//     ↓ スコープの問題です。
   for(let i = 0; i < 5; i++)

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2019/04/02 18:18

    動作確認して問題なく動くことが確認できました!!
    ありがとうございます!!

    キャンセル

+2

<script>
window.addEventListener('DOMContentLoaded', function(e){
  [].forEach.call(document.querySelectorAll('li'),function(x,y){
    x.addEventListener('click', function (e) {
      alert(x.textContent+':'+(y+1));
    });
  });
});

</script>
<ul>
  <li>one</li>
  <li>two</li>
  <li>three</li>
  <li>four</li>
  <li>five</li>
</ul>

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2019/04/02 17:42

    動作確認して問題なく動くことが確認できました!!
    jsに関してまだまだ初心者なので調べながら、参考にしてより理解を深めていきたいと思います!!
    回答いただきありがとうございます!!

    キャンセル

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

  • ただいまの回答率 88.05%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる

関連した質問

同じタグがついた質問を見る