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

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

ただいまの
回答率

88.76%

Reactのイベントが発火しない

解決済

回答 1

投稿 編集

  • 評価
  • クリップ 0
  • VIEW 1,893

susiyaki

score 39

問題点

onClickイベントでinputのvalueを取得し、stateを更新したいが、onClickイベントが発火しない

実際のコード

export default class SeartchInput extends React.Component {
  constructor(props) {
    super(props);
    this.state = {
      textValue: 'ini'
    };
  }

  changeText(e) {
    console.log('called')
    console.log(this.state.textValue)
    this.setState({textValue: this.refs.inputText.value})
    this.refs.inputText.value = ''
  }

  render() {
    return (
      <Wrapper>
        <Flex justify="center">
          <Input  width="150px"
                  height="25px"
                  placeholder="キーワード検索..."
                  type="text"
                  ref="inputText" />
        {this.state.textValue}
          <SearchBtn  type="submit"
                      onClick={() => this.changeText()} />
        </Flex>
      </Wrapper>
    )
  }
}

試したこと

とりあえずイベントが発火しているか試すために、

onClick={() => alert('clicked')}


なども試しましたが、動いていませんでした。
また、input内でonChengeを試しましたが、そちらも発火しませんでした。

import styled, { css } from "styled-components";
import React from 'react'

import { BtnIcon } from '../Atoms/Icon.js.jsx'
import { Text } from '../Utils/Text.js'
import { Flex } from '../Utils/Flex.js'

import Color from '../const/Color.js'
import Size from '../const/Size.js'

export class SearchBtn extends React.Component {
  render () {
    return (
      <Wrapper>
        <Flex hs="100%" align="center" justify="center">
          <BtnIcon url="SEARCH" type />
        </Flex>
      </Wrapper>
    )
  }
}
...略...

const Wrapper = styled.div`
  height: 14px
  width: 30px;
  padding: 5px;
  background-color: ${ Color.MAIN };
  border: 1px solid ${ Color.MAIN_SHADOW };
  cursor: pointer;
  border-radius: 4px;
  border-bottom: 3px solid ${ Color.MAIN_SHADOW };

  &:active {
    transform: translateY(2px);
    border-bottom: 1px solid ${ Color.SUB1 };
  }
`
...略...
  • 気になる質問をクリップする

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

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

    クリップを取り消します

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

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

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

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

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

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

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

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

    質問の評価を下げる

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

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

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

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

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

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

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

    詳細な説明はこちら

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

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

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

回答 1

checkベストアンサー

+1

こんにちは

以下のように、constructor の中で this.changeText = this.changeText.bind(this); を追加してみるといかがでしょうか?

修正前:

constructor(props) {
    super(props);
    this.state = {
      textValue: 'ini'
    };
}

修正後:

constructor(props) {
    super(props);
    this.state = {
      textValue: 'ini'
    };

    this.changeText = this.changeText.bind(this);  // この行を追加
}

追記

ご質問に追記いただきました SearchBtn を拝読しましたが、SeartchInput 側で

<SearchBtn  
  type="submit"
  onClick={() => this.changeText()}
/>

として渡している2つの props 、 type と onClick が、 SearchBtn で使われていないようです。渡された onClickを使っていなければ、ボタンをクリックしても何も起きないです。

理解の確認のために、これらの props を受け取って反映させる簡単なボタン SimpleBtn を作るとすれば、例えば以下のようになるかと思います。(※state不要なコンポーネントなので、Functional Component として書くべきところですが、説明のため、Class Component で書きます)

SimpleBtn.jsx

import React from 'react'

class SimpleBtn extends React.Component {
  render() {
    const { type, onClick } = this.props;
    return (
      <button type={type} onClick={onClick}>
        検索
      </button>
    );
  }
}

export default SimpleBtn;

上記のような SimpleBtn を作って SeartchInput  の中で SearchBtn の替わりに、

<SimpleBtn  
  type="button"
  onClick={() => this.changeText()}
/>

とすれば、SimpleBtn がrender されたことによる [検索] ボタンをクリックすれば、SeartchInput  から onClick として渡した、 changeText が呼ばれると思います。

上記の SimpleBtn で onClick に設定されたイベントハンドラが呼ばれることを確認した後、SearchBtn でも同様に、 onClick を this.props から取得して、それを、しかるべきコンポーネント、(詳しくいえば、Wrapper, Flex, BtnIcon のいずれか)に渡すようにすればよいかと思います。

注意すべきことはSeartchInput から渡された onClick が、子コンポーネントに引き継がれていって、最終的には、どこかの HTML要素がクリックされたときのハンドラに設定されることになっているか、という点を点検するとよいかと思います。

投稿

編集

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2019/06/10 19:06

    ありがとうございます。
    回答のほうに追記しました。ご一読頂きさらなる不明点あればまたコメントからお知らせください。

    キャンセル

  • 2019/06/10 20:08

    ありがとうございます!
    onClickはpropsと別物だと考えていたため動くと勘違いしていました...
    propsで受け継いで、bindした関数を呼び出すという感じなのですね。

    また、stateのありなし関係なくすべてFunctional Componentを使わず書いていたので、そこも変更していこうと思います

    キャンセル

  • 2019/06/10 20:11

    どういたしまして。解決されたようで、よかったです 👏

    キャンセル

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

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

関連した質問

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