A股上市公司传智教育(股票代码 003032)旗下技术交流社区北京昌平校区

 找回密码
 加入黑马

QQ登录

只需一步,快速开始

© 张云杰 高级黑马   /  2013-6-16 22:43  /  2846 人查看  /  14 人回复  /   0 人收藏 转载请遵从CC协议 禁止商业使用本文

/*
* 编写三各类Ticket、SealWindow、TicketSealCenter分别代表票信息、售票窗口、售票中心。
* 售票中心分配一定数量的票,由若干个售票窗口进行出售,利用你所学的线程知识来模拟此售票过程
*/
public class TicketSealCenter {
/*
  * 售票中心 分配售票窗口 和总票数
  */

public static void main(String[] args) {
  Ticket ticket = new Ticket(100);//初始化票数
  
  /*
   * 4个售票窗口
   */
  SealWindow sw = new SealWindow("售票窗口1",ticket);
  SealWindow sw1 = new SealWindow("售票窗口2",ticket);
  SealWindow sw2 = new SealWindow("售票窗口3",ticket);
  SealWindow sw3 = new SealWindow("售票窗口4",ticket);
  
  new Thread(sw).start();
  new Thread(sw1).start();
  new Thread(sw2).start();
  new Thread(sw3).start();
  
  
  
}
}
class SealWindow implements Runnable{
/*
  * 买票,线程开启后 这里开始做事,那么这里就必须要实现RUNNERBLE接口
  */
private String selwindowname;
private Ticket ticket;
private boolean flag=true;
/*
  * 初始化窗口 给了票的数量 以及 窗口名字
  */
public SealWindow(String selwindowname, Ticket ticket) {
  this.selwindowname = selwindowname;
  this.ticket = ticket;
}
@Override
public void run() {
  sl();
}
private synchronized void sl(){
  while(flag){
   System.out.println(selwindowname+"...."+ticket.sellTicket());
   if(ticket.sellTicket()==0){
    flag = false;
    try {
     Thread.sleep(100);
    } catch (InterruptedException e) {
    }
   }
  }
}
}
class Ticket {
private int ticketnum;
public Ticket(int ticketnum) {
  this.ticketnum = ticketnum;
}
/*
  * 卖票
  */

public synchronized int sellTicket(){
  if(ticketnum>0){
   return  ticketnum--;
  }else{
   return 0;
  }
}

}

应该是线程哪里的不同步造成的,  用了synchronized , 照理来说 线程1进去了 线程2不会进才对.. 但是 ticketnum-- 还是被4条线程同时做了计算, 不知道是什么情况 各种求解

评分

参与人数 1技术分 +1 收起 理由
孙百鑫 + 1 神马都是浮云

查看全部评分

14 个回复

