Java 对Joshua Bloch引入的Builder模式的一些修改

我使用Joshua Bloch介绍的Java构建器模式。有时，我发现某些字段使用默认值进行初始化比使用原语类型更昂贵

因此，我的策略是这样

我延迟了这些字段的默认值初始化操作

在构建过程中，我只会将它们初始化为默认值，前提是调用方以前没有设置它们

我不确定这样做是否好？有可能发生什么意外吗？比如，线程安全问题？到目前为止，我认为这没有任何问题

package sandbox;

import java.util.Calendar;

/**
 *
 * @author yccheok
 */
// Builder Pattern
public class NutritionFacts {
    private final int servingSize;
    private final int servings;
    private final int calories;
    private final int fat;
    private final int sodium;
    private final int carbohydrate;

    private final java.util.Calendar calendar;    // !!!

    public static class Builder {
        // Required parameters
        private final int servingSize;
        private final int servings;
        // Optional parameters - initialized to default values
        private int calories = 0;
        private int fat = 0;
        private int carbohydrate = 0;
        private int sodium = 0;

        // We suppose to provide a default value for calendar. However, it may
        // seem expensive. We will do it later during build.        
        private java.util.Calendar calendar = null;

        public Builder(int servingSize, int servings) {
            this.servingSize = servingSize;
            this.servings = servings;
        }
        public Builder calories(int val)
        { calories = val; return this; }
        public Builder fat(int val)
        { fat = val; return this; }
        public Builder carbohydrate(int val)
        { carbohydrate = val; return this; }
        public Builder sodium(int val)
        { sodium = val; return this; }

        public NutritionFacts build() {
            // !!!
            if (this.calendar == null) {
                this.calendar = Calendar.getInstance();
            }            
            return new NutritionFacts(this);
        }
    }

    private NutritionFacts(Builder builder) {
        servingSize = builder.servingSize;
        servings = builder.servings;
        calories = builder.calories;
        fat = builder.fat;
        sodium = builder.sodium;
        carbohydrate = builder.carbohydrate;
        calendar = builder.calendar;
    }
}

---

我看不出有什么问题。构建器模式实际上是一种帮助完成某些任务的API。只要遵循模式/API的原则，就可以在API的掩护下做任何你想做的事情。我认为延迟加载昂贵的资源是可以的，只要确保您记录了您认为开发人员应该知道的任何行为。

如果我真的希望日历为空怎么办？那你的模式就行不通了。否则，对我来说似乎没问题。

小心点，因为日历是可变的，所以你应该在NutritionFacts构造函数中复制它。

好的。这提醒了我。当用户显式设置日历时，也许我应该将布尔标志设置为dirty。然后在构建过程中，我将检查布尔标志？