2

我正在使用构建器模式构建一个 JavaBean（只有字段和 getter/setter）。

为了这个例子，假设这是我们的 bean：

public class Pizza {
  private int size;
  private boolean cheese;
  private boolean pepperoni;
  private boolean bacon;

  private Pizza(Builder builder) {
    size = builder.size;
    cheese = builder.cheese;
    pepperoni = builder.pepperoni;
    bacon = builder.bacon;
  }

  public static class Builder {
    //required
    private final int size;

    //optional
    private boolean cheese = false;
    private boolean pepperoni = false;
    private boolean bacon = false;

    public Builder(int size) {
      this.size = size;
    }

    public Builder cheese(boolean value) {
      cheese = value;
      return this;
    }

    public Builder pepperoni(boolean value) {
      pepperoni = value;
      return this;
    }

    public Builder bacon(boolean value) {
      bacon = value;
      return this;
    }

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

取自这里。

现在我一直在尝试确保其中的所有字段Pizza都是非空的，通过反射，遍历字段Pizza并检查它们不为空，但看起来（我在这里可能是错的）我的字段是'在检查发生之前不设置。Jon Skeet 的这段代码是我为了检查我的字段的非空性而改变的（而不是计数，我抛出异常）。

然后我尝试检查构建器的字段，但构建器中有额外的字段（例如，我有一个可能为空的 XMLParser 字段）。通过比萨字段对构建器字段进行子集化不起作用，因为它们具有不同的“包路径”（？），例如vsorg.GiusepesPizzaria.pizza.size org.GiusepesPizzaria.builder.size

有没有更好的方法来检查这个？在实现反射方法之前，我使用了这种构造：

if(builder.size ==null){
    throw new BadPizzaException("Eh, what're ya doin'?"+
             " Pizza Size was not set correctly");
}else{
    size=builder.size;
}

但它最终，如果你说要检查大约 10 个字段，冗长的，杂乱无章的应该是一个简单的类。

所以这就是我尝试过的。有没有更好的方法？

4

2 回答 2

2

确保设置所有变量的有趣模式是使用Step Builder 模式，其中第一个设置器只允许您设置第二个，第二个只允许第三个，依此类推。当您在最后一步时，您可以构建类，到那时您将知道所有方法都已被调用。

该帖子的简短摘录：

Panino solePanino = PaninoStepBuilder.newBuilder()
        .paninoCalled("sole panino")
        .breadType("baguette")
        .fish("sole")
        .addVegetable("tomato")
        .addVegetable("lettece")
        .noMoreVegetablesPlease()
        .build();

您必须从 panino 的名称开始，然后使用面包类型。

于 2013-05-15T17:55:51.730 回答
0

尝试这个：

public class Pizza
{
    private final boolean bacon;
    private final boolean cheese;
    private final boolean pepperoni;
    private final int size;

    private Pizza()
    {
        throw new UnsupportedOperationException();
    }

    Pizza(
        final int theSize,
        final boolean theCheese,
        final boolean thePepperoni,
        final boolean theBacon)
    {
        bacon = theBacon;
        cheese = theCheese;
        pepperoni = thePepperoni;
        size = theSize;
    }
}

// new file.
public class PizzaBuilder
{
    private boolean bacon;
    private boolean cheese;
    private boolean pepperoni;
    private int size;

    public PizzaBuilder()
    {
        size = 9; // default size.
    }

    public void setHasBacon()
    {
        bacon = true;
    }

    public void setHasNoBacon()
    {
        bacon = false;
    }

    public void setHasCheese()
    {
        cheese = true;
    }

    public void setHasNoCheese()
    {
        cheese = false;
    }

    public void setHasPepperoni()
    {
        pepperoni = true;
    }

    public void setHasNoPepperoni()
    {
        pepperoni = false;
    }

    public void setSizeNineInch()
    {
        size = 9;
    }

    public void setSizeTwelveInch()
    {
        size = 12;
    }

    public Pizza buildPizza()
    {
        return new Pizza(size, cheese, pepperoni, bacon);
    }
}

使用上面的构建器，构建器不可能生产出无效的比萨饼。

假设：仅支持 9 和 12 英寸的比萨饼。根据需要添加更多 setSize。

构建器使用我称为 NMSeters 的东西。此样式设置器允许您设置值，但不公开所述值的实现。这似乎不是我的原创发明。

于 2013-05-15T17:26:45.617 回答