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

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

ただいまの
回答率

87.58%

関数内関数におけるthisの適用範囲について

解決済

回答 1

投稿

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

score 55

以下のコードで[].forEach.call内の関数が働かない理由と、どうしたら働くのかを教えていただけますと幸いです。

<!doctype html>
<html lang="jp">
<body>
    <div class="container">
        a
    </div>
    <div class="container">
        b
    </div>
    <div class="container">
        c
    </div>
    <div class="container">
        d
    </div>
    <div class="container">
        e
    </div>
</body>
<script type="text/javascript" src="test.js" charset="UTF-8"></script>
</html>
let TEST = function()
{
    this.func_sub = function()
    {
        console.log("func_sub");
    }

    this.func = function()
    {
        this.func_sub();
        let span = document.querySelectorAll('div');
        [].forEach.call
        (span, function(s)
                {
                    s.addEventListener('load', this.func_sub, false);
                    this.func_sub();
                }
        );
    }
}
let test = new TEST();
test.func();


疑問点

  • forEach.call内のfunc_subがどちらも働かない理由
  • Uncaught TypeError: this.func_sub is not a functionというエラーがthis.func_sub();に対して出るのに、addEventListener内のthis.func_subには出ない理由
  • どうやったら、[].forEach.call内でfunc_subを働かせられるのか

以上です。よろしくお願いいたします。

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

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

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

    クリップを取り消します

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

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

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

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

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

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

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

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

    質問の評価を下げる

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

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

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

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

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

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

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

    詳細な説明はこちら

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

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

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

回答 1

checkベストアンサー

+3

JS特有のイベント駆動の時のツラミですね。
イベント登録時に使い捨ての無名関数作って渡してやってますが、
この無名関数でスコープが切られてそれに紐づくthisが生成されるので動作しなくなります。

素直に修正するならばthisを別の変数として定義して保管しましょう。

    this.func = function()
    {
        // thisをselfとして定義、慣習としていろんなライブラリでselfという変数名が使われる
        let self = this;

        this.func_sub();
        let span = document.querySelectorAll('div');
        [].forEach.call
        (span, function(s)
                {
                    // selfで再定義した為、thisの上書きも何のそので実行できる
                    s.addEventListener('load', self.func_sub, false);
                    self.func_sub();
                }
        );
    }

letを変数定義に利用しているのはES2015だからIE11は切り捨てるという意思表示でしょうか?
そうであればthis周りは初心者には解決出来ないのでES2015で追加されたクラス構文を使ってください。
https://developer.mozilla.org/ja/docs/Web/JavaScript/Reference/Classes

ES2015流儀の書き方でリファクタリングしつつ動作させるとこうなります。

// クラス名は先頭文字大文字のキャメルケースにすること
class Test {
  // メソッド名はキャメルケースにすること
  funcSub () {
    console.log('func_sub');
  }

  func () {
    this.funcSub();
    // div掴んでるのに変数名spanってなんだよ…divに変更
    const div = document.querySelectorAll('div');
    // 無名関数からアロー関数に変更することでthisの上書きを阻止
    [].forEach.call(div, it => {
      // thisはそのまま使える
      it.addEventListener('load', this.func_sub, false);
      this.func_sub();
    });
  }
}
const test = new Test();
test.func();

投稿

編集

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2019/02/26 21:37

    ありがとうございます。
    letを使ったのは、varよりもletを使うほうが良いというサイトを見たためです。
    変数名spanは最初作ったテストコードがdivじゃなくてspanだったので、書き換えるのを忘れていました。ごめんなさい。

    キャンセル

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

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

関連した質問

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