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

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

ただいまの
回答率

88.64%

setIntervalに指定する値について。

解決済

回答 3

投稿

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

nezpapapa

score 9

 前提・実現したいこと

https://reactjs.org/docs/state-and-lifecycle.html
こちらのサイトを見て勉強しているときに、不思議に感じたことがあったので質問しました。

 該当のソースコード

こちらのサイトに書かれている下のソースコードで、componentDidMountメソッドのsetIntervalの中身を

this.timerID = setInterval(this.tick, 1000);


としたときに動かない理由はなんでしょうか?
私としては、このようにすればメソッドの内容が展開され、うまく動くだろうと思ったのですが...。

class Clock extends React.Component {
  constructor(props) {
    super(props);
    this.state = {date: new Date()};
  }

  componentDidMount() {
    this.timerID = setInterval(
      () => this.tick(),
      1000
    );
  }

  componentWillUnmount() {
    clearInterval(this.timerID);
  }

  tick() {
    this.setState({
      date: new Date()
    });
  }

  render() {
    return (
      <div>
        <h1>Hello, world!</h1>
        <h2>It is {this.state.date.toLocaleTimeString()}.</h2>
      </div>
    );
  }
}

ReactDOM.render(
  <Clock />,
  document.getElementById('root')
);

 補足情報(FW/ツールのバージョンなど)

React, ReactDOM最新バージョン。

ご教授、よろしくお願いします。

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

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

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

    クリップを取り消します

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

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

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

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

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

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

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

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

    質問の評価を下げる

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

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

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

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

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

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

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

    詳細な説明はこちら

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

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

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

質問への追記・修正、ベストアンサー選択の依頼

  • jun68ykt

    2018/08/12 07:47 編集

    ※さきほど、こちらからご質問に挙げられている Clock の動作確認を書きましたが、質問の趣旨を誤読していたので消去しました。

    キャンセル

回答 3

checkベストアンサー

+2

 簡易コード

原因にフォーカスして、簡単なサンプルコードを書きます。

'use strict';
class Foo {
  constructor (a) {
    this.a = a;
  }
  bar () {
    console.log(this.a);
  }
}

var foo = new Foo(2);
var bar = foo.bar;

foo.bar();  // 2
bar();      // TypeError: Cannot read property 'a' of undefined

関数コード内の this 値は関数呼び出した時に決定します。
foo.bar() 呼び出しされると、this値は「fooに束縛」されます。
bar() 呼び出しは、特別な呼び出し方法ではない為、this値は規定値(Strict Mode なら、undefined)になります。

 setInterval

this.timerID = setInterval(this.tick, 1000);

setInterval の第一引数はコールバック関数です。
コールバック関数は変数に代入されて、後で呼び出されるのと同じ扱いになる為、this 値は規定値となります。
しかし、setInterval には第一引数となる関数のthis値を window に束縛する規定がある為、this 値は window になります。

Re: nezpapapa さん

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2018/08/13 03:09

    >関数コード内の this 値は関数呼び出した時に決定します。
    >foo.bar() 呼び出しされると、this値は「fooに束縛」されます。
    >bar() 呼び出しは、特別な呼び出し方法ではない為、...。
    thisには、通常の関数呼び出しではundefinedが、それ以外では呼び出し側のオブジェクトが設定される、と考えていいということ...ですかね。

    回答に書いていただいた、setIntervalの動きのような、例外的なパターンの動作を理解したり、覚えたりするのが大変そうだな、と感じました。
    他の方の回答や、ネット上の資料を読んで出来る限り例外的なパターンについても知っておこうと思います。

    ご回答ありがとうございました!

    キャンセル

+2

ES6 の React メソッドは明示的に束縛しなくてはいけません。
constructor() を次のようにしてください。

constructor(props) {
    super(props);
    this.state = {
        date: new Date()
    };
    this.tick = this.tick.bind(this);
}

投稿

編集

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2018/08/13 03:09

    thisがClockクラスではなく、他のオブジェクトを指してしまうことが原因だったんですね。
    thisを束縛する方法も色々あるようで、とても勉強になりました。

    ご回答ありがとうございました!

    キャンセル

+2

こんにちは。
コンストラクターの処理に  this.tick = this.tick.bind(this); を追加して、以下

constructor(props) {
    super(props);
    this.state = {date: new Date()};
    this.tick = this.tick.bind(this);
}


のようにすることによって、

this.timerID = setInterval(this.tick, 1000);


が意図通り動作します。

以下、this.tick = this.tick.bind(this);が必要な理由です。これを追加しないと、

    tick() {
        this.setState({
            date: new Date()
        });
    }

の this.setState() の this はグローバルオブジェクトを参照し、ブラウザで実行されるならば window になります。windowには setStateというメソッドはないので、

Uncaught TypeError: this.setState is not a function at tick

というエラーになってしまいます。

上記について、(英語ですが)以下の記事が分かりやすいです。

以上、参考になれば幸いです。


追記

以下も参考になるかもしれません。

投稿

編集

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2018/08/13 03:09

    >this.setState() の this はグローバルオブジェクトを参照し、ブラウザで実行されるならば window になります。
    他の回答を読んで、「this.tickはsetIntervalの中で展開されて、普通の関数呼び出しと同じように実行されるから、thisはundefinedになる。しかし、setIntervalの中でthisはwindowになるから、このような動きになる」のだと理解しました。

    回答に載せていただいた、一つ目のリンク先でもそういう雰囲気のことを言いたいのかな、と思っています。

    二つ目のサイトでは、アロー関数を使いthisを束縛(?)する方法を紹介しているようですね。
    Babelを使って、メソッド定義を比較しているところの、普通のメソッド定義という部分のソースコードは理解できませんでしたが、記事全体としては、一つ目のサイトを読んでいたおかげで納得できました。

    三つ目のサイトは、上で読んだ記事に加えて、いくつか別の解決策が載っているようですね。
    ただ、私の英語読解能力では意味が理解できないのが残念です。
    また、Proxyオブジェクトなど、いくつか知らないものが出てきたのでこれも調べておこうと思います。

    ご回答ありがとうございました!

    キャンセル

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

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

関連した質問

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