react-onsenui の Tabbar で子コンポーネントへ props が渡せません

Question

前提・実現したいこと

構成

親コンポーネント（タブバー描画）
|-- HomePage タプ
`-- SettingsPage タブ

HomePageコンポーネントでのイベント発火で親コンポーネントの state を更新し、それをもう一つの SettingsPageコンポーネントへ props として渡したいのですが渡せません。

発生している問題・エラーメッセージ

親コンポーネントのstateは更新されていますが↓

タブコンポーネントにはpropsとして渡っていません↓

該当のソースコード

// App.js

import React, {Component} from 'react';
import PropTypes from 'prop-types';
import {Page, Tab, Tabbar, Button} from 'react-onsenui';

import 'onsenui/css/onsenui.css';
import 'onsenui/css/onsen-css-components.css';
import './App.css';

// HomePage タブ
const HomePage = (props) => (
  <Page>
    <Button onClick={props.omikuji}>Click !</Button>
  </Page>
);
HomePage.propTypes = {
  omikuji: PropTypes.func,
};

// SettingsPage タブ
const SettingsPage = (props) => (
  <Page>
    {props.scores.map((score) => (
      <div key={score.created_at}>{score.fortune}</div>
    ))}
  </Page>
);
SettingsPage.propTypes = {
  scores: PropTypes.arrayOf(
    PropTypes.shape({
      created_at: PropTypes.instanceOf(Date),
      fortune: PropTypes.string,
    })
  ),
};

// 親コンポーネント
class App extends Component {
  state = {scores: []};
  scoreItems = [];

  // タブの描画
  renderTabs = () => {
    return [
      {
        content: <HomePage key="Home" omikuji={this.handleOnClick} />,
        tab: <Tab key="Home" label="Home" icon="md-home" />,
      },
      {
        content: <SettingsPage key="settings" scores={this.state.scores} />,
        tab: <Tab key="settings" label="Settings" icon="md-settings" />,
      },
    ];
  };

  // HomePage タブのボタンクリックでイベント発火
  handleOnClick = () => {
    const fortunes = ['大吉', '吉', '中吉', '小吉', '末吉', '凶', '大凶'];
    let num = Math.floor(Math.random() * fortunes.length);
    this.scoreItems.unshift({
      fortune: fortunes[num],
      created_at: new Date(),
    });
    this.setState({scores: this.scoreItems.slice()});
  };

  render() {
    return <Tabbar renderTabs={this.renderTabs} />;
  }
}

export default App;

試したこと

・コンストラクタ内の scoreItems 変数を props にすると SettingsPage には渡ります
・しかしそれでも SettingsPage 内の <Page /> では "children: Array[0]" となってしまいます

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

$ node -v
v10.14.2
$ npm -v
6.5.0

create-react-app@2.1.2
react(-dom)@16.7.0
onsenui@2.10.6
react-onsenui@1.11.2

Answer 1

こんにちは。

ドキュメント で <Tabbar /> の使い方を調べてみると、ご質問に挙げられているコードですと、<Tabbar />に与える props が足りないのでは？と思いました。

不足と思われたpropsは、以下２点

• onPreChange
• index

です。これらを追加し、また若干修正を入れた App.js が以下です。

// App.js

import React, {Component} from 'react';
import PropTypes from 'prop-types';
import {Page, Tab, Tabbar, Button} from 'react-onsenui';

import 'onsenui/css/onsenui.css';
import 'onsenui/css/onsen-css-components.css';
import './App.css';

// HomePage タブ
const HomePage = (props) => (
    <Page>
        <Button onClick={props.omikuji}>Click !</Button>
    </Page>
);
HomePage.propTypes = {
    omikuji: PropTypes.func,
};

// SettingsPage タブ
const SettingsPage = (props) => (
    <Page>
        {props.scores.map((score) => (
            <div key={score.created_at}>{score.fortune}</div>
        ))}
    </Page>
);
SettingsPage.propTypes = {
    scores: PropTypes.arrayOf(
        PropTypes.shape({
            created_at: PropTypes.instanceOf(Date),
            fortune: PropTypes.string,
        })
    ),
};

const FORTUNES = ['大吉', '吉', '中吉', '小吉', '末吉', '凶', '大凶'];

// 親コンポーネント
class App extends Component {
    state = { scores: [], index: 0 };

    // タブの描画
    renderTabs = () => {
        return [
            {
                content: <HomePage key="Home" omikuji={this.handleOnClick} />,
                tab: <Tab key="Home" label="Home" icon="md-home" />,
            },
            {
                content: <SettingsPage key="settings" scores={this.state.scores} />,
                tab: <Tab key="settings" label="Settings" icon="md-settings" />,
            },
        ];
    };

    // HomePage タブのボタンクリックでイベント発火
    handleOnClick = () => {
        const num = Math.floor(Math.random() * FORTUNES.length);
        const newItem = {
            fortune: FORTUNES[num],
            created_at: new Date(),
        };
        this.setState({ scores: [ newItem, ...this.state.scores ] });
    };

    render() {
        return (
            <Tabbar
                renderTabs={this.renderTabs}
                onPreChange={({ index }) => this.setState({ index })}
                index={this.state.index}
            />);
    }
}

export default App;

参考までに、上記と同じ App.js を含む Reactアプリを、以下の GitHubレポジトリに上げました。

• https://github.com/jun68ykt/q166449

上記のレポジトリは、古い順に以下の４つのコミットを含みます。

(1) initial commit

(2) Tabberのprops追加、Appのstateにindex追加

(3) fortunesをクリックされる度に作らないようAppの外に移動

(4) this.scoreItemsを削除

上記コミットで、(1) はご質問に挙げられている App.js をそのままコピペしたもので、次の(2) で、 <Tabbar />に不足と思われたpropsを２点追加しました。これでとりあえず意図通りに動くと思います。(3), (4) は、蛇足ながら、ちょっとしたリファクタ案ですので、採用するかどうかは質問者様にお任せします。

以下の画面キャプチャは、上記のReactアプリを yarn start して、「Click！」 を３回クリックしてから、Settings ページを表示させたところです。

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

Comments

[KentaKonno]

ご回答頂きありがとうございました！
期待通りに動作するようになりました。
またリファクタ案についても大変勉強になりました。
とくに (4) では、いったん配列に格納した上で Array.slice() するのではなく、スプレッド演算子を用いてよりスマートに処理できることを教えていただきました。
重ねてありがとうございました。
良いお年をお迎えください。

[jun68ykt]

解決されたようでよかったです！KentaKonnoさんも良いお年をお迎えください。