4

我有一个 React 组件，它根据状态呈现<ul>和插入元素。<li>

class SimpleComponent extends React.Component {
  constructor(props) {
    this.state = { menu_items: [name: "First", id: 10] }
    this.clickMenu = this.clickMenu.bind(this)
  }
  generateLinks() {
    let to_return='';
    for (var i=0;i<this.state.menu_items.length; i++) {
      to_return = to_return + `<li><a onclick={clickMenu}> ${this.state.menu_item[i]['name']} </a></li>`
     }
     return to_return;
   }
   clickMenu(e) {
     console.log(e.target)
   }
   render() {
     return(
       <ul dangerouslySetInnerHTML={this.generateLinks()}></ul>
     )
   }
}

当我单击 Anchor 时，控制台显示 Uncaught ReferenceError: clickMenu not defined。我尝试使用 this.clickMenu 代替，但没有任何反应。我注意到渲染的锚看起来像：

<a onclick="{menuClick}">

有没有办法创建这些锚元素让 React 获取 onClick 定义，而不是将它们传递给浏览器进行解释？

4

1 回答 1

6

使用 React，您应该构建组件和子组件，而不是编写长 HTML 字符串。您基本上是在破坏 React 的使用模式，这正是您使用的属性包含“危险”一词的原因。

这是实现该列表的一种方法：

class SimpleComponent extends React.Component {
  constructor(props) {
    super(props);
    this.state = { menu_items: [{ name: "First", id: 10 }] };
    this.clickMenu = this.clickMenu.bind(this);
  }
  clickMenu(id) {
    console.log(id);
  }
  render() {
    var items = this.state.menu_items.map(item => {
      let handleClick = () => {
        this.clickMenu(item.id);
      };
      return (
        <li key={item.id}>
          <a onClick={handleClick}>
            {item.name}
          </a>
        </li>
      );
    });
    return <ul>{items}</ul>;
  }
}

我正在<li>通过映射构建一个元素数组state.menu_items。由于组件在状态更改发生时会重新渲染，因此这将始终根据当前更新state（并且仅进行实际必要的更改，这是 React 的定义特征之一）。

另请注意，您的构造函数缺少super(props)调用，并且您的状态数组的单个元素未包含在花括号中。

于 2017-12-20T00:56:34.717 回答