[java]Set集合删除元素失效问题

最新推荐文章于 2023-06-06 15:35:33 发布

春风十米

最新推荐文章于 2023-06-06 15:35:33 发布

阅读量2.3k

点赞数 1

分类专栏： Java基础

原文链接：https://segmentfault.com/a/1190000021186549

版权

Java基础专栏收录该内容

26 篇文章 0 订阅

订阅专栏

[java]Set集合删除元素失效问题

java8 javaz

转自： https://segmentfault.com/a/1190000021186549

引子

记一次在工作当中，有这样一个业务场景：

在通过接口返回前，需要为一个set中的所有元素赋值，然后剔除掉一些不符合条件的元素，最后返回。代码结构大概如下：

代码

Set<Item> sets = Sets.newHashSet();
sets.addAll(items);
...
setValue(sets) ;

//剔除sets中不符合条件的元素
**sets.removeIf(Predicate filter);**

//为set中的元素设置属性
private setValue(Set<Item> sets){
    for(Item:item sets){
        ...
    }
}

看似是一段clean code，逻辑清晰，实则蕴藏杀机；大家大概一眼就能看出其中玄机，其中的removeif压根就不起作用啊，可是菜如狗的我还没发现其中一二，还在慢慢的debug，写单元测试，并单纯的以为是编译的问题...终于一位同事在review之后，一语点醒我："removeIf 是如何找到你要删除的元素的呢？"

原因

终于，我在查看了set集合removeIf与addAll的原理以后，顿悟了，主要有四个关键点：

HashSet底层实际上就是一个HashMap, 节选一段hashSet的构造方法就能发现，其中key为要添加的元素，value为Present，如下：
```
    private static final Object PRESENT = new Object();

    public HashSet() {
        map = new HashMap<>();
    }
    
```

在向set中添加元素调用addAll方法时，实际上也是循环调用set的add方法，而实际上也是调用map的put方法。

public boolean addAll(Collection<? extends E> c) {
        boolean modified = false;
        for (E e : c)
            if (add(e))
                modified = true;
        return modified;
    }

public boolean add(E e) {
        return map.put(e, PRESENT)==null;
 }

set在执行removeIf操作时，实际上调用的是 iterator的remove操作,而collection继承了iterator，set继承了collection，底层通过hashmap实现，（@w@～有点绕），实际上就是调用的map的remove方法，如下：

default boolean removeIf(Predicate<? super E> filter) {
        Objects.requireNonNull(filter);
        boolean removed = false;
        final Iterator<E> each = iterator();
        while (each.hasNext()) {
            if (filter.test(each.next())) {
                each.remove();
                removed = true;
            }
        }
        return removed;
    }

现在我们了解到，set在添加或者是删除元素均是调用的map的对应的方法，而hashMap是通过数据+链表+红黑树实现的,但是map是如何找到这些元素，并进行相应的添加或删除操作呢？如下put方法，

首先调用了hash方法计算key的hash值，在hash方法则调用了key的hashCode方法以及移位以及逻辑异或运算。

然后又调用了putVal方法，在该方法中我们了解到hashMap主要是通过key的该hash值进行寻找元素的插入位置的，若已存在，值覆盖，否则插入，其中的其他方法这里不再展开。

//首先是put方法的实现
 public V put(K key, V value) {
        return putVal(hash(key), key, value, false, true);
 }
//hash方法
static final int hash(Object key) {
        int h;
        return (key == null) ? 0 : (h = key.hashCode()) ^ (h >>> 16);
    }
//put又调用的putVal方法
final V putVal(int hash, K key, V value, boolean onlyIfAbsent,
                   boolean evict) {
        Node<K,V>[] tab; Node<K,V> p; int n, i;
        if ((tab = table) == null || (n = tab.length) == 0)
            n = (tab = resize()).length;
        if ((p = tab[i = (n - 1) & hash]) == null)
            tab[i] = newNode(hash, key, value, null);
        else {
            Node<K,V> e; K k;
            if (p.hash == hash &&
                ((k = p.key) == key || (key != null && key.equals(k))))
                e = p;
            else if (p instanceof TreeNode)
                e = ((TreeNode<K,V>)p).putTreeVal(this, tab, hash, key, value);
            else {
                for (int binCount = 0; ; ++binCount) {
                    if ((e = p.next) == null) {
                        p.next = newNode(hash, key, value, null);
                        if (binCount >= TREEIFY_THRESHOLD - 1) // -1 for 1st
                            treeifyBin(tab, hash);
                        break;
                    }
                    if (e.hash == hash &&
                        ((k = e.key) == key || (key != null && key.equals(k))))
                        break;
                    p = e;
                }
            }
            if (e != null) { // existing mapping for key
                V oldValue = e.value;
                if (!onlyIfAbsent || oldValue == null)
                    e.value = value;
                afterNodeAccess(e);
                return oldValue;
            }
        }
        ++modCount;
        if (++size > threshold)
            resize();
        afterNodeInsertion(evict);
        return null;
    }

