[英]Why my synchronized method not working properly?
我有这个打印计数器的同步方法,我有4个线程,所以我期望计数器的最终值为400000,因为我的计数器是一个静态变量。
但是每次我运行代码时,它都会给我不同的counter值。
以下是我的代码:
class MyThread implements Runnable{
private static int counter=1;
@Override
public void run() {
try {
this.syncMethod();
} catch (InterruptedException e) {
e.printStackTrace();
}
}
public synchronized void syncMethod() throws InterruptedException{
for(int i=0;i<100000;i++){
System.out.println(Thread.currentThread().getName()+" : "+counter++);
}
}
}
public class MyController {
public static void main(String[] args) throws InterruptedException {
Runnable r1=new MyThread();
Runnable r2=new MyThread();
Runnable r3=new MyThread();
Runnable r4=new MyThread();
Thread t1;
Thread t2;
Thread t3;
Thread t4;
t1=new Thread(r1,"Thread 1");
t2=new Thread(r2,"Thread 2");
t3=new Thread(r3,"Thread 3");
t4=new Thread(r4,"Thread 4");
t2.start();
t1.start();
t3.start();
t4.start();
}
}
该变量是static
,但是您synchronized
的方法不是static
。 这意味着它将在当前实例上获取监视器,并且每个线程都有一个不同的当前实例。
一个简单的解决方案是使syncMethod
方法也static
。 在这种情况下,它将对由MyThread
类的所有实例共享的监视器进行锁定:
public static synchronized void syncMethod()
Erwin Bolwidt的答案正确解决了您的问题。 作为安全地在多个线程中递增静态共享计数器的另一种方法,可以转到AtomicLong 。
定义如下:
private static AtomicLong counter = new AtomicLong();
递增为:
counter.getAndIncrement();
最后,得到结果:
counter.get();
非静态方法中的同步关键字意味着与此方法完全同步:这两个代码完全等效:
public synchronised void dojob(){
//the job to do
}
等
public void dojob(){
synchronised (this){
//the job to do
}
}
在您的情况下,您的同步方法正在不同的对象(t1,t2,t3和t4)上进行同步,因此不会彼此阻塞。 最好的解决方案是,您的线程将使用一个公共对象彼此同步。 另外一点总是更好的方法是返回其线程以执行此调用,这里是通过此2个修复程序执行所需操作的代码
class MyThread implements Runnable {
public static class JobDoer {
public synchronized void syncMethod() throws InterruptedException {
for (int i = 0; i < 100000; i++) {
System.out.println(Thread.currentThread().getName() + " : " + counter++);
}
}
}
private static int counter = 1;
public MyThread(JobDoer doer) {
this.doer = doer;
}
private JobDoer doer;
@Override
public void run() {
try {
doer.syncMethod();
} catch (InterruptedException e) {
e.printStackTrace();
}
}
public static void main(String[] args) throws InterruptedException {
JobDoer doer = new JobDoer();
Thread t1 = new Thread(new MyThread(doer), "Thread 1");
Thread t2 = new Thread(new MyThread(doer), "Thread 2");
Thread t3 = new Thread(new MyThread(doer), "Thread 3");
Thread t4 = new Thread(new MyThread(doer), "Thread 4");
t2.start();
t1.start();
t3.start();
t4.start();
t1.join();
t2.join();
t3.join();
t4.join();
}
}
声明:本站的技术帖子网页,遵循CC BY-SA 4.0协议,如果您需要转载,请注明本站网址或者原文地址。任何问题请咨询:yoyou2525@163.com.