关于多线程中synchronized的使用位置

在练习多线程之售票的时候，对于synchronized的使用有些困惑，因为我是用以下代码：

public class ThreadDemo5 {

/**
* @param args
*/
public static void main(String[] args) {
// TODO Auto-generated method stub
saleTicket_5 st = new saleTicket_5();
Thread t1 = new Thread(st);
Thread t2 = new Thread(st);
Thread t3 = new Thread(st);
Thread t4 = new Thread(st);
t1.start();
t2.start();
t3.start();
t4.start();
}

}

class saleTicket_5 implements Runnable {
private int ticket = 1000;
Object obj = new Object();

public void run() {
synchronized (obj) {

while (ticket > 0) {
try {
Thread.sleep(1000);
} catch (InterruptedException e) {
// TODO Auto-generated catch block
e.printStackTrace();
}
System.out.println(Thread.currentThread().getName() + "****"
+ ticket--);
}
}
}
}

synchronized在此虽然将共享资源的代码块上锁，但是synchronized上锁之后，线程需要把synchronized里面的代码块执行完毕后才能释放锁，此代码块里面是一个循环，这就造成了最后的结果只有Thread-0线程在执行的情况。

之后对其进行修改：

public class ThreadDemo5 {

/**
* @param args
*/
public static void main(String[] args) {
// TODO Auto-generated method stub
saleTicket_5 st = new saleTicket_5();
Thread t1 = new Thread(st);
Thread t2 = new Thread(st);
Thread t3 = new Thread(st);
Thread t4 = new Thread(st);
t1.start();
t2.start();
t3.start();
t4.start();
}

}

class saleTicket_5 implements Runnable {
private int ticket = 1000;
Object obj = new Object();

public void run() {

while (true) {
synchronized (obj) {
if (ticket > 0) {

try {
Thread.sleep(10);
} catch (InterruptedException e) {
// TODO Auto-generated catch block
e.printStackTrace();
}
System.out.println(Thread.currentThread().getName()
+ "****" + ticket--);
} else
break;
}

}
}
}


需要注意的是synchronized里面需要包含全部的共享资源，以保证对共享资源操作的原子性，另外 不要忘记退出while循环