倒序浏览
回复 使用道具 举报
遇0判断已经可以了.. 就是计算的时候 一直被线程给干了.. 求解
回复 使用道具 举报
SealWindow sw = new SealWindow("售票窗口1",ticket);
SealWindow sw1 = new SealWindow("售票窗口2",ticket);
SealWindow sw2 = new SealWindow("售票窗口3",ticket);
SealWindow sw3 = new SealWindow("售票窗口4",ticket);
楼主创建的貌似是4个线程参数的对象,这样再通过run方法调用sl()方法时,虽然加同步,但是同步函数的锁是this,也就是当前调用方法的对象,4个线程对应了4把锁,相当于各自都在用自己的锁去操作共有数据ticket,所以循环主体是分别被4个线程一起执行的,而没起到同步的作用。
回复 使用道具 举报
再补充一下,把楼主的程序中的同步函数,改写成同步代码块的形式,用一个Object类对象作为锁的话,楼主应该就能看出问题了。就是4个对象各用一个锁和4个对象公用一个锁的区别。
回复 使用道具 举报
王磊 发表于 2013-6-16 23:42
SealWindow sw = new SealWindow("售票窗口1",ticket);
SealWindow sw1 = new SealWindow("售票窗口2",tick ...

请问兄弟 那我应该怎么加? 是在while内 加个sybchronized(obj)   ???
回复 使用道具 举报
private void sl(){
                synchronized (obj) {
                int i = ticket.sellTicket();
                System.out.println(selwindowname + "...." + i);
                if (ticket.sellTicket() == 0) {
                        flag = false;
                        try {
                                Thread.sleep(1000);
                        } catch (InterruptedException e) {
                        }
                }
                }
        }






        Object obj = new Object();
        public int sellTicket() {
                synchronized (obj) {
                if (this.ticketnum > 0) {
                        return ticketnum--;
                } else {
                        return 0;
                }
                }
        }




2个分别这样加了 没效果
回复 使用道具 举报
王磊 发表于 2013-6-16 23:49
再补充一下,把楼主的程序中的同步函数,改写成同步代码块的形式,用一个Object类对象作为锁的话,楼主应该 ...

我现在纠结的是 共享数据加了 结果不同步 让我头疼.. 你说的方法 我都试过了 无效啊
回复 使用道具 举报
王磊 高级黑马 2013-6-17 00:11:21
9#
楼主这样试试,改成同步代码块,并且SealWindow类中创建这样一个Object对象private static final Object obj = new Object();

就是保证锁的唯一性。再试试
回复 使用道具 举报
貌似不用加final也可以。我这么做只是想给楼主说明一下问题,因为这么写完以后,虽然可以保证同步时锁的唯一性,但是其他线程也同样进了sl()方法,当有执行权的线程把循环运行结束后,其他线程也会在判断一次循环。就相当于把0号票又卖了3次。
所以,这个代码最优化的方法,应该是创建线程参数对象时,只创建一个,这样就能保证4个线程操作同一个对象,而不是4个线程去操作4个对象中的相同数据。
如果楼主想给线程定义名字的话,可以在创建线程对象时定义,而不是在创建线程参数对象时定义。
回复 使用道具 举报
王磊 发表于 2013-6-17 00:20
貌似不用加final也可以。我这么做只是想给楼主说明一下问题,因为这么写完以后,虽然可以保证同步时锁的唯 ...

哦 其实这里对象不能加锁.. 如果加了一样的锁 有可能线程1执行2次 打印2次, 线程2 就执行不到了.. 就没的打印, 然后下一轮的线程又开始抢, 所以这里只能用 函数同步
回复 使用道具 举报
张云杰 发表于 2013-6-17 00:40
哦 其实这里对象不能加锁.. 如果加了一样的锁 有可能线程1执行2次 打印2次, 线程2 就执行不到了.. 就没的 ...

其实就像老师讲的那样,同步前提就是要保证使用的是同一个锁,如果每个线程都有自己的锁,那么虽然都是操作共有的数据,但是还是没有一个互相约束的作用,就等于各自只完成了自己的同步,就算在同步过程中失去执行权,其他线程也只要判断自己的锁就可以了。个人感觉楼主的这个代码如果用同步函数的话,还是不能保证同步。因为都用了this做锁,而创建了4个对象,就相当于每个线程都有了自己的this,这样就是4把锁,不能对共有数据同步了。

评分

参与人数 1技术分 +1 收起 理由
孙百鑫 + 1 很用心

查看全部评分

回复 使用道具 举报
版主可在???  改成已解决,  以上代码就没有错,, 这样做就行了.. 至于同步已经有同步了.. 就按照上面做的, 我刚又重新看了一遍毕老师的卖票,  老毕说上面的现象是因为CPU造成,  也感谢下王磊兄, 这题加synchronized 跟加对象锁是一样的, 有同步, 关键得看线程是同步在哪个地方..
回复 使用道具 举报
说错了.. 这个是错的.. 我刚自己又重新做了一遍没错... 这个代码有误.. 知道原因不.. 呵呵, 是因为 我用这个方法又判断了一次0所造成的, 害我纠结了一个晚上 + 一个凌晨 突然顿悟了
回复 使用道具 举报
粗心害死人啊... 兄弟们 千万别粗心啊!!!
回复 使用道具 举报
您需要登录后才可以回帖 登录 | 加入黑马