sound2sd
2016-01-02 15:10
采纳率: 33.3%
浏览 2.2k
已采纳

求助,java多线程的问题,synchronized关键字不起作用

public class VoTest {

private int num=1;

private void increase(){

** synchronized (this){
this.num++;
}**

}

private int read(){
    return this.num;
}

public static void main(String[] args) {

    final VoTest test=new VoTest();
    for (int i = 0; i < 500; i++) {
        new Thread(new Runnable() {
            @Override
            public void run() {
                test.increase();
            }
        }).start();
    }

    //如果还有子线程在运行,主线程就主动让出cpu资源
    //直到所有的子线程都运行结束
    if(Thread.activeCount()>1){
        Thread.yield();
    }

    System.out.println("num="+test.read());
}

}
按理说,synchronized关键字可以保证原子性和可见性,但是
这段代码的结果本应该是500,结果却千奇百怪,不管num加不加volatile关键字
都一样。
结果随便截了几个,求大神解答!
第一次运行结果
第二次运行结果

  • 写回答
  • 关注问题
  • 收藏
  • 邀请回答

4条回答 默认 最新

  • 云聪 2016-01-02 17:06
    已采纳

    正确结果应该是501,尝试不用yield方法,而是用循环判断,当活跃线程数大于1,就等待1秒,直到所有线程都运行完毕

    已采纳该答案
    打赏 评论
  • devmiao 2016-01-02 15:20
    打赏 评论
  • Antway_ 2016-01-03 01:40

    感觉还是锁的问题,锁没成功

    打赏 评论
  • 何铁 2016-01-12 13:42

    main方法其实是一个静态的方法,所以你这里锁的对象出错了,不应该是this而是这个类的字节码对象

    打赏 评论

相关推荐 更多相似问题