在正確的地方使用正確的建造者

Question

我對建築師模式非常感興趣，我經常使用它，但我不確定我製造的建造者是否足夠好，並且我也懷疑我可以使用它們的所有環境。 這是我如何創建一個建設者的例子：

public class Person { 

    private String name; 
    private String secondName; 
    private int age; 

    public static class Builder { 
     private boolean isBuilt; 
     private Person person = new Person(); 

     private void check() { 
      if (isBuilt) { 
       throw new IllegalStateException(
         "The object cannot be modified after built"); 
      } 
     } 

     public Builder withName(String name) { 
      check(); 
      person.name = name; 
      return this; 
     } 

     public Builder withSecondName(String secondName) { 
      check(); 
      person.secondName = secondName; 
      return this; 
     } 

     public Builder withAge(int age) { 
      check(); 
      person.age = age; 
      return this; 
     } 

     public Person build() { 
      check(); 
      isBuilt = true; 
      return person; 
     } 
    } 

    @Override 
    public String toString() { 
     return "Name: " + name + "\nSecond name:" + secondName + "\nAge:" + age; 
    } 
} 

只是一個快速的使用例子：

Person person = new Person.Builder() 
     .withName("John") 
     .withSecondName("Smith") 
     .withAge(50) 
     .build(); 
     System.out.println(person); 

這裏我的一些疑惑：

• 你是否認爲這是真的不可變？如果不是，我該如何改進？
• 關於線程安全。那麼這可能是我的主要疑問。這真的是線程安全的嗎？ 我在互聯網上看到一些例子，說類級變量必須是最終的，並通過構造函數傳遞。我還看到一個例子，其中變量被聲明爲volatile。你怎麼看待這件事？
• 您認爲這個構建器在可以使用的場景方面會有什麼限制嗎？我的意思是適合在EJB，JSF支持bean，MDB中調用，或者成爲JPA實體...？

Answer 1

你認爲這是真的不可變的嗎？ [...]這真的是線程安全的嗎？

您的代碼的任何部分都是不可變的。這也可能妨礙線程安全;這表示以二進制方式聲明一個類是或不是線程安全的真的很難。我也不明白你爲什麼會首先在線程之間共享builder實例，但我可能會誤解你的代碼示例的簡單性。

爲了更容易地實現線程安全，您的應該是不可變的。這意味着每個withXXX()方法都應該返回一個代表新狀態的新構建器。 （有可能更聰明的方法做這個，但這將是直接的方法。）

只是爲了重申，雖然：我不知道它是絕對必要使建設者線程安全 - 大多數時間他們是具有非常短的生命週期和可視範圍的對象。無論你想讓它們不可變都取決於用例，你可能想要存儲部分填充的構建器，但這也很少見。 （主觀上它確實然而似乎對他的名字與with開始不修改就地對象的方法更爲直觀，而不是一個名字以set開始。）

你認爲這建設者將有關於可以使用的場景的任何限制？

這通常是無法回答的，但如果你讓你的Person對象不變，並通過您的建設者因此只構造的，他們將不能用作JPA實體，我的猜測是JSF支持豆類爲好。爲您創建/管理某些對象的Java框架通常比不指望它們是JavaBeans，這意味着可以通過反射調用無參數構造函數和屬性設置器來創建這些對象。