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

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

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

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

Q&A

解決済

1回答

4261閲覧

react 無限ループします... (react, redux)

yuki_nagahama

総合スコア17

React.js

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

0グッド

0クリップ

投稿2017/01/04 05:24

編集2017/01/04 05:27

現在react-reduxにてチャットアプリケーションを作成しています。
inputのvalueの参照方法がいまいちわからないのでこのようなコードでinputの中身を参照していますが、
このコードだとブラウザーが無限ループしてしまうようで、テストができずに困っています。

class InputArea extends React.Component{ constructor(){ super(); this.state = { inputText: '' } } _onChanges(e){ this.setState({ inputText: e.target.value }) } render(){ const { onSend } = this.props; return( <div> <input type="text" placeholder="input your text !!" onChange={this.state._onChanges} /> <button onClick={this.props.onSend(this.state.inputText).bind(this)}>送信</button> </div> ); } } const mapDispatchToProps = (dispatch) => { return{ onSend:(object) => {dispatch(sendMessage(object))} } }

なぜこのコードが間違っているのかと、
reduxの場合のinputの参照方法を詳しく教えていただきたいです。

ちなみに

class InputArea extends React.Component{ render(){ let inputs; const { onSend } = this.props; return( <div> <input type="text" placeholder="input your text !!" ref={(node) => { inputs = node }} /> <button onClick={this.props.onSend(inputs.value)}>送信</button> </div> ); } }

このような参照方法だと、

Uncaught TypeError: Cannot read property 'value' of undefined

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

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

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

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

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

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

guest

回答1

0

ベストアンサー

onChange={this.state._onChanges}

onChange={ e => this._onChanges(e) }
ぢゃないでしょうか?

投稿2017/01/04 05:28

k.tada

総合スコア1679

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

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

k.tada

2017/01/04 05:33

それと <button onClick={this.props.onSend(this.state.inputText).bind(this)}>送信</button> の`.bind(this)`は不要な気がします。
yuki_nagahama

2017/01/04 05:36

ご回答いただきありがとうございます。 そちらのonChangeイベントの方は教えていただいたコードでうまくいきました。 ありがとうございました。 ただ、 buttonのイベントonClick=『this.props.onSend(this.state.inputText).bind(this)』 の方で無限ループをしてしまうようです...
k.tada

2017/01/04 05:38

あーそうか、失礼しました。`bind`が不要なのじゃなくて <button onClick={this.props.onSend.bind(this, this.state.inputText)}>送信</button> でどうでしょう?
yuki_nagahama

2017/01/04 07:06

なるほどです。こちらのコードでできました。 ありがとうございました。 ちなみにもう一点質問なのですが、 通常はbind(this)なのですが、 bindの第一引数と第二引数はどういった意味なのでしょうか。
k.tada

2017/01/04 07:27

解決したようでよかったです。 bindの引数については、以下のとおりです。 * 第一引数:対象メソッドの実行コンテキストになる(対象メソッドのthisにセットされる) * 第二引数以降の引数:対象のメソッドに渡される引数となる
guest

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

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

ただいまの回答率
85.48%

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

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

質問する

関連した質問