两个基本功不足导致的bug

作为程序员，基本功不好，可能会在工作中经常碰到一些看起来很隐蔽的 bug，乍看没毛病，自己半天还找不到问题所在。

但是，如果基本功扎实的同学可能一眼就能看出来。

一、HashMap 取不到值

Map<Integer, Integer> map = new HashMap<>();
resMap.put(1, 1);
System.out.println(map.get(1L));
System.out.println(map.get(1));

大家可以看下，上面的代码输出是什么？我稍后公布答案。

1、源码分析

HashMap的get方法源码如下（已增加自己的注释）：


public V get(Object key) {Node<K,V> e;return (e = getNode(hash(key), key)) == null ? null : e.value;
}/*** Implements Map.get and related methods.** @param hash hash for key* @param 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;// 如果map不为空if ((tab = table) != null && (n = tab.length) > 0 &&(first = tab[(n - 1) & hash]) != null) {// 如果直接通过传进来的key找到了值，直接返回// 1）比较传进来key的hash值和在map中对应位置找到的结点的hash值是否一致// 2）比较传进来的key对象和在map中对应位置找到的结点的key对象（object）是否相等if (first.hash == hash && // always check first node((k = first.key) == key || (key != null && key.equals(k))))return first;// 如果通过hash找到的结点的下一个节点不为空，说明是链表if ((e = first.next) != null) {// 如果是红黑树，直接红黑树查找if (first instanceof 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);}}// 上述都不满足，返回nullreturn null;
}

如果传的 key 对应的 hash 值，能够匹配到 map 中的结点（只能说 hash 表（map）中这个位置有东西），还需要进行下面两个判断。

1）比较传进来 key 的 hash 值和在 map 中对应位置找到的结点的 hash 值是否一致

2）比较传进来的 key 对象和在 map 中对应位置找到的结点的key对象（object）是否相等。

看了上述源码分析之后，我们公布答案：

null
1

最终的差异就是

(k = first.key) == key || (key != null && key.equals(k))

这段代码，相当于 Objects.equals(key, k)。

这里比较的是，map 中存储的对象的key，命名为k，以及get方法传给map的 key，命名为 key。

相当于比较new Integer(1) 和 new Long(1L)，我们知道它们是两个不同的对象，所以结果肯定不相等。所以 key 是1L的时候，结果是 null。

2、结论

Map 获取值的时候，key类型不匹配，获取不到value。

二、日期计算

大家可以看看下面这段代码，分别输出的是什么？

long nowTime = new Date().getTime();
System.out.println(nowTime);
System.out.println(new Date(nowTime - 7 * 24 * 60 * 60 * 1000));
System.out.println(new Date(nowTime - 25 * 24 * 60 * 60 * 1000));
System.out.println(new Date(nowTime - 30 * 24 * 60 * 60 * 1000));
System.out.println(new Date(nowTime - 60 * 24 * 60 * 60 * 1000));

我们可以把代码扔给 GPT，看它是否能识别其中的 bug。

上面的代码中，存在一个潜在的问题。Date 的构造函数中传入的时间戳参数是 long 类型，而 days * 24 * 60 * 60 * 1000 的计算结果是 int 类型，存在可能的整数溢出问题。
因为 days 和其他整数相乘后超过了 int 类型能表示的范围，所以这会导致计算的结果出现错误。
修复的代码可以将 int 类型的计算结果强制转换为 long 类型，确保计算的精度不会丢失。修复后的代码如下：

int days = Integer.valueOf(daysStr);
long millisecondsInDay = (long) days * 24 * 60 * 60 * 1000;
Date startDate = new Date(System.currentTimeMillis() - millisecondsInDay);

这样就可以避免由于整数溢出而导致的计算错误。

所以，这里的知识点是 int 数组进行计算结果是 int 类型，而 int 类型的范围是 21 亿（2^31 - 1）左右。

如果用来计算时间戳，很容易就会越界，导致非预期结果。