一 与 monitor 关联的对象不能为空

public class SynchronizedTest {
    private final Object mutex = null;

    public void syncMethod() {
        synchronized (mutex) {
        }
    }
}

mutex 为 null，每个对象和一个 monitor 关联，对象都为 null，monitor 肯定无从谈起。

二  synchronized 作用域太大

public class SynchronizedTest implements Runnable{
    private final Object mutex = null;
    
    @Override
    public synchronized void run() {
       
    }
}

上面的代码对整个线程的执行逻辑单元都进行了 synchronized 同步，从而丧失了并发能力，synchronized  关键字应该尽可能地只作用于共享资源（数据）的读写作用域。

三 不同 monitor 企图锁相同的方法

public class SynchronizedTest implements Runnable {
    private final Object mutex = null;

    @Override
    public void run() {
        synchronized (mutex) {
            
        }
    }

    public static void main(String[] args) {
        for (int i = 0; i < 5; i++) {
            new Thread(SynchronizedTest::new).start();
        }
    }
}

上面代码构造了五个线程，synchronized  根本互斥不了与之对应的作用域，线程之间进行 monitor lock 的争抢只能发生在与 monitor 关联的同一引用上，上面代码每一个线程争抢的 monitor 关联引用都是彼此独立的，因此不可能起到互斥的作用。

四 多个线程交叉导致死锁

多个线程的交叉很容易引起线程出现死锁的情况，程序并没有任何错误输出，但就是不工作。

public class SynchronizedTest {
    private final Object mutexRead = new Object();
    private final Object mutexWrite = null;

    public void read() {
        synchronized (mutexRead) {
            synchronized (mutexWrite) {
            }
        }
    }

    public void write() {
        synchronized (mutexWrite) {
            synchronized (mutexRead) {
                
            }
        }
    }
}