android studio 警告 synchronization on non-final field

测试代码如下：

public class SyncNonFinalField {
    private Object object = new Object();
    public void start() {
        
        new Thread(new Runnable() {
            
            @Override
            public void run() {
                System.out.println("线程1等待锁");
                synchronized (object) {
                    System.out.println("线程1获取到了锁");
                    object = new Object();
                    System.out.println("线程1准备sleep");
                    try {
                        Thread.sleep(5000L);
                    } catch (InterruptedException e) {
                        e.printStackTrace();
                    }
                    System.out.println("线程1 sleep结束");
                }
            }
        }).start();
        
        new Thread(new Runnable() {
            
            @Override
            public void run() {
                try {
                    Thread.sleep(1000L);
                } catch (InterruptedException e) {
                    e.printStackTrace();
                }
                System.out.println("线程2等待锁");
                synchronized (object) {
                    System.out.println("线程2获取到了锁");
                }
            }
        }).start();
    }
}

输出结果：

线程1等待锁

线程1获取到了锁

线程1准备sleep

线程2等待锁

线程2获取到了锁

线程1 sleep结束

 

也就是说，在线程1的同步块中，只要一给object重新赋值，线程2立即就进入到了同步块中。

 

分析：

synchronized (object) 锁的是object这个引用所指向的对象，而不是锁引用。

线程1 sleep 时，并没有释放旧对象的锁。

但它让 object 指向了一个新对象之后，线程2就立即获取到了新对象的锁，所以线程2进入到了同步块中。

 

解决办法：

用final修饰object，避免它在同步块中被重新赋值。