4

所以我有一个带有 Hybris、Spring 等的网络项目。

我有一些类是自动生成的。假设我有一个模型类，它是自动生成的，并且从另一个类继承了一些方法来设置字段。

在编写单元测试时，开始使用 Builder 模式有用吗？因为问题是，我没有构造函数，比如 Employee(int id, String name) 等等，我只有继承的方法来设置它们（setId(int id) 等等）。

因此，例如，当我为此模型编写 Builder 类时，我将拥有方法 .withId(int id) 和 .withName(String name) 以及 build() 方法，我将在其中运行 setter 方法。

所以最后在我的测试课上我会有：

EmployeeBuilder eb = new EmployeeBuilder();
Employee emp = eb.withId(123)
                 .withName("John")
                 .build();

但是因为我已经有了 Setter-Methods 我通常有：

Employee emp = new Employee();
emp.setId(123);
emp.setName("John");

那么在这种情况下真的值得付出努力吗？或者有什么我没有真正理解的东西？

谢谢！

4

4 回答 4

6

在回答您的问题之前，我想解释一下构建器模式。

当您有很多重载的构造函数（伸缩构造函数反模式）时，通常会使用构建器模式。例如

public class Employee {

   public Employee(String firstName, String lastName){
       ...
   }

   public Employee(String firstName, String lastName, Sex sex){
       ...
   }


   public Employee(String firstName, String lastName, String salutation) {
       ...
   }
}

在这种情况下，客户端代码必须根据它拥有的数据来决定调用哪个构造函数。如果它有 afirstName并且lastName它必须调用new Employee(firstName, lastName). 如果它只有一个firstName它必须调用Employee(String firstName). 所以客户端代码可能有很多if/then/else。例如

Employee employee = null;
if(firstName != null && lastName != null && sex != null){
    employee = new Employee(firstName, lastName, sex);
} else if(firstName != null && lastName != null && salutation != null){
    employee = new Employee(firstName, lastName, salutation );
} else {
  .....
}

Employee这个例子中的类的设计包括它firstName并且lastName是 的强制属性Employee，因为每个构造函数都需要它们。属性sex和saluation是可选的。如果客户端代码决定调用哪个构造函数，这也意味着决策过程在客户端代码中重复。例如，如果客户端知道firstName,和它应该调用哪个构造函数？要么要么?lastNamesexsalutationnew Employee(firstName, lastName, sex)new Employee(firstName, lastName, saluation)

为了封装构造函数解析，您可能需要使用构建器模式。

public class EmployeeBuilder {

      public EmployeeBuilder(String firstName, String lastName){

      }

      public void setSex(Sex sex){ ... }

      public void setSalutation(Salutation salutation){ ... }

      public Employee build(){
          if(salutation != null){
             return new Emplyoee(firstName, lastName, salutation);
          } else if(sex != null){
             return new Emplyoee(firstName, lastName, sex); 
          } else {
             return new Emplyoee(firstName, lastName);
          }
      }
}

这使得客户端代码更易于阅读，并且封装了构造函数调用决策。例如

EmployeeBuidler employeeBuilder = new EmployeeBuilder(firstName, lastName);

Sex sex = ...; 
String salutation = ...;

employeeBuilder.setSex(sex);
employeeBuilder.setSalutation(salutation);

Employee employee = employeeBuilder.build();

回到你的问题

那么在这种情况下真的值得付出努力吗？

对于您的单元测试，您可能希望创建Employee具有某些属性的对象，而其他属性应设置为默认值。在这种情况下，我认为使用构建器模式是个好主意。然后我会命名构建器，例如EmployeeDefaultValuesBuilder以使其清楚。

您可能还想Employee基于其他员工对象（模板）构建 s。在这种情况下，我会向EmployeeBuilder. 例如

public EmployeeBuilder(Employee template){
  // initialize this builder with the values of the template
}

因此，如果您封装构造逻辑或增加可读性，那么值得付出努力。

于 2015-10-20T08:49:17.067 回答
4

构建器模式适用于：

  • 不可变的类，这里不是这种情况。
  • 当您需要构建许多具有细微差异的相同事物时。这也不是这里的情况。
  • 编写“流利的”API。
  • 当您有一个需要复杂构建的复杂对象时。

那么在这种情况下真的值得付出努力吗？

鉴于您发布的内容，我会说不。

最后，当使用正确的 API（例如Project Lombok或Google Auto ）时，所涉及的工作量很小。（另外，如果您使用构建器来隐藏伸缩构造函数反模式，我认为您正在滥用该模式，但是嘿......）

于 2015-10-20T09:17:22.483 回答
2

构建器模式在两种情况下很有用：

  • 结果对象是不可变的（所有字段都是final） - 构建器比具有许多参数的构造器更好。
  • 您要确保创建的对象是有效的并且不能创建不一致的对象 - 例如，如果设置了longitude字段但未设置，您可以从 build() 方法抛出错误latitude。
于 2015-10-20T08:51:32.190 回答
1

正如您在代码示例中演示的那样，使用构建器模式，您只需节省一些时间来写出您的变量 name emp，但还必须添加最终build()调用或类似内容。

在我的书中，这肯定不会为创建更多建设者的投资带来回报。

但 ...

也许您有需要填写的字段，但这与您要测试的内容确实无关。

或者您想创建多个仅在少数属性上有所不同的实例。

这些可以在构建器中很好地构建，节省大量代码行，更重要的是，使您的测试更加清晰。

于 2015-10-20T07:19:07.363 回答