多窗口异步售票java纠错

这是服务窗口类，模拟四个线程异步运行

public class TicketWindow {

 public static void main(String args[]) {
  Ticket ticket = new Ticket();
  Thread w1 = new Thread(ticket, "1号售票窗口");
  Thread w2 = new Thread(ticket, "2号售票窗口");
  Thread w3 = new Thread(ticket, "3号售票窗口");
  Thread w4 = new Thread(ticket, "4号售票窗口");
  w1.setPriority(Thread.NORM_PRIORITY);
  
  w2.setPriority(Thread.NORM_PRIORITY);
  w3.setPriority(Thread.NORM_PRIORITY);
  w4.setPriority(Thread.NORM_PRIORITY);
  
w1.start();
w2.start();
w3.start();
w4.start();
 }
}

 

以下是ticket类，控制窗口出票的逻辑

public class Ticket implements Runnable {
 private int ticketNumber = 1;

 public void sellTicket() {

  while (ticketNumber++ < 100) {
   synchronized (this) {

    System.out.println(Thread.currentThread().getName() + "正在售出第"
      + ticketNumber + "张票");

   }

  }
 }

 public void run() {

  sellTicket();

 }

 }

运行类TicketWindow，

部分结果如下：

1号售票窗口正在售出第2张票
2号售票窗口正在售出第5张票
4号售票窗口正在售出第6张票
3号售票窗口正在售出第7张票
3号售票窗口正在售出第8张票
3号售票窗口正在售出第9张票
2号售票窗口正在售出第11张票
4号售票窗口正在售出第11张票

。。。

2号售票窗口正在售出第101张票
1号售票窗口正在售出第102张票
4号售票窗口正在售出第103张票

 

按运行情况来看，存在三个问题：

1.出票不连续----票号有间断

2.票号重复--不同窗口售出重同一票号

3.多卖票--卖出票数超过100

 

以下是正确的代码：

public class Ticket implements Runnable {
 private int ticketNumber = 1;

 

 public void sellTicket() {

  while (ticketNumber < 100) {//票号递增操作应属于临界区，须放在synchronized语句块内 /
   synchronized (this) {
    if (ticketNumber <= 100)//可能存在某线程执行完后，票售完后，仍有其他线程阻塞在while语句块后等待执行的情况，因而可能可能卖出多于100张票，需再次判断票是否售空/
     System.out.println(Thread.currentThread().getName()
       + "正在售出第" + ticketNumber++ + "张票");

   }
  }
 }

 public void run() {

  sellTicket();

 }

 
}