javascript - 为什么我的 for 循环改变了 jQuery 中 toggleClass 的行为

Question

我为我的网站模板创建了一个文档，我正在尝试创建一个下拉菜单。这个想法很简单:当我点击链接名称旁边的插入符号时，它的子菜单应该打开，如果我再次点击，它应该关闭。此外，如果任何其他子菜单打开，我希望它们关闭，以便一次只打开一个子菜单。

我的解决方案:我使用了 toggleClass 和 for 循环。

我的问题是: 在我添加了两个 for 循环以允许一次只打开一个子菜单后，我的 toggleClass 停止工作。这意味着当我单击插入符号时，它会打开子菜单，但再次单击插入符号时，它不会关闭子菜单并保持打开状态。

代码如下:

jQuery(document).ready(function(){

  $(".indicator").on("click", showLinks);

    function showLinks(){

      var allMenus = $("#ab-menu, #str-menu, #ht-menu");
      var allCarets = $("#ab-caret, #str-caret, #ht-caret");

      for(i = 0; i < allMenus.length; i++){
        $(allMenus).eq(i).removeClass("show");
      }
      for(i = 0; i < allCarets.length; i++){
        $(allCarets).eq(i).removeClass("caret-rotate");
      }

      var linkId = $(this).attr("data-menu");
      var caretId = $(this).attr("data-caret");

      $(linkId).toggleClass("show");
      $(caretId).toggleClass("caret-rotate");

    };

});

#ab-menu,
#str-menu,
#ht-menu {
  background-color: #1d3557;
  max-height: 0;
  margin: 0.2em 0;
  opacity: 0.5;
  overflow: hidden;
  transition: 0.4s ease-in-out;
}
#ab-menu.show,
#str-menu.show,
#ht-menu.show {
  max-height: 500px;
  opacity: 1;
}
.fa .fa-caret-down {
  display: block;
  color: #fff;
}
.caret-rotate {
  transition: 0.4s ease-in-out;
}
.fa-caret-down.caret-rotate {
  color: #a8dadc;
  transform: rotate(-90deg);
}

<nav>
    <ul class="nav-links">
      <li class="about-panel">About <span class="fa fa-caret-down indicator" id="ab-caret" data-menu="#ab-menu" data-caret="#ab-caret"></span>
        <ul class="sub-menu about-sub-menu" id="ab-menu">
          <li class="about-links"><a href="#about">Main Features</a></li>
          <li class="about-links"><a href="#getting-started">Getting Started</a></li>
        </ul>
      </li>
    </ul>
    <ul class="nav-links">
      <li class="structure-panel">Structure <span class="fa fa-caret-down indicator" id="str-caret" data-menu="#str-menu" data-caret="#str-caret"></span>
        <ul class="sub-menu structure-sub-menu" id="str-menu">
          <li class="structure-links"><a href="#sections">Sections</a></li>
          <li class="structure-links"><a href="#grid">Bootstrap-grid</a></li>
          <li class="structure-links"><a href="#carousel">Bootstrap Carousel</a></li>
        </ul>
      </li>
    </ul>
    <ul class="nav-links">
      <li class="how-to-panel">How-to <span class="fa fa-caret-down indicator" id="ht-caret" data-menu="#ht-menu" data-caret="#ht-caret"></span>
        <ul class="sub-menu how-sub-menu" id="ht-menu">
          <li class="how-to-links"><a href="#nav">Navigation</a></li>
          <li class="how-to-links"><a href="#images">Images</a></li>
          <li class="how-to-links"><a href="#headers">Headers</a></li>
          <li class="how-to-links"><a href="#icons">Icons</a></li>
          <li class="how-to-links"><a href="#colors">Colors</a></li>
          <li class="how-to-links"><a href="#fonts">Fonts</a></li>
          <li class="how-to-links"><a href="#buttons">Buttons</a></li>
          <li class="how-to-links"><a href="#portfolio">Portfolio</a></li>
          <li class="how-to-links"><a href="#map">Map</a></li>
        </ul>
      </li>
    </ul>
    <ul class="nav-links">
      <li class="credits"><a href="#credits">Credits</a></li>
    </ul>
  </nav>

Answer 1

不需要循环。您可以像这样简化代码，我想这会解决您的问题。

var allMenus = $("#ab-menu, #str-menu, #ht-menu");
var allCarets = $("#ab-caret, #str-caret, #ht-caret");

allMenus.removeClass("show");
allCarets.removeClass("caret-rotate");