ReactチェックボックスがonChangeを送信しない


136

TLDR:チェックされた動作中のjsbinの代わりにdefaultCheckedを使用します。

チェックされたときにラベルのテキストを取り消す単純なチェックボックスを設定しようとしています。何らかの理由で、コンポーネントを使用してもhandleChangeが起動されません。誰かが私が間違っていることを説明できますか?

var CrossoutCheckbox = React.createClass({
  getInitialState: function () {
    return {
        complete: (!!this.props.complete) || false
      };
  },
  handleChange: function(){
    console.log('handleChange', this.refs.complete.checked); // Never gets logged
    this.setState({
      complete: this.refs.complete.checked
    });
  },
  render: function(){
    var labelStyle={
      'text-decoration': this.state.complete?'line-through':''
    };
    return (
      <span>
        <label style={labelStyle}>
          <input
            type="checkbox"
            checked={this.state.complete}
            ref="complete"
            onChange={this.handleChange}
          />
          {this.props.text}
        </label>
      </span>
    );
  }
});

使用法:

React.renderComponent(CrossoutCheckbox({text: "Text Text", complete: false}), mountNode);

解決:

checkedを使用しても、基になる値は(見かけ上)変更されないため、onChangeハンドラーは呼び出されません。defaultCheckedに切り替えるとこれが修正されるようです:

var CrossoutCheckbox = React.createClass({
  getInitialState: function () {
    return {
        complete: (!!this.props.complete) || false
      };
  },
  handleChange: function(){
    this.setState({
      complete: !this.state.complete
    });
  },
  render: function(){
    var labelStyle={
      'text-decoration': this.state.complete?'line-through':''
    };
    return (
      <span>
        <label style={labelStyle}>
          <input
            type="checkbox"
            defaultChecked={this.state.complete}
            ref="complete"
            onChange={this.handleChange}
          />
          {this.props.text}
        </label>
      </span>
    );
  }
});

3
まずthis.setState({checked: !this.state.checked})、値を保存するよりも簡単なonChangeを追加してみませんか 。次に、チェック済みの属性の三項演算子:checked={this.state.checked ? 'checked': null}
zackify 2014年

それが最初の方法ですが、更新されたようには見えませんでした。だから、私はそれをあちこちに吹き飛ばし始めて、発砲されなかったものをデバッグしました。理想的には、完了すると最も単純な形式に戻ります:)
jdarling

mountNodeが実際のdomノードであると想定すると、を使用する必要がありますthis.refs.complete.getDOMNode().checked。fiddle jsfiddle.net/d10xyqu1を
trekforever

彼はdomノードを取得する代わりに単に状態を使用することができます:jsfiddle.net/d10xyqu1/1それはうまくいきます、あなたは何かを間違ってタイプしたに違いありません。
zackify 2014年

2
TLDRコメントを無視-defaultCheckedが常に答えとは限らない
Chris

回答:


207

チェックボックスのチェック状態を取得するには、パスは次のようになります。

this.refs.complete.state.checked

別の方法は、handleChangeメソッドに渡されたイベントから取得することです。

event.target.checked

3
handleChangeは、handleChangeは:(呼び出されませんが、チェックボックスやラベルをクリックした場合は関係ありません、と呼ばれていないばかりん。
jdarling

13
入力で「チェック済み」の代わりにdefaultChecked = {this.state.complete}を使用してみてください。
zbyte 2014年

それだけでした...探し回って、ずっと探し続けました。他の人もこれに遭遇した場合に備えて、完全に機能する回答で質問を更新します。
jdarling 2014年

しかし、なぜ-同じ問題がありますがchecked、制御されたコンポーネントに使用することになっています:/
Dominic

4
設定とcheckedは、状態がコンポーネントの外部で管理されることを意味します。ユーザーがクリックするとhandleChange、状態が更新されないため、呼び出すものはありません。代わりにonClick、そこで状態の更新をリッスンしてトリガーする必要があります。
zbyte

29

そのような場合はrefを使用しない方が良いでしょう。使用する:

<input
    type="checkbox"
    checked={this.state.active}
    onClick={this.handleClick}
/>

いくつかのオプションがあります:

checkeddefaultChecked

前者は状態の変化クリックの両方に応答します。後者は状態変化を無視します。

onClickonChange

前者は常にクリックでトリガーされます。要素にchecked属性が存在する場合、後者はクリックでトリガーされませんinput


10

入力DOMでonChangeハンドラーを使用したくないシナリオではonClick、代わりにプロパティーを使用できます。defaultChecked、条件は、V16のIINM固定状態のままにしてもよいです。

 class CrossOutCheckbox extends Component {
      constructor(init){
          super(init);
          this.handleChange = this.handleChange.bind(this);
      }
      handleChange({target}){
          if (target.checked){
             target.removeAttribute('checked');
             target.parentNode.style.textDecoration = "";
          } else {
             target.setAttribute('checked', true);
             target.parentNode.style.textDecoration = "line-through";
          }
      }
      render(){
         return (
            <span>
              <label style={{textDecoration: this.props.complete?"line-through":""}}>
                 <input type="checkbox"
                        onClick={this.handleChange}
                        defaultChecked={this.props.complete}
                  />
              </label>
                {this.props.text}
            </span>
        )
    }
 }

これが将来誰かの役に立つことを願っています。


10

handleChange次のような関数がある場合:

handleChange = (e) => {
  this.setState({
    [e.target.name]: e.target.value,
  });
}

onChangeテキスト入力のように動作するようにカスタム関数を作成できます。

<input
  type="checkbox"
  name="check"
  checked={this.state.check}
  onChange={(e) => {
    this.handleChange({
      target: {
        name: e.target.name,
        value: e.target.checked,
      },
    });
  }}
/>

handleChangeオンにinputすべきではありませんthis.handleChangeか?
Ardhi、

5

誰かがユニバ​​ーサルイベントハンドラーを探している場合、次のコードを多かれ少なかれ使用できます(nameプロパティがすべての入力に設定されていると仮定)。

    this.handleInputChange = (e) => {
        item[e.target.name] = e.target.type === "checkbox" ? e.target.checked : e.target.value;
    }

2

onChangeは、defaultCheckedを使用する場合、モバイルでhandleChangeを呼び出しません。別の方法として、onClickとonTouchEndを使用できます。

<input onClick={this.handleChange} onTouchEnd={this.handleChange} type="checkbox" defaultChecked={!!this.state.complete} />;

弊社のサイトを使用することにより、あなたは弊社のクッキーポリシーおよびプライバシーポリシーを読み、理解したものとみなされます。
Licensed under cc by-sa 3.0 with attribution required.