黑马程序员技术交流社区

标题: java多线程问题 [打印本页]
作者: vipzh    时间: 2013-3-13 22:33
标题: java多线程问题
本帖最后由 张号 于 2013-3-15 12:04 编辑

创建了6个线程，同步后为什么总是只有一个线程在走？
  1. /售票窗口
  2. public class SealWindow
  3. {
  4.         public static void main(String[] args)
  5.         {
  6.                 new Thread(new TicketSealCenter(), "1号窗口").start();
  7.                 new Thread(new TicketSealCenter(), "2号窗口").start();
  8.                 new Thread(new TicketSealCenter(), "3号窗口").start();
  9.                 new Thread(new TicketSealCenter(), "4号窗口").start();
  10.                 new Thread(new TicketSealCenter(), "5号窗口").start();
  11.                 new Thread(new TicketSealCenter(), "6号窗口").start();

  13.         }

  15. }

  17. // 售票中心
  18. class TicketSealCenter implements Runnable
  19. {

  21.         private void sale() throws Exception
  22.         {
  23.                 Ticket ticket = new Ticket();
  24.                 // 售票中心分配火车票
  25.                 ticket.setTicketNum(100);

  27.                 while (ticket.isEmple())
  28.                 {
  29.         
  30.                         System.out.println(Thread.currentThread().getName() + "售出一张"
  31.                                         + "还剩下" + ticket.getTicketNum() + "张票");
  32.                         
  33.                         if (ticket.getTicketNum() == 0)
  34.                         {
  35.                                 System.out.println("票已经卖完了，明天再来吧！！！");
  36.                         }

  38.                 }
  39.         }

  41.         @Override
  42.         public void run()
  43.         {
  44.                 try
  45.                 {
  46.                         synchronized (TicketSealCenter.class)
  47.                         {
  48.                                 
  49.                                 sale();
  50.                         }

  52.                 }
  53.                 catch (Exception e)
  54.                 {
  55.                         e.printStackTrace();
  56.                 }
  57.         }
  58. }

  60. // 火车票类
  61. class Ticket
  62. {

  64.         private int ticketNum; // 火车票张数

  66.         // 获得火车票的数量
  67.         public int getTicketNum()
  68.         {
  69.                 return ticketNum;
  70.         }

  72.         // 设置火车票数量
  73.         public void setTicketNum(int ticketNum)
  74.         {
  75.                 this.ticketNum = ticketNum;
  76.         }

  78.         // 判断是否有票
  79.         public  boolean isEmple() throws Exception
  80.         {

  82.                 if (ticketNum > 0)
  83.                 {

  85.                         ticketNum--;
  86.                         Thread.sleep(1000);
  87.                         return true;
  88.                 }
  89.                 else
  90.                 {
  91.                         System.exit(0);
  92.                         return false;
  93.                 }
  94.         }

  96. }
复制代码
作者: HM赵磊    时间: 2013-3-13 22:46
[ 本帖最后由 csatshell 于 2013-3-13 22:49 编辑 ]\n\n兄弟，你的程序运行后，只有1号窗口买票，原因是你的同步代码块的锁选择的问题 看如下代码（更改你的锁标志）

                                        //售票窗口
public class SealWindow
{
        public static void main(String[] args)
        {
                new Thread(new TicketSealCenter(), "1号窗口").start();
                new Thread(new TicketSealCenter(), "2号窗口").start();
                new Thread(new TicketSealCenter(), "3号窗口").start();
                new Thread(new TicketSealCenter(), "4号窗口").start();
                new Thread(new TicketSealCenter(), "5号窗口").start();
                new Thread(new TicketSealCenter(), "6号窗口").start();        }}// 售票中心
class TicketSealCenter implements Runnable
{        private void sale() throws Exception
        {
                Ticket ticket = new Ticket();
                // 售票中心分配火车票
                ticket.setTicketNum(100);                while (ticket.isEmple())
                {
        
                        System.out.println(Thread.currentThread().getName() + "售出一张"
                                        + "还剩下" + ticket.getTicketNum() + "张票");
                        
                        if (ticket.getTicketNum() == 0)
                        {
                                System.out.println("票已经卖完了，明天再来吧！！！");
                        }                }
        }        //@Override
  Object obj=new Object();//添加的
        public void run()
        {
                try
                {
                        synchronized (obj)
                        {
                                
                                sale();
                        }                }
                catch (Exception e)
                {
                        e.printStackTrace();
                }
        }
}// 火车票类
class Ticket
{        private int ticketNum; // 火车票张数        // 获得火车票的数量
        public int getTicketNum()
        {
                return ticketNum;
        }        // 设置火车票数量
        public void setTicketNum(int ticketNum)
        {
                this.ticketNum = ticketNum;
        }        // 判断是否有票
        public  boolean isEmple() throws Exception
        {                if (ticketNum > 0)
                {                        ticketNum--;
                        Thread.sleep(1);
                        return true;
                }
                else
                {
                        System.exit(0);
                        return false;
                }
        }}
                 