接着是remove方法源码如下，我们发现，在remove方法中，仍然首先调用hash方法计算key的hash值，然后调用removeNode进行删除操作，这里在删除时移动其他元素的方法不再展开，我们可以知道，remove方法仍然是通过key的hash来查询要删除的元素的。

public V remove(Object key) {
        Node<K,V> e;
        return (e = removeNode(hash(key), key, null, false, true)) == null ?
            null : e.value;
    }

    /**

     *
     * @param hash hash for key
     * @param key the key
     * @param value the value to match if matchValue, else ignored
     * @param matchValue if true only remove if value is equal
     * @param movable if false do not move other nodes while removing
     * @return the node, or null if none
     */
    final Node<K,V> removeNode(int hash, Object key, Object value,
                               boolean matchValue, boolean movable) {
        Node<K,V>[] tab; Node<K,V> p; int n, index;
        if ((tab = table) != null && (n = tab.length) > 0 &&
            (p = tab[index = (n - 1) & hash]) != null) {
            Node<K,V> node = null, e; K k; V v;
            if (p.hash == hash &&
                ((k = p.key) == key || (key != null && key.equals(k))))
                node = p;
            else if ((e = p.next) != null) {
                if (p instanceof TreeNode)
                    node = ((TreeNode<K,V>)p).getTreeNode(hash, key);
                else {
                    do {
                        if (e.hash == hash &&
                            ((k = e.key) == key ||
                             (key != null && key.equals(k)))) {
                            node = e;
                            break;
                        }
                        p = e;
                    } while ((e = e.next) != null);
                }
            }
            if (node != null && (!matchValue || (v = node.value) == value ||
                                 (value != null && value.equals(v)))) {
                if (node instanceof TreeNode)
                    ((TreeNode<K,V>)node).removeTreeNode(this, tab, movable);
                else if (node == p)
                    tab[index] = node.next;
                else
                    p.next = node.next;
                ++modCount;
                --size;
                afterNodeRemoval(node);
                return node;
            }
        }
        return null;
    }
```

通过分析上述的removeIf与addAll源码可知，底层均调用的map的put与remove方法。同样的，set在add元素时，首先会计算元素的hash值，来寻找元素的插入位置；
在删除元素时，同样会重新计算元素的hash值来查询要删除的元素；而hash方法主要是通过元素的hashCode方法来计算的，那么出现删除失效的问题一定是与hahsCode方法有关的，我查了下该类，果然，hashCode方法被重写了........所以在add与remove时，对应元素的hashCode不同，因为这之间我为元素赋了值。

也就是说该问题是由 元素在add进集合时hashCode与remove时的hashcode不一致造成的。
....当然，能找到原因还是很开心的～

解决办法

最笨的，再新建一个集合，遍历，把符合条件的元素留下
挪一下方法的位置，不要在add与remove之间为相应元素属性复制，这当然要看你自己的业务场景与hashCode的重写逻辑咯
重写hashcode方法与equals方法。

春风十米

关注

1
点赞
踩
3

收藏

觉得还不错? 一键收藏
0
评论
[java]Set集合删除元素失效问题

[java]Set集合删除元素失效问题java8javaz转自：https://segmentfault.com/a/1190000021186549引子记一次在工作当中，有这样一个业务场景：在通过接口返回前，需要为一个set中的所有元素赋值，然后剔除掉一些不符合条件的元素，最后返回。代码结构大概如下：代码Set<Item> sets = Sets.newHashSet();sets.addAll(items);...setValue(set...
复制链接

扫一扫