Javascript --如何使此React/Redux代码干燥

我有重复的代码，我不知道如何使干燥不要重复自己

这里有两个组件通过dispatch和React的自动重新渲染进行对话

此.map重复两次

此模块将在单击时分派操作

import React from 'react';
import { connect } from 'react-redux';
class Icon extends React.Component {
  constructor(props) {
    super(props);
    this.map = {
      paper: 'bg_paper.jpg',
      light_wood: 'bg_wood.jpg',
      graph: 'bg_graph.jpg'
    };
  }

  flip () {
    this.props.dispatch({type: 'updateIcon', bg_key: $A.nextKey(this.map, this.props.state.bg_key)});
  }

  render () {
    const style = {
      // ... snip
    }

    return (
      <img id = 'bar_icon' onClick={this.flip.bind(this)} style={style} src='_images/sv_favicon.svg'/>
    )

  }
}

const mapStateToProps = state => {
  return {
    state: state.Icon
  }
}

export default connect(mapStateToProps)(Icon);

而此组件将自动重新渲染。一切都很好。我只想把它弄干

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

// ... snip

class FrameBody extends React.Component {

  constructor(props) {
    super(props);
    this.map = {
      paper: 'bg_paper.jpg',
      light_wood: 'bg_wood.jpg',
      graph: 'bg_graph.jpg'
    };
  }

  render () {

    const style = {
      backgroundImage: 'url(' + '_images/' + this.map[this.props.state.bg_key] + ')'
    };

    return (

      <div id='contents' style={style}>
      </div>

    )
  }
}

const mapStateToProps = state => {
  return {
    state: state.Icon
  }
}

export default connect(mapStateToProps)(FrameBody);

---

我该怎么做才能使this.map没有两个实例？

您可以将this.map的逻辑提取到类函数中

getBackgroundImageKey = () => {
  const backgroundMap = {
    paper: 'bg_paper.jpg',
    light_wood: 'bg_wood.jpg',
    graph: 'bg_graph.jpg'
  }
  return backgroundMap[this.props.bg_key]
}

更进一步，添加另一个函数以返回URL并添加字符串插值

getBackgroundImageURL(){
  const backgroundMap = {
    paper: 'bg_paper.jpg',
    light_wood: 'bg_wood.jpg',
    graph: 'bg_graph.jpg'
  }
  return `url(_images/${backgroundMap[this.props.bg_key]})`;
}

这样就可以定义这样的样式

const backgroundImage = this.getBackgroundImageURL()
const style = { backgroundImage };

---

既然您已经在使用Redux并分派一个动作来翻转，为什么不将该逻辑移到那里呢

将当前图像保存在商店中，这样您就可以将其连接起来，使您的翻转动作创建器能够保存该地图并决定下一个图像是什么

---

您的代码没有干涸，而是缺少关注点的分离。如果只在用户单击“翻转”时调用道具，则开关/图标UI组件将更易于重用和简洁。将此onFlip连接到我提到的action creator，您在一个地方有逻辑，在另一个地方有交互的UI。

这类问题可能是一个很好的选择，但问题是如何使代码干燥。我仍然需要两个组件中的函数。DRY并不意味着一个函数只能调用一次，特别是在不同的上下文中。然而，这确实意味着，您不需要在多个地方重新定义相同的数据，而无状态的答案实现了这一点。