对于代码的线程安全问题

Question

import java.util.ArrayList;
import java.util.List;

public class Main {

    // ThreadLocal<Integer> threadLocal = new ThreadLocal<>();
    int i;

    public void increment() {
        i++;
    }
    public static void main(String[] args) {

        Main main = new Main();
        
        Runnable runnable = new Runnable() {
            @Override
            public void run() {
                
                for (int i = 0; i < 10000; i++) {
                    main.increment();
                    System.out.println(main.i);
                }
            }
        };
        List<Thread> threads = new ArrayList<>(10);
        for (int i = 0; i < 3; i++) {
            Thread thread = new Thread(runnable);
            threads.add(thread);
            thread.start();
        }  
        boolean flag = false;
        while (!flag) {
            flag = true;
            for (Thread thread : threads) {
                if (thread.isAlive()) {
                    flag = false;
                }
            }

        }
        System.out.println(main.i);

    }
}

对于这个for循环中的System.out.println(main.i);
加上了这句话之后，每次输出的结果都是30000，这是为什么

Answer 1

不是线程安全是一定的，使用30,300个线程试试就知道了，让你产生安全的错觉在System.out.println是一个耗时的同步方法，就是它在很大程度上掩盖了i++的可见性和原子性的问题。使用AtomicInteger或synchronized吧

Answer 2

开始以为是
public void println(int x) {
        synchronized (this) {
            print(x);
            newLine();
        }
}

后来发现我错了
方法本身的synchronized和被调用的方法的synchronized
确实是线程不安全的
跑300的结果
2999915