从动态数组中删除项目

时间:2016-12-16 09:49:59

标签: javascript jquery arrays checkbox

我有一些列表和复选框可用作过滤器,我需要做的是当你选择一个复选框时,将该值添加到标题旁边的小标签中,取消选中后删除该值。

我正在做这项工作,将每个值保存到一个数组中,并在取消选中时尝试查找该值的索引并删除,但我认为我与数组中的位置有冲突。如果您逐个单击然后逐个删除它然后它可以工作,但如果您单击所有这些并且仅取消选中第一个,它将删除所有内容。有一个线索如何解决它,但不知道如何继续。

请帮忙吗? https://jsfiddle.net/panconjugo/qsgwvp63/2/

$(function() {
  $(".filter-nav > li").each(function(index) {
    var brands = [];
    $(this).find("input:checkbox").each(function(index) {
      $(this).on("click", function() {
        if ($(this).is(":checked")) {
          console.log("adding: " + index);
          brands.push($(this).val());
          console.log(brands);
          $(this).parent().parent().prev().find(".selected-group").text(brands.join(", "));
        } else {
          console.log("removing:" + index);
          brands.splice(index);
          console.log(brands);
          $(this).parent().parent().prev().find(".selected-group").text(brands.join(", "));
        }
      });
    });
  });
});

2 个答案:

答案 0 :(得分:3)

将代码brands.splice(index);更改为brands.splice(brands.indexOf($(this).val()), 1);

首先你必须从数组中找到项索引,然后使用该索引从数组中删除

答案 1 :(得分:1)

不是通过添加/删除项来维护数组,而是挂钩复选框的change事件然后使用包含数据的map()构建新数组要简单得多。该组中的所选项目。然后,您可以将它们连接在一起并根据需要显示文本。试试这个:



$('.filter-nav input:checkbox').on("change", function() {
    var $parent = $(this).closest('.filter-nav > li');
    var items = $parent.find('input:checkbox:checked').map(function() {
    	return this.value;
    }).get();
    $parent.find('.selected-group').text(items.join(', '));
});

.filter-nav {
  list-style: none;
  padding: 0;
}
.filter-nav li {
  line-height: 40px;
}
.filter-nav > li > h3 {
  background-color: #222;
  color: white;
  padding-left: 20px;
  cursor: pointer;
}
.selected-group {
  color: yellow;
}

<script src="https://ajax.googleapis.com/ajax/libs/jquery/2.1.1/jquery.min.js"></script>
<ul class="filter-nav">
  <li>
    <h3>Brand <small class="selected-group"></small></h3>
    <ul>
      <li>
        <label for="brand1">Brand1</label>
        <input type="checkbox" id="brand1" value="Brand 1">
      </li>
      <li>
        <label for="brand1">Brand2</label>
        <input type="checkbox" id="brand2" value="Brand 2">
      </li>
      <li>
        <label for="brand1">Brand3</label>
        <input type="checkbox" id="brand3" value="Brand 3">
      </li>
    </ul>
  </li>
  <li>
    <h3>Size <small class="selected-group"></small></h3>
    <ul>
      <li>
        <label for="xs">XS</label>
        <input type="checkbox" id="xs" value="xs">
      </li>
      <li>
        <label for="m">M</label>
        <input type="checkbox" id="m" value="m">
      </li>
      <li>
        <label for="l">L</label>
        <input type="checkbox" id="l" value="l">
      </li>
    </ul>
  </li>
  <li>
    <h3>Color <small class="selected-group"></small></h3>
    <ul>
      <li>
        <label for="blue">Blue</label>
        <input type="checkbox" id="blue" value="blue">
      </li>
      <li>
        <label for="green">Green</label>
        <input type="checkbox" id="green" value="green">
      </li>
      <li>
        <label for="red">Red</label>
        <input type="checkbox" id="red" value="red">
      </li>
    </ul>
  </li>
</ul>
&#13;
&#13;
&#13;