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(); 
} 

这可以给你初始化生成器的方式更灵活，甚至可以有几种生成器方法做的不完全一样的东西，在“预”的条款（并且因为它们是方法，所以在命名上比在构造函数上更灵活:)）