2

我有一个 react (with redux & react-router) 应用程序，它在各个方面都需要根据商店中某些数据的状态以及与商店中的数据相关的一堆其他东西来了解用户是否登录.

为了实现这一点，我创建了一个高阶组件，如下所示：

import React from 'react';
import {connect} from 'react-redux';

export function masterComponent(Component){

  class MasterComponent extends React.Component {

    constructor(props){
      super(props);
    }

    loggedIn(){
      return this.props.player.token? true : false
    }

    render(){
      return (
        <Component {...this.props} player={{loggedIn: this.loggedIn()}}/>
      );
    }
  }

  UserComponent.propTypes = {
    player: React.PropTypes.object
  };


  function mapStateToProps(state) {
    return state;
  }

  return connect(
    mapStateToProps
  )(MasterComponent);

}

我的问题是如何最好地将它包裹在我的各个组件周围？目前我已经实现了它，以便每个单独的组件在传递给 connect 时都会被包装，这工作正常但感觉有点低效。IE：

import {masterComponent} from '../HigherOrderComponents/MasterComponent';

export class ResultsPage extends React.Component {

  constructor(props){
.....
  }


  render(){
    return (
       .....
    );
  }
}



export default connect(
  mapStateToProps,
  mapDispatchToProps
)(masterComponent(ResultsPage));

有一个更好的方法吗？

4

1 回答 1

2

这真的取决于您的用例，我看到了 3 种可能的解决方案。下面的代码是未经测试和半伪的。

你最初的提议，重构

export function masterComponent(Component) {

  const MasterComponent = (props) => {
    const loggedIn = props.player.token? true : false;

    // initial solution would override player, so you need 
    // something like this
    const player = { 
      ...this.props.player,
      loggedIn,
    };

    return <Component {...this.props} player={player} />
  }

  // Important! Here is the crucial part of the refactoring, 
  // not the coding style changes. Map only the relevant 
  // state properties, otherwise the component would rerender 
  // whenever any state property changed. 
  return connect(state => ({ player: this.state.player }))(MasterComponent);
}

Redux 中间件（我个人的选择）

function loginMiddleware() {
  return store => next => action => {
    if (action.type === 'ADD_CREDENTIALS') {
      const { player } = store.getState();
      loginService.setPlayer(player);
    }

    // removePlayer if action.type === 'LOGOUT';

    return next(action);
  }
}

在您可以使用的任何组件中

loginService.getPlayer()

验证是否有人已登录

与您的方法相比的缺点'ADD_CREDENTIALS'：如果已调度操作，但请求的组件loginService.getPlayer()没有重新呈现/更新，您可能最终会出现不一致的状态。我相信你可以通过更具体的应用知识来解决这个问题。

背景（我最不喜欢的）

使用 React上下文并将所需的属性传递给所有其他组件。

于 2017-02-21T17:14:36.413 回答