Question

反应版本：16.8.2

const { useState } = React;

const Dropdown = ({
  options,
  onClick,
  selected,
}) => {
  const [showDropDown, setShowDropDown] = useState(false);

  function toggleDropDown() {
    setShowDropDown(v => !v);
  }

  function createOption(o) {
    return (
      <div
        key={o}
        className='drop-down__option'
        onClick={() => {
          onClick(o);
          toggleDropDown();  
          // The following isn't working either
          // setShowDropDown(false);
        }}
       >
        {o}
      </div>
    );
  }

  return (
    <div
      onClick={toggleDropDown}
      className='drop-down'
    >
      <div>{selected}</div>
        {showDropDown && (
          <div className='drop-down__list'>
            {options.map(createOption)}
          </div>
        )}
    </div>
  );
};

const App = () => {
  const [selected, setSelected] = useState(0);

  return (
    <Dropdown
      selected={selected}
      options={[0,1,2,3]}
      onClick={v=>{setSelected(v)}}
    />
  );
}

ReactDOM.render(
  <App />,
  document.getElementById('root'),
);

这是一个简单的Dropdown，当您单击容器时，它会切换Dropdown。

但是这里的问题是，单击选项之一后，选项已更新，但是Dropdown尚未关闭。

因此，以下代码中的toggleDropDown();无效。

        onClick={() => {
          onClick(o);
          toggleDropDown();  
          // The following isn't working either
          // setShowDropDown(false);
        }}

可以在此处找到复制示例：https://codepen.io/vensa-albertgao/pen/ommGZe

我想念什么？

Answer 1

将toggleDropDown函数更改为

function toggleDropDown() {
    setShowDropDown(!showDropDown);
}

分叉了例子

https://codepen.io/anon/pen/XOOzaZ

Answer 2

您的代码中的问题是，即使在click选项上，dropdown也被传播到了您再次调用toggleDropdown的父onClick上，因此触发发生了两次，导致它不会明显发生。您所需要做的就是停止对选项的传播，点击

const { useState } = React;

const Dropdown = ({
  options,
  onClick,
  selected,
}) => {
  const [showDropDown, setShowDropDown] = useState(false);

  function toggleDropDown() {
    console.log('called');
    setShowDropDown(v => { console.log(v); return !v});
  }

  function createOption(option) {
    return (
      <div
        key={option}
        className='drop-down__option'
        onClick={e => {
          e.stopPropagation();
          onClick(option);
          toggleDropDown();  
        }}
       >
        {option}
      </div>
    );
  }

  return (
    <div
      onClick={toggleDropDown}
      className='drop-down'
    >
      <div>{selected}</div>
        {showDropDown && (
          <div className='drop-down__list'>
            {options.map(createOption)}
          </div>
        )}
    </div>
  );
};

const App = () => {
  const [selected, setSelected] = useState(0);

  return (
    <Dropdown
      selected={selected}
      options={[0,1,2,3]}
      onClick={v=>{setSelected(v)}}
    />
  );
}

ReactDOM.render(
  <App />,
  document.getElementById('root'),
);

Working demo

反应挂钩无法从事件处理程序中正常工作

2 个答案: