jQuery 在单击操作期间选择了错误的元素

Question

我这里有以下代码示例：

$(document).ready(function() {

  $('.children a.txtlink').on("click", function(e) {
    // toggle the icons when the text link is pressed

    jQuery(this).parent().find('ul:first').toggle(); // toggle the next UL

    if (jQuery(this).parent().find('i').hasClass("fa-chevron-down")) {
      jQuery(this).parent().find('i').removeClass("fa-chevron-down").addClass("fa-chevron-up");
    } else {
      jQuery(this).parent().find('i').removeClass("fa-chevron-up").addClass("fa-chevron-down");
    }

    e.stopPropagation();
    e.preventDefault();
  });

  $('.children i.arrows').on("click", function(e) {
    // toggle the icons when the chevron is pressed

    jQuery(this).parent().find('ul:first').toggle(); // toggle the next UL

    if (jQuery(this).hasClass("fa-chevron-down")) {
      jQuery(this).removeClass("fa-chevron-down").addClass("fa-chevron-up");
    } else {
      jQuery(this).removeClass("fa-chevron-up").addClass("fa-chevron-down");
    }

    e.stopPropagation();
    e.preventDefault();
  });


});

.navbar_menu {
  background: #222222;
}

.menu {
  border-top: 1px solid #2D2D2D;
}

.menu ul {
  padding: 0;
  margin: 0;
}

.menu li {
  list-style: none;
  position: relative;
}

.menu li i {
  padding-right: 3px;
}

.menu a.txtlink {
  color: #ccc;
  display: block;
  font-size: 16px;
  padding: 10px;
  padding-right: 0;
  border-bottom: 1px solid #2D2D2D;
  text-decoration: none;
}

.menu a.txtlink:hover {
  background: #111111;
  color: #ffffff;
  text-decoration: none;
}

.menu li.home a.txtlink::before {
  content: '\f015';
  display: inline-block;
  font: normal normal normal 14px/1 FontAwesome;
  font-size: inherit;
  text-rendering: auto;
  -webkit-font-smoothing: antialiased;
  -moz-osx-font-smoothing: grayscale;
  transform: none;
}

.menu ul li {
  position: relative;
}

.menu ul li.children i.arrows {
  /* needed to show an icon in the mobile menu */
  position: absolute;
  top: 0;
  right: 0;
  padding: 14px;
  border-left: 1px solid #333333;
  margin: 0;
  cursor: pointer;
  display: block;
  color: #CCCCCC;
}

.menu .sub {
  display: none;
}

.menu ul li ul li a.txtlink {
  padding-left: 20px;
}

.menu ul li ul li ul li a.txtlink {
  padding-left: 40px;
}

.menu i.arrows:hover {
  background-color: #111111;
}

.menu ul li ul li {
  background-color: #333333;
}

<script src="https://code.jquery.com/jquery-3.2.0.min.js"></script>
<link href="https://maxcdn.bootstrapcdn.com/bootstrap/3.3.7/css/bootstrap.min.css" rel="stylesheet" />
<link href="https://maxcdn.bootstrapcdn.com/font-awesome/4.7.0/css/font-awesome.min.css" rel="stylesheet" />
<nav class="navbar_menu">

  <div class="menu">
    <ul>
      <li class="home"><a href="#" class="txtlink">Home</a></li>
      <li class="children">
        <a href="#" class="txtlink"><i class="fa fa-folder"></i>Menu 1</a>
        <ul class="sub">
          <li class="children">
            <a href="#" class="txtlink"><i class="fa fa-search"></i>Menu 1.1</a>
            <ul class="sub">
              <li><a href="#" class="txtlink"><i class="fa fa-user"></i>Menu 1.1.1</a></li>
              <li><a href="#" class="txtlink"><i class="fa fa-tag"></i>Menu 1.1.2</a></li>
            </ul>
            <i class="arrows fa fa-chevron-down"></i>
          </li>
          <li><a href="#" class="txtlink">Menu 1.2</a></li>
          <li><a href="#" class="txtlink">Menu 1.3</a></li>
          <li><a href="#" class="txtlink"><i class="fa fa-clock-o"></i>Menu 1.4</a></li>
          <li><a href="#" class="txtlink"><i class="fa fa-search"></i>Menu 1.5</a></li>
        </ul>
        <i class="arrows fa fa-chevron-down"></i>
      </li>
      <li><a href="#" class="txtlink"><i class="fa "></i>Menu 2</a></li>
      <li class="children">
        <a href="#" class="txtlink"><i class="fa fa-folder"></i>Menu 3</a>
        <ul class="sub">
          <li class="children">
            <a href="#" class="txtlink"><i class="fa fa fa-search"></i>Menu 3.1</a>
            <ul class="sub">
              <li><a href="#" class="txtlink"><i class="fa fa-user"></i>Menu 3.1.1</a></li>
              <li><a href="#" class="txtlink"><i class="fa fa-bookmark-o"></i>Menu 3.1.2</a></li>
            </ul>
            <i class="arrows fa fa-chevron-down"></i>
          </li>
          <li><a href="#" class="txtlink"><i class="fa"></i>Menu 3.2</a></li>
          <li><a href="#" class="txtlink"><i class="fa"></i>Menu 3.3</a></li>
          <li><a href="#" class="txtlink"><i class="fa fa fa-calendar"></i>Menu 3.4</a></li>
          <li><a href="#" class="txtlink"><i class="fa fa fa-search"></i>Menu 3.5</a></li>
        </ul>
        <i class="arrows fa fa-chevron-down"></i>
      </li>
      <li><a href="#" class="txtlink"><i class="fa fa-comments"></i>Menu 4</a></li>
    </ul>
  </div>

</nav>

问题是当我使用小箭头 expand/shrink 菜单时它工作正常并且箭头被正确交换。但是，当我单击文本元素时，菜单中的图标全部被箭头替换。

示例：使用箭头时，搜索图标保留在菜单中。当我单击文本 link 时，它会被箭头替换。

我想我在这里使用了错误的 jquery 选择器，但由于我是 jquery 的新手，所以我无法修复它。那么我这里的错误是什么？

当我点击箭头时

Answer 1

使用.children()代替.find()以确保只直接 children 包含在查询中。
你的onclick事件可以合并使用 $('.children > a.txtlink, .children > i.arrows') 转换为 1 个事件侦听器。同样，使用 > 直接 child 选择器来确保我们没有查询嵌套的 children.

现在应该可以正常工作了：

$(document).ready(function() {

  $('.children > a.txtlink, .children > i.arrows').on("click", function(e) {
    // toggle the icons when the text link is pressed

    jQuery(this).parent().children('ul:first').toggle(); // toggle the next UL

    if (jQuery(this).parent().children('i.arrows').hasClass("fa-chevron-down")) {
      jQuery(this).parent().children('i.arrows').removeClass("fa-chevron-down").addClass("fa-chevron-up");
    } else {
      jQuery(this).parent().children('i.arrows').removeClass("fa-chevron-up").addClass("fa-chevron-down");
    }

    e.stopPropagation();
    e.preventDefault();
  });
});

.navbar_menu {
  background: #222222;
}

.menu {
  border-top: 1px solid #2D2D2D;
}

.menu ul {
  padding: 0;
  margin: 0;
}

.menu li {
  list-style: none;
  position: relative;
}

.menu li i {
  padding-right: 3px;
}

.menu a.txtlink {
  color: #ccc;
  display: block;
  font-size: 16px;
  padding: 10px;
  padding-right: 0;
  border-bottom: 1px solid #2D2D2D;
  text-decoration: none;
}

.menu a.txtlink:hover {
  background: #111111;
  color: #ffffff;
  text-decoration: none;
}

.menu li.home a.txtlink::before {
  content: '\f015';
  display: inline-block;
  font: normal normal normal 14px/1 FontAwesome;
  font-size: inherit;
  text-rendering: auto;
  -webkit-font-smoothing: antialiased;
  -moz-osx-font-smoothing: grayscale;
  transform: none;
}

.menu ul li {
  position: relative;
}

.menu ul li.children i.arrows {
  /* needed to show an icon in the mobile menu */
  position: absolute;
  top: 0;
  right: 0;
  padding: 14px;
  border-left: 1px solid #333333;
  margin: 0;
  cursor: pointer;
  display: block;
  color: #CCCCCC;
}

.menu .sub {
  display: none;
}

.menu ul li ul li a.txtlink {
  padding-left: 20px;
}

.menu ul li ul li ul li a.txtlink {
  padding-left: 40px;
}

.menu i.arrows:hover {
  background-color: #111111;
}

.menu ul li ul li {
  background-color: #333333;
}

