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

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

ただいまの
回答率

90.34%

  • React.js

    917questions

    Reactは、アプリケーションのインターフェースを構築するためのオープンソースJavaScriptライブラリです。

Reactで変更したstateがすぐに反映されずに困っています。

解決済

回答 1

投稿 編集

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

k10a

score 12

 前提・実現したいこと

Reactで変更したstateが画面上ですぐに反映されずに困っています。Itemの数量を子コンポーネントのクリックに応じて増やすという仕様で、console.logで見てみると、問題なく増えているのですが、画面でrenderしている箇所には反映されていません。

 試してみたこと

ReactではsetStateは即座に反映されない

上記の記事を参考に、変数を加えてみたのですが現状は変わっていません。
また、setStateで書き換えて見ましたがこちらもうまくいきませんでした。

 該当のソースコード

◾️Select.js

import React, { Component } from 'react';
import ItemList from './common/ItemList';

class Select extends Component {
  constructor(props) {
    super(props);
    this.state = {
      items: [
        {
          id: 1,
          name: 'apple',
          chosen: false,
          quantity: 0
        },
        {
          id: 2,
          name: 'banana',
          chosen: false,
          quantity: 0
        }
      ]
    };
  }

  _handleAdd = id => event => {
      this.setState((prevState, props) => ({
        items: prevState.items.map(prevItem => {
          if(prevItem.id !== id) return prevItem;
          const quantity = prevItem.quantity + 1;
          return { quantity, ...prevItem };
        })
      }));

    console.log('state: ', this.state);
  }

  render() {
    return (
      <div className="App">
        <h1>SelectItem</h1>
              <ItemList
                items={this.state.items}
                handleAdd={this._handleAdd}
              />
      </div>
    );
  }
}

export default Select;

◾️SelectList.js

import React from 'react';
import SelectedItem from './common/SelectedItem';

const SelectedItemList = ({ items, handleAdd }) => (
  items.map(item => (
    <SelectedItem
      key={item.id}
      item={item}
      handleAdd={handleAdd}
    />
  ))
);

export default SelectedItemList

◾️SelectedItem.js

import React from 'react';

const SelectedItem = ({ item, handleAdd }) => (
  <button onClick={handleAdd(item.id) />
  <p>{item.quantity}個購入します</p>
);

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

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

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

    クリップを取り消します

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

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

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

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

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

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

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

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

    質問の評価を下げる

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

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

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

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

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

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

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

    詳細な説明はこちら

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

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

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

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

  • og24715

    2018/07/17 14:52

    state の確認は render メソッド内で行ったほうが確実です

    キャンセル

  • k10a

    2018/07/17 15:15

    貴重なご指摘ありがとうございます。

    キャンセル

回答 1

checkベストアンサー

0

Reactの場合、state内の値はthis.setStateを使って書き換えないと、Reactが変更を認識できません。

そして、前のstateを見て次のstateを作るような場合、それ用のコールバックが必要となります(React公式)。

  _handleAdd = id => event => {
    this.setState((prevState, props) => ({
      items: prevState.items.map(prevItem => {
        if(prevItem.id !== id) return prevItem;
        const percent = prevItem.quantity + 1;
        return { percent, ...prevItem };
      })
    }));
  }

投稿

編集

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2018/07/17 10:06

    ご回答頂きまして誠にありがとうございます。頂いた資料と、ソースコードを参考に更新してみたのですが、console上も変化なしとなってしまいました。
    bindなどする必要があるのでしょうか。

    キャンセル

  • 2018/07/17 10:34

    実際のstateは、非同期で変更されることになるので、
    _handleAddのconsole.logを処理するタイミングでは、変更が反映されないですね。

    実際のコントロールも、表示されないのでしょうか?

    キャンセル

  • 2018/07/17 10:58

    ご回答ありがとうございます。
    <p>{item.cart}個購入します</p>
    の部分も0のままとなっています。
    子コンポーネントを参照している点も何か関係しているのでしょうか。

    キャンセル

  • 2018/07/17 11:12

    質問のソースは、maisumakunさんの回答にしたがって修正した内容ですか?

    今の内容だと、item.quantityに個数が入るように見えます。
    prevItem.cartを更新しないといけないように思いますが、どうでしょう?

    キャンセル

  • 2018/07/17 12:01

    大変申し訳ございませんでした。こちら、cartとquantityが分けて記されていました。
    quantityで統一して、ソースコードを書き換えて見たのですが、やはり動きませんでした。

    キャンセル

  • 2018/07/17 14:05

    Select.jsでimportしているItemListとSelectListは別物ですよね?
    そこのつながりが影響しているような・・・

    キャンセル

  • 2018/07/17 14:11

    ありがとうございます。一度、ソースコードを見直して見ます。

    キャンセル

  • 2018/07/17 15:15

    正しいやり方かは分かりませんが、下記で正常に動きました。

    ```
    item.quantity = item.quantity - 5;
    this.setState({
    item
    });
    ```

    キャンセル

  • 2018/07/17 15:17

    @maisumakun さん
    > そして、前のstateを見て次のstateを作るような場合、それ用のコールバックが必要となります

    ちょっと語弊を感じました。
    setState は updater と callback の2つを引数に取りますが、恐らく maisumakun さんの差す コールバックとは updater です。
    更には、前の state を元に新しい state を作り出すなら updater は関数である必要はありません。

    this.setState({ count: this.state.count + 1 })

    とすることでカウンターをインクリメント出来ます。

    updater が関数(?コールバック)である必要は、以前の state を保証するためです。
    setState が非同期であるが故に、複数の setState の呼び出しによりキューイングが発生した時、 state が期待通りの値でない可能性があるため、以前の state を保証するために updater に関数を使用し、prevState 引数として以前の state を取ります。

    キャンセル

  • 2018/07/17 17:36

    @og24715 さん

    最終的には、最後のsetStateで渡されたstateになるはずなので、その呼出が行われた際のstateが保証されないと、正しい値にならない可能性があると思いますが、どうでしょう?

    もしそうであれば、maisumakunさんの回答が、正しいように思います。

    キャンセル

  • 2018/07/17 18:11

    @Meganezaru さん
    > どうでしょう?

    その解釈で間違いないありません。
    maisumakun さんの回答が間違いとも言ってません。

    "以前の state に依存した state の更新" は必ずしも updater がコールバック関数である必要はないという指摘です。

    キャンセル

  • 2018/07/18 09:27

    @og24715 さん

    this.setState({ count: this.state.count + 1 })
    が正しく動作するかどうかは、状況依存ですよね?
    this.state.countに必ず最新状態が保存されていると確信できる時。

    setStateが呼び出される間隔が十分に長い、
    という条件(あやふやで申し訳ないですw)かな?

    そういう意味合いでいけば、
    直前ステータスを利用してステータスを更新する場合は、
    コールバックを使いましょう!の方が安心、ってことだと思います。
    (maisumakunさん、違ってたらごめんなさいw)

    コールバックを利用することに、弊害があるのであれば、
    それとトレードオフする必要はあると思いますが、
    特にない気もします。学習コスト?とかですかね?
    ちょっと、考えてみたいと思います。

    キャンセル

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

  • React.js

    917questions

    Reactは、アプリケーションのインターフェースを構築するためのオープンソースJavaScriptライブラリです。