java - 为什么这段代码不是线程安全的？

Question

在下面的代码片段中，将 doThings() 方法声明为静态方法将使类线程安全。这是因为如果启动了多个 TestSeven 线程，并且由于 x 是静态变量，可能会出现竞争条件吗？

public class TestSeven extends Thread{

    private static int x;

    public synchronized void doThings(){
        int current = x;
        current++;
        x = current;
    }

    public void run(){
        doThings();
    }

    public static void main(String args[]){
        TestSeven t = new TestSeven();
        Thread thread = new Thread(t);
        thread.start();
    }
}

Answer 1

是的，没错。 doThings 的同步 性质只会阻止它被多个线程在同一实例 同时调用。变量 x 是在全局的基础上共享的，而不是在每个实例的基础上共享的，因此它是不安全的。

在现实世界中，可以把它想象成一个有几扇门的浴室——有人可以打开一扇门然后锁上它，但这并不能阻止其他人从另一扇门进来...