<script src="https://code.jquery.com/jquery-3.2.0.min.js"></script>
<link href="https://maxcdn.bootstrapcdn.com/bootstrap/3.3.7/css/bootstrap.min.css" rel="stylesheet" />
<link href="https://maxcdn.bootstrapcdn.com/font-awesome/4.7.0/css/font-awesome.min.css" rel="stylesheet" />
<nav class="navbar_menu">

  <div class="menu">
    <ul>
      <li class="home"><a href="#" class="txtlink">Home</a></li>
      <li class="children">
        <a href="#" class="txtlink"><i class="fa fa-folder"></i>Menu 1</a>
        <ul class="sub">
          <li class="children">
            <a href="#" class="txtlink"><i class="fa fa-search"></i>Menu 1.1</a>
            <ul class="sub">
              <li><a href="#" class="txtlink"><i class="fa fa-user"></i>Menu 1.1.1</a></li>
              <li><a href="#" class="txtlink"><i class="fa fa-tag"></i>Menu 1.1.2</a></li>
            </ul>
            <i class="arrows fa fa-chevron-down"></i>
          </li>
          <li><a href="#" class="txtlink">Menu 1.2</a></li>
          <li><a href="#" class="txtlink">Menu 1.3</a></li>
          <li><a href="#" class="txtlink"><i class="fa fa-clock-o"></i>Menu 1.4</a></li>
          <li><a href="#" class="txtlink"><i class="fa fa-search"></i>Menu 1.5</a></li>
        </ul>
        <i class="arrows fa fa-chevron-down"></i>
      </li>
      <li><a href="#" class="txtlink"><i class="fa "></i>Menu 2</a></li>
      <li class="children">
        <a href="#" class="txtlink"><i class="fa fa-folder"></i>Menu 3</a>
        <ul class="sub">
          <li class="children">
            <a href="#" class="txtlink"><i class="fa fa fa-search"></i>Menu 3.1</a>
            <ul class="sub">
              <li><a href="#" class="txtlink"><i class="fa fa-user"></i>Menu 3.1.1</a></li>
              <li><a href="#" class="txtlink"><i class="fa fa-bookmark-o"></i>Menu 3.1.2</a></li>
            </ul>
            <i class="arrows fa fa-chevron-down"></i>
          </li>
          <li><a href="#" class="txtlink"><i class="fa"></i>Menu 3.2</a></li>
          <li><a href="#" class="txtlink"><i class="fa"></i>Menu 3.3</a></li>
          <li><a href="#" class="txtlink"><i class="fa fa fa-calendar"></i>Menu 3.4</a></li>
          <li><a href="#" class="txtlink"><i class="fa fa fa-search"></i>Menu 3.5</a></li>
        </ul>
        <i class="arrows fa fa-chevron-down"></i>
      </li>
      <li><a href="#" class="txtlink"><i class="fa fa-comments"></i>Menu 4</a></li>
    </ul>
  </div>

</nav>

Answer 2

首先，在尝试这个和提供代码方面做得很好。也就是说，有很多方法可以改进这一点，我没有时间深入探讨，但已经做了一些改变来弥补这一差距：

