java - 这段代码线程安全吗？

Question

我正在阅读 post on JavaCodeGeeks

public User build() {
    User user = new User(this);
    if (user.getAge() > 120) {
        throw new IllegalStateException(“Age out of range”); // thread-safe
    }
    return user;
}

和

public User build() {
    if (age > 120) {
        throw new IllegalStateException(“Age out of range”); // bad, not thread-safe
    }
    // This is the window of opportunity for a second thread to modify the value of age
    return new User(this);
}

我有一个疑问，怎么能说前者的代码是线程安全的而后者不是。

Answer 1

如果两个线程像这样使用相同的 UserBuilder:

    User.UserBuilder builder = new User.UserBuilder("Jhon", "Doe");

    // First thread
    Thread t1 = new Thread(){
    @Override
    public void run()
    {
        builder.age(30).build();         
    }};

    // Second thread
    Thread t2 = new Thread(){
        @Override
        public void run()
        {
            // Just changing age temporarily in builder.
            builder.age(140);
            builder.age(35).build();         
        }};

   t1.start();
   t2.start();

通过 build() 的线程安全实现，您将永远不会拥有年龄超过 120 岁的用户，因为将抛出非法状态异常。

在第二种情况下，当 t1 检查条件时，可能会发生 age == 30 但当 new User(this) 被调用时 age == 140 。在这种情况下，你不会得到任何异常，并且关于年龄的不变性被打破了。这在第一种情况下不会发生。