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

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

新規登録して質問してみよう
ただいま回答率
85.50%
React.js

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

Q&A

2回答

2945閲覧

React HOCを使用した共通コンポーネントの作り方について

ss-nantoka

総合スコア18

React.js

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

0グッド

0クリップ

投稿2019/04/30 09:20

前提・実現したいこと

ReactのHOCを使用して、ダイアログ(メッセージボックス)の共通コンポーネントを作成したいです。
ダイアログコンポーネントを作成し、そのコンポーネントをwithBasedComponent.jsでラップして、呼び出し元のコンポーネントでthis.props.xxxの形で使えるようにしたいです。
※他にもスナックバーなどをwithBasedComponent.jsに追加していく予定です
ソースコードは抜粋ですが、実装したところ、ダイアログコンポーネントの指定したメソッドがないとのエラーになりました。
解決方法あるいは上記を実現するにあたり、より良い実装方法があれば、教えていただけないでしょうか?

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

エラーメッセージ TypeError: _this.msgBoxComponent.current.show is not a function

該当のソースコード

msgbox.js

JSX

1import React, { Component, Fragment } from 'react'; 2import { 3 Dialog, 4 DialogContent, 5 DialogContentText, 6 DialogActions, 7 Button 8 } from '@material-ui/core'; 9 10class MsgBox extends Component { 11 constructor(props) { 12 super(props); 13 this.state = { 14 isOpen: false, 15 callback: null 16 }; 17 }; 18 callbackfunc = () => { 19 this.state.callback(); 20 }; 21 msgBoxClose = () => { 22 this.setState({ isOpen: false }); 23 }; 24 msgBoxSetting = (msgCode, callback) => { 25 // ここでDialogの設定をおこなうコードを書く 26 }; 27 show = (msgCode, callback) => { 28 console.log('msgCode', msgCode) 29 console.log('callback', callback) 30 this.msgBoxSetting(msgCode, callback); 31 }; 32 render() { 33 return ( 34 <Fragment> 35 <Dialog 36 open={this.state.isOpen} 37 onClose={this.msgBoxClose} 38 > 39 test 40 </Dialog> 41 </Fragment> 42 ); 43 } 44} 45 46export default MsgBox;

withBasedComponent.js

JSX

1import React, { Component, Fragment } from 'react'; 2 3import MsgBox from './msgbox'; 4 5export default function withBasedComponent(WrappedComponent) { 6 return class extends Component { 7 constructor(props) { 8 super(props); 9 this.state = { 10 }; 11 this.msgBoxComponent = React.createRef(); 12 13 }; 14 showMessageBox = (msgCode, callback) => { 15 console.log('this.msgBoxComponent.current', this.msgBoxComponent.current) 16 this.msgBoxComponent.current.show(msgCode, callback) // ★showメソッドがなしエラーになる 17 }; 18 render() { 19 const newProps = { 20 showMessageBox: this.showMessageBox 21 }; 22 return ( 23 <Fragment> 24 <MsgBox ref={this.msgBoxComponent} /> 25 <WrappedComponent {...this.props} {...newProps} /> 26 </Fragment> 27 ); 28 } 29 } 30}

呼び出し側の例

import withBasedComponent from './with-based-component'; this.props.showMessageBox('001', null) ※第2引数には、ダイアログでOK押下時に実行させたいメソッドがあれば、指定する(this.xxx() のように) export default withBasedComponent(Login);

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

react: ^16.6.3
react-dom: ^16.6.3
@material-ui/core: ^3.6.2

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

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

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

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

guest

回答2

0

React16を使っているならContextAPIを使う方法が良さそうに思えます。
簡単に作ってみました。

https://codesandbox.io/embed/p12rlom4j

投稿2019/05/09 07:19

編集2019/05/09 07:20
odyu

総合スコア548

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

0

ReactのHOCを使用して、ダイアログ(メッセージボックス)の共通コンポーネントを作成したいです。

という内容に対してHOCを使うのが少し違うかなと感じています。
HOCはcontextから値を取りたいなど、直接の親とは別のところから値を取り、子供のpropsに受け渡す程度にするべきだと思います。

多分素直にコンポーネントをちゃんと作成し(同じようなコンポーネントが複数できても良い)、あとから共通化できるところを、どう共通化するかを議論するほうが良さそうに思えます。

よくあるコンポーネントだとreduxを使うとかeventemitterとかを併用するとかが見かけます。

投稿2019/05/09 05:58

odyu

総合スコア548

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

まだベストアンサーが選ばれていません

会員登録して回答してみよう

アカウントをお持ちの方は

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

ただいまの回答率
85.50%

質問をまとめることで
思考を整理して素早く解決

テンプレート機能で
簡単に質問をまとめる

質問する

関連した質問