java中Set中添加mutable变量修改后contains返回false

背景

题目有点复杂......语文太差....贴上代码吧

public static void containTest() {
        List<String> list = new ArrayList<>();
        list.add("a");
        Set<List<String>> set = new HashSet<>();
        set.add(list);
        System.out.println(set.contains(list));
        list.add("b");
        System.out.println(set.contains(list));
}

上面的代码返回如下:

true
false

而对于如下代码:

public static void sbContainTest() {
        StringBuilder sBuilder = new StringBuilder("a");
        Set set = new HashSet<>();
        set.add(sBuilder);
        System.out.println(set.contains(sBuilder));
        
        sBuilder.append("b");
        System.out.println(set.contains(sBuilder));
    }

返回的结果却是:

true 
true

我们来通过查看源码来了解这背后的原理.

分析

首先查看HashSet的源码, 大致贴一下其内部域以及几个和本文相关的方法如下:

public  class HashSet<E>
        extends AbstractSet<E>
        implements Set<E>, Cloneable, java.io.Serializable{
        static final long serialVersionUID = -5024744406713321676L;
        private transient HashMap<E, Object> map;
        
        // Dummy value to associate with an Object in the backing Map
        private static final Object PRESENT = new Object();

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

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

        public boolean contains(Object o) {
            return map.containsKey(o);
        }

        //other methods
}

可以看到HashSet内部是使用一个HashMap存储数据的, 而contains方法调用的是HashMapcontains方法.

我们再查看HashMap源码, 看看contiansKey的实现. 同上, 贴一些关键代码

