为何synchronized(obj)失效致结果为-1或0？synchronized(this)却正常

嘿，这个问题的核心是锁对象的唯一性，咱们结合你的代码来理清楚：

先补全你的完整代码（还原while循环逻辑） #

public class RunnableSales { 
    public static void main(String[] args) { 
        Ticket1 t = new Ticket1(); 
        Thread t1 = new Thread(t); 
        Thread t2 = new Thread(t); 
        Thread t3 = new Thread(t); 
        Thread t4 = new Thread(t); 
        t1.start(); 
        t2.start(); 
        t3.start(); 
        t4.start(); 
    } 
} 
class Ticket1 implements Runnable { 
    private int tick = 10; 
    public void run() { 
        Object obj = new Object(); 
        while (true) {
            synchronized(obj) {
                if (tick > 0) {
                    try {
                        Thread.sleep(100);
                    } catch (InterruptedException e) {
                        e.printStackTrace();
                    }
                    System.out.println(Thread.currentThread().getName() + " 卖出第 " + tick-- + " 张票");
                } else {
                    break;
                }
            }
        }
    }
}

为什么synchronized(obj)会失效？ #

你把Object obj = new Object();写在了run()方法内部，这就意味着：

• 每个线程启动后，执行run()时都会创建一个全新的Object实例
• 四个线程各自持有完全不同的锁对象，它们的同步块之间没有任何互斥关系
• 多个线程可以同时进入各自的同步块操作tick变量，自然会出现线程安全问题（比如tick被减到-1、0）

简单说就是：你锁的不是同一个东西，等于没锁！

为什么synchronized(this)能正常工作？ #

当你把同步块改成synchronized(this)时：

• this指向的是当前Ticket1的实例，也就是你在main方法里创建的t对象
• 四个线程都是基于同一个t实例创建的，所以它们争抢的是同一把锁
• 同一时间只有一个线程能进入同步块操作tick，完美保证了线程安全，不会出现超卖的情况

额外优化思路 #

除了用synchronized(this)，你也可以把obj定义成类的成员变量（比如private Object obj = new Object();），这样所有线程共享同一个锁对象，也能达到同步效果哦～

内容的提问来源于stack exchange，提问作者user9443945