java - volatile中i++的原子性问题

Question

/**
 * @create on 17/3/27
 * @description
 */
public class Main {
    static volatile int i = 0;
    public static class PlusTask implements Runnable{
        @Override
        public void run(){
            for(int k=0; k<10000; k++)
                i++;
        }
    }

    public static void main(String[] args) throws InterruptedException{
        Thread[] threads = new Thread[10];
        for(int i=0;i<10;i++){
            threads[i] = new Thread(new PlusTask());
            threads[i].start();
        }

        for(int i=0;i<10;i++){
            threads[i].join();
        }

        System.out.println(i);
    }
}

请教各位大牛 为什么这里的输出总是小于10000？ 已经调用了thread.join

Answer 1

你可能需要这样：

public void run() {
            for (int k = 0; k < 10000; k++)
                synchronized (Main.class) {
                    i++;
                }
        }

或者定义一个AtomicInteger

• volatile只是起一个多线程缓存一致性作用，并不能保证某时刻只有一个线程去写变量。

Answer 2

volitile不保证原子性

Answer 3

多线程操作共享变量问题，volitale不能不保证复合操作的原子性，也就是说后一个线程并不能总是看到前一个线程修改后的值i，解决方法加锁，或者原子操作