记录日常工作中一些容易被忽视的错误及细节，持续更新......
一、问题：HashMap<Long, String>中，用get(Integer key)取不到值
Map<Long, String> map = new HashMap<Long, String>(); map.put(1L, "1"); System.err.println(map.get(1));// null System.err.println(map.get(1L));// 1
1.首先想到Long与Integer的hashCode方法不同，Integer-value Long-(int)(value ^ (value >>> 32))
但是！！计算出的hashCode值是相同的，不是问题所在
2.查看HashMap源码：注意加亮部分
先比较key.hash，然后first.key == key || key.equals(first.key)
/** * 先比较key.hash，然后first.key == key || key.equals(first.key) */ 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 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); } } return null; }
先看first.key == key："=="比较地址值，l是Long cache[]中的1，o是Integer cache[]中的1，false
Long l = 1L; Object o = 1; System.err.println(l == o);// false // 反编译后： Long l = Long.valueOf(1L); Object o = Integer.valueOf(1); System.err.println(l == o);
然后看key.equals(first.key)：先检查类型，false
//Long的equals方法 public boolean equals(Object obj) { if (obj instanceof Long) { return value == ((Long)obj).longValue(); } return false; }
引发新的问题：为什么这个是true？——反编译解决
Long l = 1L; System.err.println(l == 1);// true // 反编译后： Long l = Long.valueOf(1L); System.err.println(l.longValue() == 1L);//编译器直接将1转成1L
二、两个值相等的Integer不“==”
Integer c = 99999; Integer d = 99999; System.out.println(c == d);// false
Integer c = 99999;// 反编译：Integer c = Integer.valueOf(99999);
查看Integer源码：
-128 <= i <= 127时，直接在Integer cache[]中取；否则，new Integer(i)
public static Integer valueOf(int i) { if (i >= IntegerCache.low && i <= IntegerCache.high) return IntegerCache.cache[i + (-IntegerCache.low)]; return new Integer(i); }
结论：
int a = 99999; int b = 99999; System.err.println(a == b);// true Integer c = 99999; Integer d = 99999; System.out.println(c == d);// false Integer e = 127; Integer f = 127; System.out.println(e == f);// true
三、List.remove()方法调用错误
注意list两个remove方法，remove(int index) remove(Object o)
public static void main(String[] args) { List<Integer> list = new LinkedList<Integer>(); for (int i = 0; i < 9999999; i++) { list.add(i); } // remove(int index) long before = System.currentTimeMillis(); int i = 8888888; list.remove(i); long after = System.currentTimeMillis(); System.err.println("index=" + (after - before));// 6ms // remove(Object o) long before = System.currentTimeMillis(); Integer i = 8888888; list.remove(i); long after = System.currentTimeMillis(); System.err.println("Object=" + (after - before));// 96ms }
四、三目运算符与自动拆装箱
Map<String,Boolean> map = new HashMap<String, Boolean>(); Boolean b = (map!=null ? map.get("test") : false);// Exception in thread "main" java.lang.NullPointerException
查问题：
NullPointerException找不出原因
反编译看： ((Boolean)map.get("test")) == null
HashMap map = new HashMap(); Boolean boolean1 = Boolean.valueOf(map == null ? false : ((Boolean)map.get("test")).booleanValue());
结论：
三目运算符的语法规范，参见 jls-15.25。
三目运算符 当第二，第三位操作数分别为基本类型和对象时，其中的对象就会拆箱为基本类型进行操作。
以后注意：
1、保证三目运算符的第二第三位操作数都为对象类型
Map<String,Boolean> map = new HashMap<String, Boolean>(); Boolean b = (map!=null ? map.get("test") : Boolean.FALSE);
2、自动拆装箱问题
Integer integer = 1; // 装箱 Integer integer=Integer.valueOf(1); new Integer() int i = integer; // 拆箱 int i=integer.intValue();
1）包装对象的数值比较，不能简单的使用==（只有-128到127之间IntegerCache内的数字可以，但是这个范围之外还是需要使用equals比较）。
2）自动拆箱，如果包装类对象为null，那么自动拆箱时就有可能抛出NPE。
3）如果一个for循环中有大量装箱操作，会浪费很多资源。
五、switch语句忘记break
本来我跟你现在想的一样，一定不会忘，直到遇到了这个问题。
int i = 3; switch (i) { case 1: System.out.println(1); break; case 2: System.out.println(2); break; case 3: System.out.println(3); // 没有break, 不会有问题 }
当你需要在之后接着case的时候，直接复制case 3，就bug了。
（1）case完一定break，除非特别需要穿透到下一个case；
（2）复制代码前后都要检查是否有问题。
六、数值溢出问题
// 为了更好的展示问题，代码举出的是较极端的情况 public void overFlow(int a) { int b = 999999 * a; // 6个9 int最大值=2147483647 int limit = 999999999; // 9个9 if (b < limit) { System.out.println("a*b小于limit"); } }
如果a传入一个较大的int值，a*999999之后会超过int的最大值
而默认结果是int类型，会将a*999999的结果强转成int，导致不是想要的结果的结果
即使a*999999是大于limit的，强转成int后，b可能会比limit小，甚至可能是负数
解决：
// 用long类型计算（还会用一定风险） public void overFlow(int a) { long b = 999999L * a; int limit = 999999999; if (b < limit) { System.out.println("a*999999小于limit"); } } // 将加法和乘法转变成减法和除法运算 public void overFlow(int a) { int limit = 999999999; if (a < limit/999999) { System.out.println("a*999999小于limit"); } }
七、对象引用问题
public static void main(String[] args) { Map<Integer, Inner> map = new HashMap<Integer, Inner>(); // inner1.list [1] Inner inner1 = new Inner(new LinkedList<Integer>()); inner1.getList().add(1); map.put(1, inner1); // inner2.list [1, 2] Inner inner2 = new Inner(map.get(1).getList()); inner2.getList().add(2); map.put(2, inner2); for (Entry<Integer, Inner> entry : map.entrySet()) { System.err.println("" + entry.getKey() + " " + entry.getValue().getList()); } /** * 目的：inner1.list [1] inner2.list [1, 2] * 结果：inner1.list [1, 2] inner2.list [1, 2] */ } static class Inner { List<Integer> list; public List<Integer> getList() { return list; } public Inner(List<Integer> list) { this.list = list; } }
分析：
1.将inner1.list的引用给了inner2.list，nner1.list inner2.list指向的是同一个List。
2.很简单的问题，开发时习惯了构造方法里这样写：this.list = list;
解决：
this.list = list; 改成 this.list = new LinkedList<Integer>(list);