毕老师11天10视频中的代码

1. class Ticket implements Runnable{
   
2.         private int tic=100;
   
3.         
   
4.         public void run(){
   
5.                 while(true){
   
6.                         synchronized (new Object()) {
   
7.                                 if(tic>0){
   
8.                                         try {
   
9.                                                 Thread.sleep(10);//出现-1、-2号票 有安全问题
   
10.                                         } catch (InterruptedException e) {
    
11.                                                 e.printStackTrace();
    
12.                                         }
    
13.                                         System.out.println(Thread.currentThread().getName()+"ticket"+tic--);
    
14.                                 }
    
15.                         }
    
16.                 }
    
17.         }
    
18. }
    
19. public class TicketDemo {
    
20.         public static void main(String[] args) {
    
21. 
    
22.                 Ticket t=new Ticket();
    
23.                 Thread t1=new Thread(t);
    
24.                 Thread t2=new Thread(t);
    
25.                 Thread t3=new Thread(t);
    
26.                 Thread t4=new Thread(t);
    
27.                 t1.start();
    
28.                 t2.start();
    
29.                 t3.start();
    
30.                 t4.start();
    
31.                
    
32.         }
    
33. }

复制代码

输出结果依然有-1，-2号的票  这是怎么回事？

你的锁有问题。
建议把锁定义在外面。
Object obj = new Object();
然后用obj当锁。

你这样子每次循环都会新new出一个锁来。所以是锁不住的。

问题出在 synchronized (new Object()) 这里 ，每次都会new出一个新的对象，即使用的不是同一把锁。应该是
class Ticket implements Runnable{
        ......
        ......
        private Object obj = new Object();
        public void run(){
                while(true){
                        synchronized (obj) {//确保每次使用的都是同一把锁
                        .....
                        .....
                        .....

杨玄文 发表于 2013-4-21 22:50
你的锁有问题。
建议把锁定义在外面。
Object obj = new Object();

哦 现在好了

楼主你好：
你的同步代码块里面的锁有问题，因为你用的是匿名对象，每次判断锁的时候，都是不同的对象。
因为同步代码块中的锁必须是同一个锁并且可以是任意锁，可以是this锁，也可以是.class文件。
所谓的锁也就是synchronized (锁){};中传的参数你可以把对象改为如下所示：

1. class Ticket implements Runnable {
   
2. private int tic = 100;
   
3. private Object obj = new Object();                  //这里定义的代码,就是定义的锁对象
   
4. 
   
5. public void run() {
   
6. while (true) {
   
7. synchronized (obj) {                       //这里就是装的锁对象，其实也可以不用定义锁对象，直接传个任意对象就行，如this或者某类的.class文件都可以，保证锁的唯一
   
8. if (tic > 0) {
   
9. try {
   
10. Thread.sleep(10);// 出现-1、-2号票 有安全问题
    
11. } catch (InterruptedException e) {
    
12. e.printStackTrace();
    
13. }
    
14. System.out.println(Thread.currentThread().getName()
    
15. + "ticket" + tic--);
    
16. }
    
17. }
    
18. }
    
19. }
    
20. }
    
21. 
    
22. public class TicketDemo {
    
23. public static void main(String[] args) {
    
24. 
    
25. Ticket t = new Ticket();
    
26. Thread t1 = new Thread(t);
    
27. Thread t2 = new Thread(t);
    
28. Thread t3 = new Thread(t);
    
29. Thread t4 = new Thread(t);
    
30. t1.start();
    
31. t2.start();
    
32. t3.start();
    
33. t4.start();
    
34. 
    
35. }
    
36. }

复制代码

希望可以帮助楼主。

不能在锁对象那new对象的，这样每次都是新的对象