d.jpg (63.36 KB, 下载次数: 25)

d.jpg
作者: 马甲大王    时间: 2013-3-13 22:58
本帖最后由 张栓紧 于 2013-3-13 23:00 编辑
  1. public class SealWindow
  2. {
  3.         public static void main(String[] args)
  4.         {
  5.                 new Thread(new TicketSealCenter(), "1号窗口").start();
  6.                 new Thread(new TicketSealCenter(), "2号窗口").start();
  7.                 new Thread(new TicketSealCenter(), "3号窗口").start();
  8.                 new Thread(new TicketSealCenter(), "4号窗口").start();
  9.                 new Thread(new TicketSealCenter(), "5号窗口").start();
  10.                 new Thread(new TicketSealCenter(), "6号窗口").start();

  12.         }

  14. }

  16. // 售票中心
  17. class TicketSealCenter implements Runnable
  18. {
  19.          
  20.        // 售票中心分配火车票
  21.       

  23.         private void sale() throws Exception
  24.         {
  25.                  Ticket ticket = new Ticket();
  26.                     ticket.setTicketNum(10);

  28.                 while (ticket.isEmple())
  29.                 {
  30.         
  31.                         System.out.println(Thread.currentThread().getName() + "售出一张"
  32.                                         + "还剩下" + ticket.getTicketNum() + "张票");
  33.                         
  34.                         if (ticket.getTicketNum() == 0)
  35.                         {
  36.                                 System.out.println("票已经卖完了，明天再来吧！！！");
  37.                         }

  39.                 }
  40.         }

  42.         @Override
  43.         public void run()
  44.         {
  45.                 try
  46.                 {
  47.                         synchronized (this)
  48.                         {
  49.                                 
  50.                                 sale();
  51.                         }

  53.                 }
  54.                 catch (Exception e)
  55.                 {
  56.                         e.printStackTrace();
  57.                 }
  58.         }
  59. }

  61. // 火车票类
  62. class Ticket
  63. {

  65.         private int ticketNum; // 火车票张数

  67.         // 获得火车票的数量
  68.         public int getTicketNum()
  69.         {
  70.                 return ticketNum;
  71.         }

  73.         // 设置火车票数量
  74.         public void setTicketNum(int ticketNum)
  75.         {
  76.                 this.ticketNum = ticketNum;
  77.         }

  79.         // 判断是否有票
  80.         public  boolean isEmple() throws Exception
  81.         {

  83.                 if (ticketNum > 0)
  84.                 {

  86.                         ticketNum--;
  87.                         Thread.sleep(1000);
  88.                         return true;
  89.                 }
  90.                 else
  91.                 {
  92.                         System.exit(0);
  93.                         return false;
  94.                 }
  95.         }

  97. }
复制代码
你把TicketSealCenter类锁死了，所以就只是一个线程在运行
作者: filter    时间: 2013-3-14 00:44
失眠了，半夜来论坛逛下，我修改了下并测试了下代码，至少是能保证安全问题，肯定是可以优化的，明天有空的话我再来看看，希望别的同学帮着优化下
  1. package com.itheima;

  3. public class Test11
  4. {
  5.         public static void main(String[] args)
  6.         {
  7.                 [color=Red]Ticket1 ticket = new Ticket1(100);//初始化票的数量[/color]               
  8. new Thread(new TicketSealCenter1(ticket), "1号窗口").start();
  9.                 new Thread(new TicketSealCenter1(ticket), "2号窗口").start();
  10.                 new Thread(new TicketSealCenter1(ticket), "3号窗口").start();
  11.                 new Thread(new TicketSealCenter1(ticket), "4号窗口").start();
  12.                 new Thread(new TicketSealCenter1(ticket), "5号窗口").start();
  13.                 new Thread(new TicketSealCenter1(ticket), "6号窗口").start();

  15.         }

  17. }

  19. // 售票中心
  20. class TicketSealCenter1 implements Runnable
  21. {
  22.          
  23.        // 售票中心分配火车票
  24.       
  25.       [color=Red]private Ticket1 ticket;//Ticket对象最好定义在成员位置上[/color]
  26.         //ticket.setTicketNum(100);
  27.         TicketSealCenter1(Ticket1 ticket){
  28.                 this.ticket = ticket;
  29.         }
  30.       
  31.         private void sale() throws Exception
  32.         {
  33.                  //Ticket1 ticket = new Ticket1();
  34.                   // ticket.setTicketNum(100);
  35.                        
  36.                 while (ticket.isEmple())
  37.                 {
  38.                         [color=Red]int count = ticket.getTicketNum();//定义临时变量，记录票的数量[/color]                       
  39.                                         System.out.println(Thread.currentThread().getName()
  40.                                                         + "售出一张" + "还剩下" + count
  41.                                                         + "张票");
  42.                         
  43.                                         if (count == 0) {
  44.                                                 System.out.println("票已经卖完了，明天再来吧！！！");
  45.                                         }
  46.                 }
  47.         }

  49.         @Override
  50.         public void run()
  51.         {
  52.                 try
  53.                 {
  54.                         [color=Red]synchronized (this)//方法的锁最好定义为调用者this[/color]                        {
  55.                                 
  56.                                 sale();
  57.                         }

  59.                 }
  60.                 catch (Exception e)
  61.                 {
  62.                         e.printStackTrace();
  63.                 }
  64.         }
  65. }

  67. // 火车票类
  68. class Ticket1
  69. {

  71.         private int ticketNum = 100; // 火车票张数
  72.         Ticket1(int ticketNum){
  73.                 this.ticketNum = ticketNum;
  74.         }

  76.         // 获得火车票的数量
  77.         [color=Red]public synchronized int getTicketNum()//被多条线程操作的数据，要加同步[/color]        {
  78.                 return --ticketNum;
  79.         }

  81.         // 设置火车票数量
  82.         public void setTicketNum(int ticketNum)
  83.         {
  84.                 this.ticketNum = ticketNum;
  85.         }

  87.         // 判断是否有票
  88.        [color=Red] public synchronized boolean isEmple() throws Exception//被多条线程操作的数据，要加同步[/color]        {

  90.                 if (ticketNum > 0)
  91.                 {

  93.                                
  94.                         Thread.sleep(1000);
  95.                         return true;
  96.                 }
  97.                 else
  98.                 {
  99.                         System.exit(0);
  100.                         return false;
  101.                 }
  102.         }

  104. }
