2017-02-11 231 views
1

當父狀態傳遞給子組件更改時,我無法重新渲染組件。 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: '[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

邊注:'的className = { 「面板標題」}'可以更簡單地被寫入'的className = 「面板標題」'。當值是一個簡單的字符串時,不需要'{}'。 –

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