Шаблон Builder Джошуа Блоха и предупреждения PMD

О том, как убрать дублирование.

Я получу больше отрицательных голосов :) А может быть, что-то вроде этого?

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

public static class Builder {
    private Pizza pizza = new Pizza();

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

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

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

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

    public Pizza build() {
        return pizza;
    }
}

private Pizza() {
}
}

частные поля в классах Pizza и Builder дублируются. Это будет раздражать, когда количество приватных полей станет больше. Есть ли способ избежать этого?

Я лично обхожу это, используя третий закрытый класс объекта статического значения, который содержит все поля и использует его как в построителе, так и в основном классе (доступ к полям обрабатывается делегированием). Конечно, это может привести к увеличению количества строк/классов, но это бесценно, если ваши конструкторы окажутся сложными с большим количеством полей и проверок.

Кроме того, не помешает предоставить статический метод в классе Pizza, который создает объект Pizza с обязательными полями. Если, конечно, вы не уверены, что такое обязательные поля, или боитесь, что обязательные поля могут измениться в ходе эволюции вашего класса. Дело в том, что пока вы можете оправдать свои действия после долгих размышлений (как выразился Джошуа Блох), вы можете спокойно игнорировать эти предупреждения, зная, что вы знаете, что делаете. :-)

Отрывок на выброс:

public class Pizza {

    private final PizzaVO vo;

    private static class PizzaVO {

        int size;

        boolean cheese;

        boolean pepperoni;

        boolean bacon;
    }

    public static class Builder {

        private final PizzaVO vo = new PizzaVO();

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

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

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

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

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

    private Pizza(PizzaVO vo) {
        this.vo = vo;
    }

    public int getSize() {
        return vo.size;
    }

    // other getter setter methods as per your taste

}