复制代码
作者: filter    时间: 2013-3-14 00:46
我去，论坛还玩儿不转，怎么设置注释的字体啊？？{:soso_e127:}
作者: 李辉    时间: 2013-3-14 02:13
本帖最后由 李辉 于 2013-3-14 02:16 编辑

这里的问题不少啊！
第一点：最重要的也是最根本的问题是对多线程共享资源的不理解！除了李志阳，前几位创建的线程完全没有共享资源，每个线程都独立的拥有100张票（有一个人定义了10张），我要在这里说清楚恐怕得写太多。大家自己先好好学学。
第二点：类名。包含主函数的类才应该叫“TicketSealCenter“ 因为是这个类运行的时候创建了多个线程并且在调度这些线程，每个线程代表一个售票窗口。实现Runnable接口的类应该叫“SealWindow”。类名不单单是个单词，它代表着你对售票模型以及面向对象的理解。
第三点：同步的理解。同步是防止一个线程对共享数据操作一半的时候另一个线程开始操作。前几位都没有共享数据，同步只能起到一个作用：就是降低程序运行速度。
作者: 0.00    时间: 2013-3-14 04:57
表示弄了很久还是没弄好，起床再搞了，困死了 都天亮了，
你代码存在很多问题，正如楼上所说，锁只是锁共享的数据，而数据只有票在变，要锁住票， 还有一个就是 多线程不能像你那样传对象，那样的对象是独立的线程了，
需要同时传同一个对象进去！
作者: filter    时间: 2013-3-14 09:58
楼上说的很到位，给你提供了思路，具体问题还是你自己解决吧，另外我多线程学的也不是很好，最后加的锁是嵌套的死锁，只卖几百张票可能还行，多了就出大问题了
，可能跟设计的方法有关吧
晚上再来看下
作者: 0.00    时间: 2013-3-14 19:20
public class SaleWindow
{
       public static void main(String[] args)
     {
                Thread t1=new Thread(new TicketSealCenter("1号窗口"));
                Thread t2=new Thread(new TicketSealCenter("2号窗口"));
                Thread t3=new Thread(new TicketSealCenter("3号窗口"));
                Thread t4=new Thread(new TicketSealCenter("4号窗口"));
                Thread t5=new Thread(new TicketSealCenter("5号窗口"));
                Thread t6=new Thread(new TicketSealCenter("6号窗口"));
                t1.start();t2.start();t3.start();t4.start();t5.start();t6.start();
               
                System.out.println("关门下班");
        }
}
//新建一个类
class  TicketSealCenter implements Runnable
{      
        private String iname;
       
        private static int inumber=100;
       
        Object o=new Object();
       
        public  void run()
    {
                if(inumber>0)
          { while(true)
                        {
                synchronized (o)
            {
                if(inumber>0)
                        {
                         System.out.println(this.getIname()+"火车票卖出一张,"+"剩余"+--inumber+"张.");
                         try
                            {Thread.sleep(1);}
                         catch (Exception e)
                                {System.out.println("系统异常");}
                        } else
                  break;
                        }
                       
                       
            }
         
         }
          
          }       
                       
       
       
        public String getIname()
          {
        return iname;
          }
        TicketSealCenter(String iname)
          {
         this.iname=iname;
          }
       
}
你试试这个代码
作者: 0.00    时间: 2013-3-14 19:23
还是没锁住，坐等大神出现
作者: 0.00    时间: 2013-3-14 20:34
把锁的对象改为 类名.class，否则不是同一个锁



欢迎光临 黑马程序员技术交流社区 (http://bbs.itheima.com/) 黑马程序员IT技术论坛 X3.2