2017-02-11 10 views
1

子コンポーネントに親ステートが渡されたときにコンポーネントを再レンダリングするのに問題があります。 MapNavコンポーネントには、現在表示されているポップアップを変更するメソッドをトリガーするボタンが含まれています。私の問題は、誰かがzalogujまたはwyloguj、それが動作するはずですがクリックしたときに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: '[email protected]', 
 
    }; 
 

 
    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" />

+2

サイドノート: 'クラス名は{ "パネル見出しが"}'より簡単に書くことができる= 'クラス名= "パネル見出し" '。値が単純な文字列の場合、 '{}'は必要ありません。 –

+0

チップもありがとう、私はrunnableを追加しました – habibi07

+1

サイドノート2:あなたの 'logIn'などは' this'を閉じる矢印関数です。 'bind'を呼び出しても' this'は変更されません(ただし、すでに正しいので、 'bind'は必要ありません)。 –

答えて

0

私はそれを解決してきました。私のコードに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" />

関連する問題