我正在实施Builder模式吗？

Question

我有一个编程任务，我必须创建一个需要很多步骤的对象。 我决定申请Builder。我做得对吗？

我复杂的类（在模式的产品）

public class ProductExchangePlan 
{ 
    #region FixMembers 

    [Display(Name="Ütköző")] 
    public Bumper Bumper { set; get; } 

    [Display(Name = "Behelyezőnyomás")] 
    public float InsertionPressure { set; get; } 

    [Display(Name = "Magazin")] 
    public Magazin Magazin { set; get; } 

    #endregion 

    #region SimpleTags 

    [Display(Name = "Programszám")] 
    public string ProgramNumber { set; get; } 

    [Display(Name = "Kemény és puha minta")] 
    public string HardAndSoftSample { set; get; } 

    #endregion 

    [Display(Name = "Csaphossz")] 
    public float TapLength { set; get; } 

    [Display(Name = "Beigazító fej")] 
    public AdjustmentHead AdjustmentHead { set; get; } 

    [Display(Name = "Forgásirány")] 
    public RotationDirection RotationDirection { set; get; } 

    [Display(Name = "Vizsgálótüske")] 
    public TestPin TestPin { set; get; } 

    [Display(Name = "Betétdarab")] 
    public InsertionPiece InsertionPiece { set; get; } 

    [Display(Name = "Görgők behelyezése")] 
    public Station RollerInsertionStation { set; get; } 

    [Display(Name = "Nyomórugók behlyezése")] 
    public Station SpringInsertionStation { set; get; } 

    [Display(Name = "Csereütköző")] 
    public ExchangeBumper ExchangeBumper { set; get; } 

    [Display(Name="Zsírzófej")] 
    public GreasingHead GreasingHead { set; get; } 

    [Display(Name = "Zsírzóventilátor")] 
    public GreasingVentilator GreasingVentilator { set; get; } 
} 

而这些都是实施的其他部分：

/// <summary> 
/// The 'Builder' interface 
/// </summary> 
interface IProductExchangePlanBuilder 
{ 
    ExchangePlanGeneratorViewModel Input { set; get; } 
    ProductExchangePlan ProductExchangePlan { set; get; } 

    void BuildFixMembers(); 
    void BuildSimpleTags(); 
    void BuildTapLength(); 
    void BuildAdjustmentHead(); 
    void BuildRotationDirection(); 
    void BuildTestPin(); 
    void BuildInsertionPiece(); 
    void BuildRollerInsertionStation(); 
    void BuildSpringInsertionStation(); 
    void BuildExchangeBumper(); 
    void BuildGreasingHead(); 
    void BuildGreasingVentilator(); 
} 

/// <summary> 
/// The concrete builder 
/// </summary> 
public class ProductExchangePlanBuilder : IProductExchangePlanBuilder 
{ 
    public ProductExchangePlanBuilder(ExchangePlanGeneratorViewModel input) 
    { 
     this.Input = input; 
     this.ProductExchangePlan = new ProductExchangePlan(); 
    } 

    public ExchangePlanGeneratorViewModel Input 
    { 
     get 
     { 
      ... 
     } 
     set 
     { 
      ... 
     } 
    } 

    public ProductExchangePlan ProductExchangePlan 
    { 
     get 
     { 
      ... 
     } 
     set 
     { 
      ... 
     } 
    } 

    public void BuildFixMembers() 
    { 
     ... 
    } 

    public void BuildSimpleTags() 
    { 
     ... 
    } 

    public void BuildTapLength() 
    { 
     ... 
    } 

    public void BuildAdjustmentHead() 
    { 
     ... 
    } 

    public void BuildRotationDirection() 
    { 
     ... 
    } 

    public void BuildTestPin() 
    { 
     ... 
    } 

    public void BuildInsertionPiece() 
    { 
     ... 
    } 

    public void BuildRollerInsertionStation() 
    { 
     ... 
    } 

    public void BuildSpringInsertionStation() 
    { 
     ... 
    } 

    public void BuildExchangeBumper() 
    { 
     ... 
    } 

    public void BuildGreasingHead() 
    { 
     ... 
    } 

    public void BuildGreasingVentilator() 
    { 
     ... 
    } 
} 

/// <summary> 
/// The Director 
/// </summary> 
public class ProductExchangePlanDirector 
{ 
    public void Construct(IProductExchangePlanBuilder builder) 
    { 
     builder.BuildAdjustmentHead(); 
     builder.BuildExchangeBumper(); 
     builder.BuildFixMembers(); 
     builder.BuildGreasingHead(); 
     builder.BuildGreasingVentilator(); 
     builder.BuildInsertionPiece(); 
     builder.BuildRollerInsertionStation(); 
     builder.BuildRotationDirection(); 
     builder.BuildSimpleTags(); 
     builder.BuildSpringInsertionStation(); 
     builder.BuildTapLength(); 
     builder.BuildTestPin(); 
    } 
} 

所有的BuildXX方法使用的的ExchangePlanGeneratorViewModel Input财产具体的建设者，并要求在数据库中查询。

所以我的问题是，这是一个很好的执行这项任务？你怎么能改进它？

Answer 1

这里有3个问题，你的设计，你应该看看：

1. 你建设者的客户端（ProductExchangePlanDirector）不 以增量方式配置建设者所以没有 需要IProductExchangePlanBuilder到成为这样一个健谈的界面。它很容易忘记在客户端代码中调用所需的 Build...方法之一。

2. 您的构建器类使用公共访问器公开通过属性 构建的对象。这意味着客户端代码可以检索ProductExchangePlan的部分完成的实例，这是构建器模式应该解决的问题中的一个 。 IProductExchangePlanBuilder应改为有Build方法 将返回的 ProductExchangePlan完整构建和有效的实例，或者抛出一个异常，如果不是全部需要 数据是从它的 ExchangePlanGeneratorViewModel实例。

3. 在你的设计，ProductExchangePlanBuilder似乎 负责从视图模型都构建ProductExchangePlan的 依赖关系，构建从自身 ProductExchangePlan想必一些验证 （除非视图模型分析用户输入和构建 依赖关系本身）。如果是这样的话，你可能要进一步 独立的这些责任，由具有从视图模型 ProductExchangePlanDirector解析输入 构建依赖关系，并呼吁 Build它之前配置的建设者。