使用变量使代码更高效、更易读。存储 $(this) 和 $(this).parent() 是您代码中的两个必须条件
你的问题的核心是 $parent.find('i').hasClass('fa-chevron-down')，它将所有 <i> 元素定位在该父元素下，如果有 class，它是会是真的。我很快将其更改为仅查找 i.fa-chevron-down，但使用 jQuery.children() 可能也是更好的选择
为了避免$冲突，避免$(document).ready而使用jQuery(document).ready是一个很好的经验法则； $ 然后可以在内部使用，如果你调用命名参数它；例如 .ready(function($){

jQuery(document).ready(function($) {
  $('.children a.txtlink').on("click", function(e) {
    // toggle the icons when the text link is pressed

    var $this = $(this),
      $parent = $this.parent(),
      $ul = $parent.find('ul:first');

    $ul.toggle(); // toggle the next UL

    if ($parent.find('i.fa-chevron-down').length) {
      $parent.find('i.fa-chevron-down').removeClass("fa-chevron-down").addClass("fa-chevron-up");
    } else {
      $parent.find('i.fa-chevron-up').removeClass("fa-chevron-up").addClass("fa-chevron-down");
    }

    e.stopPropagation();
    e.preventDefault();
  });

  $('.children i.arrows').on("click", function(e) {
    // toggle the icons when the chevron is pressed
    
    var $this = $(this),
      $parent = $this.parent(),
      $ul = $parent.find('ul:first');

    $ul.toggle(); // toggle the next UL

    if ($this.hasClass("fa-chevron-down")) {
      $this.removeClass("fa-chevron-down").addClass("fa-chevron-up");
    } else {
      $this.removeClass("fa-chevron-up").addClass("fa-chevron-down");
    }

    e.stopPropagation();
    e.preventDefault();
  });


});

.navbar_menu {
  background: #222222;
}

.menu {
  border-top: 1px solid #2D2D2D;
}

.menu ul {
  padding: 0;
  margin: 0;
}

.menu li {
  list-style: none;
  position: relative;
}

.menu li i {
  padding-right: 3px;
}

.menu a.txtlink {
  color: #ccc;
  display: block;
  font-size: 16px;
  padding: 10px;
  padding-right: 0;
  border-bottom: 1px solid #2D2D2D;
  text-decoration: none;
}

.menu a.txtlink:hover {
  background: #111111;
  color: #ffffff;
  text-decoration: none;
}

.menu li.home a.txtlink::before {
  content: '\f015';
  display: inline-block;
  font: normal normal normal 14px/1 FontAwesome;
  font-size: inherit;
  text-rendering: auto;
  -webkit-font-smoothing: antialiased;
  -moz-osx-font-smoothing: grayscale;
  transform: none;
}

.menu ul li {
  position: relative;
}

.menu ul li.children i.arrows {
  /* needed to show an icon in the mobile menu */
  position: absolute;
  top: 0;
  right: 0;
  padding: 14px;
  border-left: 1px solid #333333;
  margin: 0;
  cursor: pointer;
  display: block;
  color: #CCCCCC;
}

.menu .sub {
  display: none;
}

.menu ul li ul li a.txtlink {
  padding-left: 20px;
}

.menu ul li ul li ul li a.txtlink {
  padding-left: 40px;
}

.menu i.arrows:hover {
  background-color: #111111;
}

.menu ul li ul li {
  background-color: #333333;
}

<script src="https://code.jquery.com/jquery-3.2.0.min.js"></script>
<link href="https://maxcdn.bootstrapcdn.com/bootstrap/3.3.7/css/bootstrap.min.css" rel="stylesheet" />
<link href="https://maxcdn.bootstrapcdn.com/font-awesome/4.7.0/css/font-awesome.min.css" rel="stylesheet" />
<nav class="navbar_menu">

  <div class="menu">
    <ul>
      <li class="home"><a href="#" class="txtlink">Home</a></li>
      <li class="children">
        <a href="#" class="txtlink"><i class="fa fa-folder"></i>Menu 1</a>
        <ul class="sub">
          <li class="children">
            <a href="#" class="txtlink"><i class="fa fa-search"></i>Menu 1.1</a>
            <ul class="sub">
              <li><a href="#" class="txtlink"><i class="fa fa-user"></i>Menu 1.1.1</a></li>
              <li><a href="#" class="txtlink"><i class="fa fa-tag"></i>Menu 1.1.2</a></li>
            </ul>
            <i class="arrows fa fa-chevron-down"></i>
          </li>
          <li><a href="#" class="txtlink">Menu 1.2</a></li>
          <li><a href="#" class="txtlink">Menu 1.3</a></li>
          <li><a href="#" class="txtlink"><i class="fa fa-clock-o"></i>Menu 1.4</a></li>
          <li><a href="#" class="txtlink"><i class="fa fa-search"></i>Menu 1.5</a></li>
        </ul>
        <i class="arrows fa fa-chevron-down"></i>
      </li>
      <li><a href="#" class="txtlink"><i class="fa "></i>Menu 2</a></li>
      <li class="children">
        <a href="#" class="txtlink"><i class="fa fa-folder"></i>Menu 3</a>
        <ul class="sub">
          <li class="children">
            <a href="#" class="txtlink"><i class="fa fa fa-search"></i>Menu 3.1</a>
            <ul class="sub">
              <li><a href="#" class="txtlink"><i class="fa fa-user"></i>Menu 3.1.1</a></li>
              <li><a href="#" class="txtlink"><i class="fa fa-bookmark-o"></i>Menu 3.1.2</a></li>
            </ul>
            <i class="arrows fa fa-chevron-down"></i>
          </li>
          <li><a href="#" class="txtlink"><i class="fa"></i>Menu 3.2</a></li>
          <li><a href="#" class="txtlink"><i class="fa"></i>Menu 3.3</a></li>
          <li><a href="#" class="txtlink"><i class="fa fa fa-calendar"></i>Menu 3.4</a></li>
          <li><a href="#" class="txtlink"><i class="fa fa fa-search"></i>Menu 3.5</a></li>
        </ul>
        <i class="arrows fa fa-chevron-down"></i>
      </li>
      <li><a href="#" class="txtlink"><i class="fa fa-comments"></i>Menu 4</a></li>
    </ul>
  </div>

</nav>

jQuery 在单击操作期间选择了错误的元素

jQuery is selecting wrong elements during on click action

javascript

css

jquery

jquery-selectors