public class HashMap<K,V> extends AbstractMap<K,V>
        implements Map<K,V>, Cloneable, Serializable {
        private static final long serialVersionUID = 362498820763181265L;
        transient Node<K, V>[] table;
        transient int size;
        
        public boolean containsKey(Object key) {
            return getNode(hash(key), key) != null;
        }
        
        public V put(K key, V value) {
            return putVal(hash(key), key, value, false, true);
        }

        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);    //look at this line!!!!!
            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;
        }
        /**
        *@param: hash hash for key, key the key
        *@return: the node, or null if none
        **/
        final Node<K, V> getNode(int hash, Object key) {
            Node<K, V>[] tab; 
            Node<K, V> first, e; 
            int n; 
            K k;
            if (( tab = table) != null && (n = tab.length) > 0 && (first = tab[(n-1) & hash]) != null) {
                if (first.hash == hash && ((k = first.key) == key || (key != null && key.equals(k)))) {
                    return first;
                }
                if ((e = first.next) != null) {
                    if (first isinstanceof TreeNode){
                        return ((TreeNode<K, V> first).getTreeNode(hash, key);
                    do {
                        if (e.hash == hash && ((k = e.key) == key || (key != null && key.equals(k)))) {
                            return e;
                        } while ((e = e.next) != null);
                }
            }
            return null
        }
        
        static final int hash(Object key) {
            int h;
            return (key == null) ? 0 : (h = key.hashCode()) ^ (h >>>16);
        }

}
                

通过查看上面的代码我们就知道原因了, 简单地用语言描述一下

原因

我们运行如下代码:

public static void hashCodeTest() {
        List<String> list = new ArrayList<>();
        list.add("a");
        System.out.println(list.hashCode());
        list.add("b");
        System.out.println(list.hashCode());
        
        StringBuilder sBuilder = new StringBuilder("a");
        System.out.println(sBuilder.hashCode());
        sBuilder.append("b");
        System.out.println(sBuilder.hashCode());
    }

返回结果如下

128
4066
1094834071
1094834071

首先我们设list里面只有"a"时的hashcode的值为a, 添加了"b"之后的hashcode的值为b.
当我们第一运行set.add(list)时候
当运行set.add(list)时, 注意观察上面我贴的HashMap的源码第21行(我用look at this line!!!!!注释了), 可知HashMap是根据此时的hashcode(也就是a)来确定list存储在set中的索引.
而我们第二次运行set.contains(list)时HashMap会用新的索引(也就是b)来寻找目标, 自然找不到了. 于是就返回了false.
而sBuilder的hashcode的值没有改变, 自然也就能被找到, 返回true了.

收获

以后使用和hash相关的数据结构的时候一定要注意 hash值得变化, 变了可能就找不到了. hash固然快, 但是也是有缺点呐.

©著作权归作者所有,转载或内容合作请联系作者
  • 序言:七十年代末,一起剥皮案震惊了整个滨河市,随后出现的几起案子,更是在滨河造成了极大的恐慌,老刑警刘岩,带你破解...
    沈念sama阅读 213,014评论 6 492
  • 序言:滨河连续发生了三起死亡事件,死亡现场离奇诡异,居然都是意外死亡,警方通过查阅死者的电脑和手机,发现死者居然都...
    沈念sama阅读 90,796评论 3 386
  • 文/潘晓璐 我一进店门,熙熙楼的掌柜王于贵愁眉苦脸地迎上来,“玉大人,你说我怎么就摊上这事。” “怎么了?”我有些...
    开封第一讲书人阅读 158,484评论 0 348
  • 文/不坏的土叔 我叫张陵,是天一观的道长。 经常有香客问我,道长,这世上最难降的妖魔是什么? 我笑而不...
    开封第一讲书人阅读 56,830评论 1 285
  • 正文 为了忘掉前任,我火速办了婚礼,结果婚礼上,老公的妹妹穿的比我还像新娘。我一直安慰自己,他们只是感情好,可当我...
    茶点故事阅读 65,946评论 6 386
  • 文/花漫 我一把揭开白布。 她就那样静静地躺着,像睡着了一般。 火红的嫁衣衬着肌肤如雪。 梳的纹丝不乱的头发上,一...
    开封第一讲书人阅读 50,114评论 1 292
  • 那天,我揣着相机与录音,去河边找鬼。 笑死,一个胖子当着我的面吹牛,可吹牛的内容都是我干的。 我是一名探鬼主播,决...
    沈念sama阅读 39,182评论 3 412
  • 文/苍兰香墨 我猛地睁开眼,长吁一口气:“原来是场噩梦啊……” “哼!你这毒妇竟也来了?” 一声冷哼从身侧响起,我...
    开封第一讲书人阅读 37,927评论 0 268
  • 序言:老挝万荣一对情侣失踪,失踪者是张志新(化名)和其女友刘颖,没想到半个月后,有当地人在树林里发现了一具尸体,经...
    沈念sama阅读 44,369评论 1 303
  • 正文 独居荒郊野岭守林人离奇死亡,尸身上长有42处带血的脓包…… 初始之章·张勋 以下内容为张勋视角 年9月15日...
    茶点故事阅读 36,678评论 2 327
  • 正文 我和宋清朗相恋三年,在试婚纱的时候发现自己被绿了。 大学时的朋友给我发了我未婚夫和他白月光在一起吃饭的照片。...
    茶点故事阅读 38,832评论 1 341
  • 序言:一个原本活蹦乱跳的男人离奇死亡,死状恐怖,灵堂内的尸体忽然破棺而出,到底是诈尸还是另有隐情,我是刑警宁泽,带...
    沈念sama阅读 34,533评论 4 335
  • 正文 年R本政府宣布,位于F岛的核电站,受9级特大地震影响,放射性物质发生泄漏。R本人自食恶果不足惜,却给世界环境...
    茶点故事阅读 40,166评论 3 317
  • 文/蒙蒙 一、第九天 我趴在偏房一处隐蔽的房顶上张望。 院中可真热闹,春花似锦、人声如沸。这庄子的主人今日做“春日...
    开封第一讲书人阅读 30,885评论 0 21
  • 文/苍兰香墨 我抬头看了看天上的太阳。三九已至,却和暖如春,着一层夹袄步出监牢的瞬间,已是汗流浃背。 一阵脚步声响...
    开封第一讲书人阅读 32,128评论 1 267
  • 我被黑心中介骗来泰国打工, 没想到刚下飞机就差点儿被人妖公主榨干…… 1. 我叫王不留,地道东北人。 一个月前我还...
    沈念sama阅读 46,659评论 2 362
  • 正文 我出身青楼,却偏偏与公主长得像,于是被迫代替她去往敌国和亲。 传闻我的和亲对象是个残疾皇子,可洞房花烛夜当晚...
    茶点故事阅读 43,738评论 2 351

推荐阅读更多精彩内容

  • 1. Java基础部分 基础部分的顺序:基本语法,类相关的语法,内部类的语法,继承相关的语法,异常的语法,线程的语...
    子非鱼_t_阅读 31,604评论 18 399
  • 本文出自 Eddy Wiki ,转载请注明出处:http://eddy.wiki/interview-java.h...
    eddy_wiki阅读 1,156评论 0 16
  • 从三月份找实习到现在,面了一些公司,挂了不少,但最终还是拿到小米、百度、阿里、京东、新浪、CVTE、乐视家的研发岗...
    时芥蓝阅读 42,220评论 11 349
  • 1.let 由javavscript的变量作用域其实是函数的内部,所以我们在for循环等语句块中是无法定义具有局部...
    youngiyang_打码少年阅读 705评论 0 0
  • 生活改变了我,也改变了你,以前碰到自己不喜欢的吃的东西,就不会去动下筷子,如今,即使以前如何的不喜欢,也会去尝试一...
    王光磊_滁州阅读 219评论 1 2