原生js实现菜单折叠的效果，一个bug不知道 如何解决

Question

我的目的是点击商品管理或文章管理，折叠或者关闭。但是现在有个bug当我点击选项里的子选项（比如文章分类或文章列表），照样会折叠。贴上代码帮忙分析下

<ul>
<li class="explode" key="" name="menu" onclick = "toggleMenu(this)">
    文章管理        <ul style="display: none;">
          <li class="menu-item"><a href="javascript:void(0);" onmousedown="showConent('articlecat.php')">文章分类</a></li>
          <li class="menu-item"><a href="javascript:void(0);" onmousedown="showConent('article.php')">文章列表</a></li>    
        </ul>
      </li>

</ul>
<script>
function toggleMenu(e) {
  // console.log(e.childNodes[1]);
  e.childNodes[1].style.display = e.childNodes[1].style.display=='block'?'none':'block';

}

</script>

Answer 1

因为点击事件冒泡了，你可以看下这篇关于 js 冒泡的描述 JS 中的事件冒泡与捕获

你点击的虽然是子元素，但是冒泡到父亲身上，看下面这个图：

文章管理的范围要大一些，我对你代码稍微修改了一下：

<li class="explode" key="" name="menu" onclick = "toggleMenu()">
    文章管理        <ul style="display: none;">
          <li class="menu-item"><a href="javascript:void(0);" onmousedown="showConent('articlecat.php')">文章分类</a></li>
          <li class="menu-item"><a href="javascript:void(0);" onmousedown="showConent('article.php')">文章列表</a></li>    
        </ul>
      </li>

</ul>
<script>
function toggleMenu(e) {
  // console.log(e.childNodes[1]);
  e = e || window.event; //获得点击事件
  if(e.target.className == 'explode'){ //判断点击事件是否是想要的元素
    e.target.childNodes[1].style.display = e.target.childNodes[1].style.display=='block'?'none':'block';
  }
  e.stopPropagation(); //停止冒泡

}
</script>

我把 toggleMenu 传入的 this 给删了，然后获取点击事件 e，判断点击事件的 class 是不是我们想要的。

这个时候点击子元素，因为 className 不等于 explode 并不会执行，e.stopPropagation() 会让停止冒泡。

希望可以帮到你。