这段java多线程代码有问题吗？

Question

publicclassMain{
publicstaticvoidmain(String[]args){
MyThreadmt=newMyThread();
newThread(mt).start();
newThread(mt).start();
newThread(mt).start();
}
}
classMyThreadimplementsRunnable{
privateintticket=20;
publicvoidrun(){
for(inti=0;i<20;i++){
if(this.ticket>0){
System.out.println("线程:"+Thread.currentThread().getName()+"卖票：ticket"+this.ticket--);
}
}
}
}

Answer 1

不知道你想要干什么。不过要是买票，可能下面才是对的。publicsynchronizedvoidrun()....

Answer 2

你这个涉及到一个共享变量的问题，ticket变量被三个线程同时访问，就可能导致多个线程读到同样的值，这显然是错误的。而且这只是其中一种情况。更复杂的是--操作并不是原子的，这就会产生另一种情况，如果ticket不是int，而是long，递增／递减是分步进行的，这中间可能另一个线程同时向ticket写值，从而产生更离奇的输出。所以就要对共享变量进行同步控制，synchronized可以用在方法上，也可以以同步块的方式进行。更简单的上使用java.util.concurrent.atomic.AtomicInteger，它提供原子的递增／递减方法。而且，在输出的地方也会出问题。因为字符串的拼接也不是原子的，所以肯出现输出混乱的情况。