Java的嵌套生成器模式重複字段

Question

，我已經遇到網上通常有這樣的事情嵌套生成器模式：

class Person{ 

    private int id; 
    private String name; 
    private int age; 
    ... so on 

    private Person(Builder builder){ 
     this.id = builder.id; 
     this.name = builder.name; 
     this.age = builder.age; 
    } 

    public static class Builder{ 

     private int id; 
     private String name; 
     private int age; 
     ... so on 

     public Builder id(int id){ 
      this.id = id; 
      return this; 
     } 
     public Builder name(String name){ 
      this.name = name; 
      return this; 
     } 
     .... so on 

     public Person build(){ 
      return new Person(this); 
     } 

    } 

} 

我的問題是，是否有必要在複製和Person領域Builder？它看起來像很多冗餘代碼。而我的第二個問題是，下面的代碼是否會成爲一個可行的替代品，爲什麼？爲什麼？

class Person{ 

    private int id; 
    private String name; 
    private int age; 
    ... so on 

    private Person(){} 

    public static class Builder{ 

     private Person person = new Person(); 

     public Builder id(int id){ 
      this.person.id = id; 
      return this; 
     } 
     public Builder name(String name){ 
      this.person.name = name; 
      return this; 
     } 
     .... so on 

     public Person build(){ 
      return person; 
     } 
     // UPDATED -- another build method 
     public Person build(){ 
      Person built = this.person; 
      this.person = new Person(); 
      return built; 
     } 

    } 

} 

注：我理解這個話題可能剛愎自用，有可能不是一個「正確」的答案，但我只是想聽到不同的想法和意見。我不是在尋找ultimate truth。

Answer 1

您的代碼就可以了，只要：

1. 你把你的個人成員變量私人（你這樣做）
2. 你不提供，讓這些成員變量的修飾方法（在你顯示的代碼不會這樣做，但是你省略了部分代碼）
3. 這些成員變量是不可變的，或者你確保getter提供它們的副本。通常更好的是成員已經是不可變的了（提示：甚至java集合）。否則您將在每個getX調用中創建實例。
4. 一旦調用了Builder.build，沒有人必須能夠修改Person實例狀態，甚至不需要Builder本身。 這不會發生在您發佈的代碼中
5. 構建器不公開正在構建的「時態實例」（如果有的話）。沒有一個實例必須暴露在構建方法的返回之外。

有關於哪種方式是首選方式或大多數時候不喜歡味道的意見。但就正確與否而言，這種方法在經過一些修改後會很好。最後，調用構建之前發生的事情純粹是Builder的內部。這是一個實施問題。重要的是以前的規則得到滿足。

要修復規則4：您的Builder.build方法應返回正在使用的臨時實例的深度克隆（有多種方法可以無需指定每個字段）。或者，您應該在構建器中有一個標誌，禁止在構建實例中調用任何其他方法，一旦構建被調用。

附註：我通常更喜歡Builder類也使用私有構造函數。我想有這樣的Person類：

public static Builder builder() { 
    return new Builder(); 
} 

這可以給你初始化生成器的方式更靈活，甚至可以有幾種生成器方法做的不完全一樣的東西，在「預」的條款（並且因爲它們是方法，所以在命名上比在構造函數上更靈活:)）