Ищете более элегантный способ написания нескольких компонентов с использованием похожих функций

#javascript #reactjs #function #material-ui #higher-order-components

#javascript #reactjs #функция #материал-пользовательский интерфейс #компоненты более высокого порядка

Вопрос:

У меня есть этот компонент, который отображает данные, но также должен сохранять и управлять состоянием. У меня есть 5 других компонентов, которые точно такие же, за исключением небольших нюансов внутри их функций. Мне нужна помощь в поиске более элегантного способа написания этого кода, вместо того, чтобы иметь 6 файлов, которые более или менее одинаковы.

Я пытался понять и использовать HOC для этого экземпляра, но у каждого компонента есть свой собственный вызов отправки, который требует данных, специфичных для этого компонента. Итак, я не могу найти способ сделать специальную работу.

Это мои функции:

   componentDidMount () {
    setTimeout(() => {
      this.setState({ baseForm: this.props.baseData })
      this.getDisable()
      this.setState({ loading: false })
    }, 2000)
  }

  handleSwitch = item => event => {
    this.setState({ [item]: event.target.checked })
  }

  handlePRESubmit () {
    const array = this.state.baseForm
    array.forEach(item => {
      item.isTemplate = false
      item.mark = this.state.mark
      item.isVisible = this.state.visible
      item.genesisId = item._id
    })
  }

  handleSubmit = () => {
    this.handlePRESubmit()
    let formData = this.state.baseForm[0]
    fetch(APIURL, {
      method: 'POST',
      body: JSON.stringify(formData),
    }).then(response => {
      response.json().then(data => {
        let orgId = localStorage.getItem('orgId')
        let sku = { skuId: data.data._id, type: 'verification' }
        fetch(APIURL, {})
          .then(response => response.json())
          .then(data => {})
      })
    })
  }

  toggleDisabled () {
    if (this.state.assigned !== undefined) {
      this.setState({ disabled: !this.state.disabled })
    }
  }

  getDisable () {
    setTimeout(() => {
      const result = this.props.assignedSku.find(e => e.name === 'Base')
      this.setState({ assigned: result })
      if (this.state.assigned !== undefined) {
        this.setState({ mark: true })
        this.setState({ visible: true })
      }
    }, 1000)
  }

  handleMenu = event => {
    this.setState({ anchorEl: event.currentTarget })
  }

  handleClose = () => {
    this.setState({ anchorEl: null })
  }
  

И это моя визитная карточка

         <Card id='partner-sku-card'>
          <CardHeader
            title={base.name}
            subheader={'$'   ' '   base.price}
            action={
              <div>
                <IconButton onClick={this.handleMenu}/>
              </div>
            }
          />
          <Menu
            anchorEl={anchorEl}
            open={Boolean(anchorEl)}
            onClose={this.handleClose}
          >
            <MenuItem disabled={this.state.disabled ? 'disabled' : ''}>
              Edit
            </MenuItem>
          </Menu>
          <CardActions>
            <FormControlLabel
              control={
                <Switch
                  checked={this.state.mark}
                  onChange={this.handleSwitch('mark')}
                  value='mark'
                />
              }
            />
            <FormControlLabel
              control={
                <Switch
                  checked={this.state.visible}
                  onChange={this.handleSwitch('visible')}
                  value='visible'
                />
              }
            />
            <Button onClick={this.handleSubmit}>
              Submit
            </Button>
          </CardActions>
        </Card>
  

Опять же, весь этот код снова записывается в 5 других файлах. Мне нужен элегантный способ заменить слово «Base» / «base» в каждом аспекте этого. Допустим, у меня есть Base, Mid, Top. Мне нужно, чтобы все эти функции работали для всех 3, и в конце все равно получалась одна и та же карта.
Спасибо за помощь!

Ответ №1:

Создайте компонент с именем CardWrapper или что-то в этом роде. Затем в каждом другом вызове файла CardWrapper следующим образом:

 class First extends Component {
    handleSwitch = () => {
        //its own implementation
    }

    handlePreSubmit = () => {
        //its own implementation
    }

    handleSubmit = () => {
        //its own implementation
    }
    //other methods that you need for each component to be different

    render(){
        return (
            <CardWrapper handlePreSubmit={this.handlePreSubmit} handleSubmit={this.handleSubmit} handleSwitch={this.handleSwitch} />
        )
    }
}
  

Помните, что вы должны добавить все это во все файлы, которые совместно CardWrapper используют компонент.

А затем CardWrapper вы можете получить к нему доступ с помощью this.props . Ex. в конце, когда у вас Submit Button это изменится, например:

 <Button onClick={this.props.handleSubmit}>
    Submit
</Button>
  

Комментарии:

1. о, хорошо, значит, у каждого компонента будет своя собственная логика, но затем используйте CardWrapper в качестве «шаблона» для визуальной карты?

2. @PhilipKappaz да. Вы можете поделиться столько вещей, сколько хотите внутри CardWrapper, но те методы, которые должны быть разными для каждого компонента, тогда они должны быть в их собственных компонентах

3. Как насчет изменения состояния? Например, <Switch/> в CardWrapper вызывает {this.state.disabled ? ‘diabled’: «} это просто изменится на this.props.disabled ?

4. @PhilipKappaz нет, вам не нужно передавать его через props, поскольку вы уже находитесь в CardWrapper, вы просто получаете к нему доступ как есть this.state.disabled .