【问题标题】:Rerender functional component when prop changesprop 更改时重新渲染功能组件
【发布时间】:2017-02-11 11:35:13
【问题描述】:

当传递给子组件的父状态发生更改时,我无法重新渲染组件。 MapNav 组件包含触发方法更改当前显示的弹出窗口的按钮。我的问题是当有人点击 zalogujwyloguj 时重新渲染 UserPopup 按钮,它应该可以工作,但它没有,我也无法弄清楚出了什么问题想要在 isLoggedIn 状态改变并且另一件事 logIn 方法正确改变状态时触发动画。 isLoggedIn 状态只是暂时的,稍后我会将其更改为使用存储。我正在尝试编写尽可能多的功能组件,但在某些情况下我无法避免使用状态,我敢打赌我的代码中存在问题,所以我会感谢所有改进提示。

const MapNav = (props) => {
  return       <div className={'mapNavContainer'}>
        <ul className={'navList'}>
          <li><a href="#about"><i className={"glyphicon glyphicon-fullscreen"} /></a></li>
          <li><a href="#about"><i className={"glyphicon glyphicon-chevron-left"} /></a></li>
          <li><a href="#about"><i className={"glyphicon glyphicon-chevron-right"} /></a></li>
          <li><a href="#about"><i className={"glyphicon glyphicon-map-marker"} /></a></li>
          <li><a href="#" onClick={props.onUserIconClick}><i className={"glyphicon glyphicon-user"} /></a></li>
          <li><a href="#" onClick={props.onInfoClick}><i className={"glyphicon glyphicon-info-sign"} /></a></li>
        </ul>
      </div>
}

const UserPopup = (props) => {
  return <div>
    {(props.isLoggedIn) ? (
      <div className={'navPopups col-sm-3 panel panel-default'}>
        <div className={"panel-heading"}>Zalogowany</div>
        <div className={"panel-body"}>
          <button type='submit' onClick={props.logIn}>wyloguj</button>
        </div>
      </div>
      ) : (
        <div className={'navPopups col-sm-3 panel panel-default'}>
          <div className={"panel-heading"}>Niezalogowany</div>
          <div className={"panel-body"}>
            <button type='submit' onClick={props.logIn}>Zaloguj</button>
          </div>
        </div>
      )
    }
  </div>;
};

function Welcome(props) {
  return <div className={'navPopups col-sm-3 panel panel-default'}>
    <div className={"panel-heading"}>Informacje o Autorze.</div>
    <div className={"panel-body"}>
        Basic panel example
    </div>
  </div>;
}

class MapHeader extends React.Component {
  constructor(props) {
    super(props);
    this.state = {
      currentPopup: null,
      isLoggedIn: false,
      username: 'gregorowicz.k@gmail.com',
    };

    this.onInfoClick = this.onInfoClick.bind(this);
    this.onUserIconClick = this.onUserIconClick.bind(this);
    this.logIn = this.logIn.bind(this);
  }

  onInfoClick = (e) => {
    e.preventDefault();
    this.setPopup(<Welcome />);
  }

  onUserIconClick = (e) => {
    e.preventDefault();
    const loginPopup = <UserPopup isLoggedIn={this.state.isLoggedIn} username={this.state.username} logIn={this.logIn} />;
    this.setPopup(loginPopup);
  }

  setPopup = (obj) => {
    this.state.currentPopup === null ? this.setState({ currentPopup: obj }) : this.setState({ currentPopup: null });
  }

  logIn = (e) => {
    this.setState({ isLoggedIn: !this.state.isLoggedIn });
  }

  render() {
    return (
      <nav className={'header'}>
        <div className={'container-fluid'}>
          <div className={"navbar-header"}>
            <button type="button" className={"navbar-toggle collapsed"} data-toggle="collapse" data-target="#navbar" aria-expanded="false" aria-controls="navbar">
              <span className={"sr-only"}>Toggle navigation</span>
              <span className={"icon-bar"} />
              <span className={"icon-bar"} />
              <span className={"icon-bar"} />
            </button>
          </div>

          <div id={"navbar"} className={'navContainer'}>
            {MapNav({onInfoClick:this.onInfoClick, onUserIconClick:this.onUserIconClick})}
    {this.state.currentPopup ? this.state.currentPopup : null}
          </div>
        </div>
      </nav>
    )
  }
}

// Render it
ReactDOM.render(
  <MapHeader />,
  document.getElementById("react")
);
<div id="react"></div>
<script src="https://cdnjs.cloudflare.com/ajax/libs/react/15.1.0/react.min.js"></script>
<script src="https://cdnjs.cloudflare.com/ajax/libs/react/15.1.0/react-dom.min.js"></script>
<link rel="stylesheet" href="https://maxcdn.bootstrapcdn.com/bootstrap/latest/css/bootstrap.min.css" />

