多线程的问题

[90后北京程序员]

public class Demo{
public static int rocket = 21;
public static void main(String[] args){
MyThread mt1 = new MyThread();
Thread t1 = new Thread(mt1);
MyThread2 t2 = new MyThread2();

t1.start();
t2.start();
}
}
class MyThread implements Runnable{
public synchronized void run(){
while(Demo.rocket>0){
if(Demo.rocket==1){
try{
Thread.sleep(1000);
}catch(Exception e){
e.printStackTrace();
}
}
Demo.rocket--;
System.out.println("1111卖票，剩余:"+Demo.rocket+"张票");

}

}
}
class MyThread2 extends Thread{
public synchronized void run(){

while(Demo.rocket>0){
Demo.rocket--;
System.out.println("22222222卖票，剩余:"+Demo.rocket+"张票........");
}

}
}

看看有啥问题

[dyflovexlp]

加在方法上的锁，其实是锁住了 MyThread1 和MyThread2 的对象，你要锁住Demo这个才行。 都改为：

public   void run(){
 synchronized(Demo.class){
while(Demo.rocket>0){
Demo.rocket--;
System.out.println("22222222卖票，剩余:"+Demo.rocket+"张票........");
}
}

吧

[xuan.ye]

多线程调用一个方法，和两个线程调用两个方法。 LZ写的是两个方法锁吧？

[其实我是真性情]

4楼正解，你要同步的必须是公共资源，出现-1这种情况，是因为MyThread在Demo.rocket=1的时候，进入了Thread.sleep(1000);而1秒钟的等待时间足够MyThread2进行一次运算，当MyThread2进行while(Demo.rocket>0)的判断的时候MyThread还在睡眠当中，然后MyThread2对Demo.rocket进行了-1.这个时候Demo.rocket已经是0了，当MyThread睡眠结束后，又对他进行了-1就出现-1了，所以你这个程序-1应该都是出现在1111卖的票里。

[90后北京程序员]

引用 4 楼 dyflovexlp 的回复:

加在方法上的锁，其实是锁住了 MyThread1 和MyThread2 的对象，你要锁住Demo这个才行。 都改为：

public   void run(){
 synchronized(Demo.class){
while(Demo.rocket>0){
Demo.rocket--;
System.out.println("22222222卖票，剩余:"+Demo.rocket+"张票........");
}
}

吧

这样的话，得到的结果是当1111进入的时候，直到111把所有的票都卖光了，2222222也没出来

[jdgdf566]

差不多了。。。

[90后北京程序员]

什么叫差不多了？

[90后北京程序员]

为啥我加了锁但线程结果Demo.rocket的值还是有可能为-1

[ntwarren]

引用 7 楼 alai_5186_cdsn 的回复:

[quote=引用 4 楼 dyflovexlp 的回复:] 加在方法上的锁，其实是锁住了 MyThread1 和MyThread2 的对象，你要锁住Demo这个才行。 都改为：

public   void run(){
 synchronized(Demo.class){
while(Demo.rocket>0){
Demo.rocket--;
System.out.println("22222222卖票，剩余:"+Demo.rocket+"张票........");
}
}

吧

这样的话，得到的结果是当1111进入的时候，直到111把所有的票都卖光了，2222222也没出来[/quote]

public   void run(){
while(true){
 synchronized(Demo.class){
     if(Demo.rocket>0)
     {
         Demo.rocket--;
         System.out.println("22222222卖票，剩余:"+Demo.rocket+"张票........");
     }
     else
         break;
  }
}
}