java集合HashMap中的源码疑问?

学习java集合HashMap中的源码发现,在putVal方法中,判断对应索引已存在元素是否相等的条件不明白为什么这样多此一举判断hash是否相等(下面注释行),因为这里的hash即使不相等但是肯定都一定在数组同一个索引位,所以这里判断hash是否相等感觉是多余的,

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;
    }

不知道是否可以把判断条件优化成如下

if ((k = p.key) == key || (key != null && key.equals(k)))
阅读 1.5k
2 个回答

equals 相等,则 hashCode 必然相等。
hashCode 的比较速度比 equals 快,能加快判断。
image.png

因为它本身判断是否为同一个key的条件就是两个:

  1. 先判断key的hashcode是否一样,如果不一样,就没必要继续判断第二个条件了
  2. 如果hashcode一样,继续判断是否 ==equals为true

所以这里是不可以优化的,因为如果只满足equals条件的话,是不足以判断为同一个key的,hashcode不一定相等

撰写回答
你尚未登录,登录后可以
  • 和开发者交流问题的细节
  • 关注并接收问题和回答的更新提醒
  • 参与内容的编辑和改进,让解决方法与时俱进
推荐问题
宣传栏