Java多线程模拟售票问题

package cn.zxyy.multiThread.saleTicket;

public class SaleTicketTest {
    public static void main(String[] args) {
        SaleTicket st = new SaleTicket();
        Thread t1 = new Thread(st,"A");
        Thread t2 = new Thread(st,"B");
        Thread t3 = new Thread(st,"C");
        Thread t4 = new Thread(st,"D");

        t1.start();
        t2.start();
        t3.start();
        t4.start();
    }
}

package cn.zxyy.multiThread.saleTicket;

public class SaleTicket extends Thread {
    private static int ticket = 100;

    public SaleTicket(){

    }

    public SaleTicket(String name){
        super();
        this.setName(name);
    }

    @Override
    synchronized public void run() {
        super.run();

        while(true){
            if(ticket>0){
                try {
                    Thread.sleep(100);
                } catch (InterruptedException e) {
                    e.printStackTrace();
                }
                System.out.println("由"+Thread.currentThread().getName()+"号售票员卖第"+ticket+"张票");
                ticket--;
            }else {
                break;
            }
        }
    }
}

问一问，为什么所有的票全部由线程A在售出，是程序有问题吗？

synchronized
这个方法是同步的

所以它在执行的时候，别的方法不能执行
而它一个方法就是一个循环，一直把票减光了。

你要多个线程同时卖，需要去掉synchronized
而在while循环里，对每次卖票（判断ticket>0，ticket--）单独同步

由于卖票方法上锁了，在A线程执行的时候，别的线程是无法执行的，因为被锁了，
只有当 while 循环结束之后才会释放锁，然后B线程在执行，但是B线程执行的时候票已经被A线程卖完了

因为你的run方法上锁了，而run方法里的循环是一直执行直到票卖完了才释放锁
所以第一个执行run方法的A会一直执行卖票直到票卖完了才释放锁，而此时已经没有票了，所以后面的BCD都不会执行

卖票方法单独加锁，没毛病；直接while加锁有毛病

@Override
    public void run() {
        super.run();

        while (true) {
            if (Sale() == false)
                break;
        }
    }

    synchronized public boolean Sale() {
        if (ticket > 0) {
            try {
                Thread.sleep(100);
            } catch (InterruptedException e) {
                e.printStackTrace();
            }
            System.out.println("由" + Thread.currentThread().getName() + "号售票员卖第" + ticket + "张票");
            ticket--;
            return true;
        } else {
            return false;
        }
    }