1. ホーム
  2. javascript

[解決済み] 別のコンポーネントをレンダリング中にコンポーネントを更新できない警告

2022-03-07 13:57:07

質問

reactでこのような警告が表示されます。

index.js:1 Warning: Cannot update a component (`ConnectFunction`) 
while rendering a different component (`Register`). To locate the 
bad setState() call inside `Register` 

スタックトレースで示された場所に行き、すべてのsetstatesを削除しましたが、警告はまだ続いています。reduxのディスパッチから発生する可能性はあるのでしょうか?

私のコード

レジスタ.js

class Register extends Component {
  render() {
    if( this.props.registerStatus === SUCCESS) { 
      // Reset register status to allow return to register page
      this.props.dispatch( resetRegisterStatus())  # THIS IS THE LINE THAT CAUSES THE ERROR ACCORDING TO THE STACK TRACE
      return <Redirect push to = {HOME}/>
    }
    return (
      <div style = {{paddingTop: "180px", background: 'radial-gradient(circle, rgba(106,103,103,1) 0%, rgba(36,36,36,1) 100%)', height: "100vh"}}>
        <RegistrationForm/>
      </div>
    );
  }
}

function mapStateToProps( state ) {
  return {
    registerStatus: state.userReducer.registerStatus
  }
}

export default connect ( mapStateToProps ) ( Register );

register.js から呼び出される registerForm コンポーネントの警告をトリガーする関数です。

handleSubmit = async () => {
    if( this.isValidForm() ) { 
      const details = {
        "username": this.state.username,
        "password": this.state.password,
        "email": this.state.email,
        "clearance": this.state.clearance
      }
      await this.props.dispatch( register(details) )
      if( this.props.registerStatus !== SUCCESS && this.mounted ) {
        this.setState( {errorMsg: this.props.registerError})
        this.handleShowError()
      }
    }
    else {
      if( this.mounted ) {
        this.setState( {errorMsg: "Error - registration credentials are invalid!"} )
        this.handleShowError()
      }
    }
  }

スタックトレース

解決方法は?

私はこの問題を、register components render メソッドから componentwillunmount メソッドへのディスパッチを削除することで解決しました。これは、ログインページにリダイレクトする直前にこのロジックを発生させたかったからです。一般に、すべてのロジックを render メソッドの外に置くのがベストプラクティスなので、私のコードは以前の書き方がまずかっただけです。これが将来的に誰かの役に立つことを願っています :)

私がリファクタリングしたレジスターコンポーネントです。

class Register extends Component {

  componentWillUnmount() {
    // Reset register status to allow return to register page
    if ( this.props.registerStatus !== "" ) this.props.dispatch( resetRegisterStatus() )
  }

  render() {
    if( this.props.registerStatus === SUCCESS ) { 
      return <Redirect push to = {LOGIN}/>
    }
    return (
      <div style = {{paddingTop: "180px", background: 'radial-gradient(circle, rgba(106,103,103,1) 0%, rgba(36,36,36,1) 100%)', height: "100vh"}}>
        <RegistrationForm/>
      </div>
    );
  }
}