java++锁的对象_你竟然用Integer作为synchronized的锁对象？

在使用多线程编程时，往往会使用一些手段保证线程安全，也就是加锁，但是加锁也必须合理，如使用synchronized对对象加锁时，如果不注意，还可能发生错误的加锁。

先看一段小测试，在这个小测试中，启动了1000个线程，每个线程在对integer加1前都先获得integer的锁，这看似是线程安全的，并且预期可能会得到1000这个值，而然并不然，在运行多次之后他总是输出<=1000的值，那么，这犊子是哪出问题了。

static Integer integer =new Integer("0");

static CountDownLatch mCountDownLatch =new CountDownLatch(1000);

public static void main(String[] args) throws InterruptedException {

for (int i = 1; i <=1000; i++) {

new Thread(()->{

synchronized (integer){

integer++;

mCountDownLatch.countDown();

}

}).start();

}

mCountDownLatch.await();

System.out.println(integer);

}

这其实就是上述所说："错误的加锁对象"，也就是Integer作为锁对象这是个不正确的选择。

那在说Integer，不少人可能会见过这样的题。

Integer i1 =125;

Integer i2 =125;

System.out.println(i1==i2);

Integer i3 =135;

Integer i4 =135;

System.out.println(i3==i4);

答案是true，false，这样的结果是由Integer的缓存导致，在直接对Integer=xxx时候，其实调用了Integer.valueOf，(可以使用javap命令反编译查看)

而Integer.valueOf中，如果参数值在一定范围内，就从IntegerCache缓存中返回，也就是说在一定范围内多个相同的值是同一个对象，超出的话则return new Integer(i)返回一个新的Integer。而这个范围在-128-127，

在上面的135显然不在范围，则返回的是新对象，又由于==是比较内存地址，所以上述会出现false。如果要比较包装类是否相等，正确的做法是equals方法，Integer也重写了equals，判断内部基本数据类型是否一样。

这样一来，synchronized(integer)原因就知道了， integer++是创建了一个新的Integer对象，并将引用赋值给integer。因为integer一直在变，线程每次加锁可能都加在了不同对象的实例上，导致临界区控制出现问题。

解决办法也很容易，只要加在一个不变的对象上即可。

static Integer integer =new Integer("0");

static CountDownLatch mCountDownLatch =new CountDownLatch(1000);

static Object sObject= new Object();

public static void main(String[] args) throws InterruptedException {

for (int i = 1; i <=1000; i++) {

new Thread(()->{

synchronized (sObject){

integer++;

mCountDownLatch.countDown();

}

}).start();

}

mCountDownLatch.await();

System.out.println(integer);

}

(不只Integer，Character、Short、Long也有缓存，但是Float和Double却没有)