【问题讨论】:

  • 旁注:className={"panel-heading"} 可以更简单地写成className="panel-heading"。当值是一个简单的字符串时,不需要{}
  • 感谢您的提示,我也添加了可运行文件
  • 旁注 2:您的 logIn 等是关闭 this 的箭头函数。对他们调用bind 不会改变他们的this(但它已经正确,所以你不需要bind)。
  • 要使问题发生,请单击用户图标(从底部开始的第二个图标),然后在 isLoggedIn 状态更改后单击 zaloguj,但它不会重新呈现 UserPopup,因为它应该根据传递的 isLoggedIn 显示不同的按钮和面板标题道具。
  • zaloguj 按钮被点击时,父组件 MapHeader 中的状态被正确更改,所以要完全理解我的问题是点击用户图标 > 点击 zaloguj 然后单击用户图标两次。正如您将看到的,显示了不同的弹出窗口,因此更改了 isLoggedIn,但它不会立即触发重新渲染,必须销毁并重新创建 UserPopup 才能看到结果。

标签: javascript reactjs functional-programming


【解决方案1】:

我已经解决了。我的代码中有 currentPopup 状态,它存储了当前显示的弹出窗口,要查看更改,我必须在 isLogggedIn 状态更改后再次卸载和安装组件,所以我改变了显示当前弹出窗口的方式。我只存储提供有关当前组件信息的简单字符串值,而不是将整个组件存储在状态中。无论如何,谢谢你的提示,我一定会在我的代码中应用它们。

const MapNav = (props) => {
  return       <div className={'mapNavContainer'}>
        <ul className={'navList'}>
          <li><a href="#about"><i className={"glyphicon glyphicon-fullscreen"} /></a></li>
          <li><a href="#about"><i className={"glyphicon glyphicon-chevron-left"} /></a></li>
          <li><a href="#about"><i className={"glyphicon glyphicon-chevron-right"} /></a></li>
          <li><a href="#about"><i className={"glyphicon glyphicon-map-marker"} /></a></li>
          <li><a href="#" onClick={props.onUserIconClick}><i className={"glyphicon glyphicon-user"} /></a></li>
          <li><a href="#" onClick={props.onInfoClick}><i className={"glyphicon glyphicon-info-sign"} /></a></li>
        </ul>
      </div>
}

const UserPopup = (props) => {
  return <div>
    {(props.isLoggedIn) ? (
      <div className={'navPopups col-sm-3 panel panel-default'}>
        <div className={"panel-heading"}>Zalogowany</div>
        <div className={"panel-body"}>
          <button type='submit' onClick={props.logIn}>wyloguj</button>
        </div>
      </div>
      ) : (
        <div className={'navPopups col-sm-3 panel panel-default'}>
          <div className={"panel-heading"}>Niezalogowany</div>
          <div className={"panel-body"}>
            <button type='submit' onClick={props.logIn}>Zaloguj</button>
          </div>
        </div>
      )
    }
  </div>;
};

function Welcome(props) {
  return <div className={'navPopups col-sm-3 panel panel-default'}>
    <div className={"panel-heading"}>Informacje o Autorze.</div>
    <div className={"panel-body"}>
        Basic panel example
    </div>
  </div>;
}

class MapHeader extends React.Component {
  constructor(props) {
    super(props);
    this.state = {
      currentPopup: null,
      isLoggedIn: null,
    };

    this.onInfoClick = this.onInfoClick.bind(this);
    this.onUserIconClick = this.onUserIconClick.bind(this);
    this.logIn = this.logIn.bind(this);
  }
                  
  onInfoClick = (e) => {
    e.preventDefault();
    this.setState({ currentPopup: this.state.currentPopup === 'info' ? null : 'info' });
  }
    
  onUserIconClick = (e) => {
    e.preventDefault();
    this.setState({ currentPopup: this.state.currentPopup === 'user_login' ? null : 'user_login' });
  }

  setPopup = (obj) => {
    this.setState({ currentPopup: this.state.currentPopup === 'user_login' ? null : 'user_login' });
  }

  logIn = (e) => {
    this.setState({ isLoggedIn: !this.state.isLoggedIn });
  }

  render() {
    return (
      <nav className={'header'}>
        <div className={'container-fluid'}>
          <div className={"navbar-header"}>
            <button type="button" className={"navbar-toggle collapsed"} data-toggle="collapse" data-target="#navbar" aria-expanded="false" aria-controls="navbar">
              <span className={"sr-only"}>Toggle navigation</span>
              <span className={"icon-bar"} />
              <span className={"icon-bar"} />
              <span className={"icon-bar"} />
            </button>
          </div>

          <div id={"navbar"} className={'navContainer'}>
            {MapNav({onInfoClick:this.onInfoClick, onUserIconClick:this.onUserIconClick})}

              {this.state.currentPopup === 'info' ? <Welcome /> : null}
              {this.state.currentPopup === 'user_login' ? (
                <UserPopup isLoggedIn={this.state.isLoggedIn} logIn={this.logIn} />
              ) : null}

          </div>
        </div>
      </nav>
    )
  }
}

// Render it
ReactDOM.render(
  <MapHeader />,
  document.getElementById("react")
);
<div id="react"></div>
<script src="https://cdnjs.cloudflare.com/ajax/libs/react/15.1.0/react.min.js"></script>
<script src="https://cdnjs.cloudflare.com/ajax/libs/react/15.1.0/react-dom.min.js"></script>
<link rel="stylesheet" href="https://maxcdn.bootstrapcdn.com/bootstrap/latest/css/bootstrap.min.css" />

【讨论】: