Question

我有一个图像库，我循环通过图像对象，我想将i传递给我的onClick函数。这是我的图片库代码：

<div className="gallery clearfix">
  { block.gallery.map((item, i) => (
    i < 1 ?
     <div className="gallery-image" key={i} onClick={this.toggle}>
       <a href='' className="inner">
          <img src={item.images.thumbnail_sm} alt={block.title} srcSet={`${item.images.thumbnail_md} 1x, ${item.images.thumbnail_lg} 2x`} className="img-fluid image"/>
       </a>          
     </div>
     : null
     ))}
     <div className="gallery-thumbs">
       <div className="row">
        { block.gallery.map((item, i) => (
          i > 0 && i < (limit + 1) ?
          <div className="gallery-item" key={i} onClick={this.toggle}>
          <a href='' className="inner">
            <img src={item.images.thumbnail_sm} alt={block.title} srcSet={`${item.images.thumbnail_md} 1x, ${item.images.thumbnail_lg} 2x`} className="img-fluid image" title="" />
              { block.gallery.length > (limit + 1) && i == limit ?
                <div className="img-overlay">
                   <span className="img-indicator">{ block.gallery.length - (limit + 1) }+ <span className="hidden-xs">Foto's</span></span>
                </div>
             : null 
            }
          </a>
        </div>
        : null
        ))}
     </div>
   </div>
</div>

这是我的reactstrap模式，我想要显示点击的图像：

<Modal isOpen={this.state.modal} toggle={this.toggle} className={this.props.className}>
      <ModalBody>            
        <img src={block.gallery[this.state.clickedImage].item.images.thumbnail_lg}/>
      </ModalBody>
    </Modal>

这是切换函数，我想传递clickedImage id：

toggle(id) {
    this.setState({
      clickedImage: id,
      modal: !this.state.modal
    });
  }

Answer 1

对于最佳实践，我不建议在onClick中绑定，这会导致它在每次单击时调用绑定。如果你使用的是ES6，那么你应该在构造函数中绑定它：

Class MyComponent extends React.Component {
  constructor(props){
    super(props);
    this.toggle = this.toggle.bind(this);
  }
}

和

<div className="gallery-item" key={i} onClick={(i) => this.toggle(i)}></div>

更新：就像评论说的那样。这实际上并不是最好的方法，最好的方法是不要创建新的函数并在每个渲染中附加事件，这意味着它应该只是

<div className="gallery-item" key={i} onClick={this.toggle}></div>

但是由于你需要传递id，最好的办法是将它重构为较小的组件，如<GalleryItem>，并将id传递给props

值得一读：this

更新：另请注意评论，使用dataset.index和data-index更好

带有onClick函数的{ReactJS Pass键

1 个答案: