多线程阿里笔试题1

题目

下面addOne()函数是否线程安全,请回答原因

public class Test {

   private volatile int value = 0;

   void addOne(){
       ++value;
   }
}

测试代码

   public static void main(String[] args) throws InterruptedException {

        Test test = new Test();

        for (int i = 0; i < 100; i++){
            new Thread(()->{
                try {
                    Thread.sleep(10);
                } catch (InterruptedException e) {
                    e.printStackTrace();
                }
                test.addOne();
            }).start();
        }
        
        Thread.sleep(5000);
        System.out.println(test.getValue());
    }

测试结果

通过以上代码进行测试,每次的结果都不同,如果线程是安全的,那么最终的打印结果应该是100.
所以说addOne()函数线程不安全.

分析

volatile 关键字只能保证数据被一个线程修改之后,及时的对其它线程可见,addOne方法没有加同步锁,当两个线程并行执行的时候,无法保证数据的原子性,