यहोशू बलोच के बिल्डर डिजाइन पैटर्न में सुधार?


12

2007 में वापस, मैंने जोशुआ ब्लोच के बारे में एक लेख पढ़ा "बिल्डर पैटर्न" पर और यह कैसे बिल्डरों और बसने वालों के अति प्रयोग को बेहतर बनाने के लिए संशोधित किया जा सकता है, खासकर जब एक ऑब्जेक्ट में बड़ी संख्या में गुण होते हैं, जिनमें से अधिकांश वैकल्पिक होते हैं। इस डिज़ाइन पैटर्न का एक संक्षिप्त सारांश यहाँ व्यक्त किया गया है

मुझे यह विचार पसंद आया, और तब से इसका उपयोग कर रहा हूं। इसके साथ समस्या, जबकि यह ग्राहक के दृष्टिकोण से उपयोग करने के लिए बहुत साफ और अच्छा है, इसे लागू करने से चूतड़ में दर्द हो सकता है! ऑब्जेक्ट में बहुत सारे अलग-अलग स्थान हैं जहां एक एकल संपत्ति संदर्भ है, और इस प्रकार ऑब्जेक्ट का निर्माण होता है, और एक नई संपत्ति जोड़ने में बहुत समय लगता है।

तो ... मुझे एक विचार था। सबसे पहले, जोशुआ बलोच की शैली में एक उदाहरण वस्तु:

जोश बलोच शैली:

public class OptionsJoshBlochStyle {

    private final String option1;
    private final int option2;
    // ...other options here  <<<<

    public String getOption1() {
        return option1;
    }

    public int getOption2() {
        return option2;
    }

    public static class Builder {

        private String option1;
        private int option2;
        // other options here <<<<<

        public Builder option1(String option1) {
            this.option1 = option1;
            return this;
        }

        public Builder option2(int option2) {
            this.option2 = option2;
            return this;
        }

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

    private OptionsJoshBlochStyle(Builder builder) {
        this.option1 = builder.option1;
        this.option2 = builder.option2;
        // other options here <<<<<<
    }

    public static void main(String[] args) {
        OptionsJoshBlochStyle optionsVariation1 = new OptionsJoshBlochStyle.Builder().option1("firefox").option2(1).build();
        OptionsJoshBlochStyle optionsVariation2 = new OptionsJoshBlochStyle.Builder().option1("chrome").option2(2).build();
    }
}

अब मेरा "बेहतर" संस्करण:

public class Options {

    // note that these are not final
    private String option1;
    private int option2;
    // ...other options here

    public String getOption1() {
        return option1;
    }

    public int getOption2() {
        return option2;
    }

    public static class Builder {

        private final Options options = new Options();

        public Builder option1(String option1) {
            this.options.option1 = option1;
            return this;
        }

        public Builder option2(int option2) {
            this.options.option2 = option2;
            return this;
        }

        public Options build() {
            return options;
        }
    }

    private Options() {
    }

    public static void main(String[] args) {
        Options optionsVariation1 = new Options.Builder().option1("firefox").option2(1).build();
        Options optionsVariation2 = new Options.Builder().option1("chrome").option2(2).build();

    }
}

जैसा कि आप मेरे "उन्नत संस्करण" में देख सकते हैं, 2 कम स्थान हैं जिसमें हमें किसी भी अतिरिक्त गुण (या विकल्प, इस मामले में) के बारे में कोड जोड़ना होगा! एकमात्र नकारात्मक जिसे मैं देख सकता हूं कि बाहरी वर्ग के उदाहरण चर अंतिम नहीं हो पा रहे हैं। लेकिन, इसके बिना भी वर्ग अपरिवर्तनीय है।

क्या वास्तव में रखरखाव में इस सुधार के लिए कोई नकारात्मक पहलू है? एक कारण होना चाहिए जो उन्होंने नेस्टेड वर्ग के भीतर गुणों को दोहराया जो मैं नहीं देख रहा हूं?


यह मेरे यहाँ C # में बिल्डर पैटर्न के बहुत समान लगता है
१२:०२ पर मैटडेवी

जवाबों:


12

आपकी भिन्नता काफी अच्छी है। लेकिन यह उपयोगकर्ताओं को ऐसा करने देता है:

Options.Builder builder = new Options.Builder().option1("firefox").option2(1);
Options optionsVariation1 = builder.build();
assert optionsVariation1.getOption1().equals("firefox");
builder.option1("chrome");
assert optionsVariation1.getOption1().equals("firefox"); // FAILURE!

जो वस्तु को हराता है।

आप ऐसा buildकरने के लिए विधि बदल सकते हैं :

public Options build() {
    Options options = this.options;
    this.options = null;
    return options;
}

जो इसे रोक देगा - कॉल करने के बाद बिल्डर पर कोई भी सेटर विधि में कोई कॉल buildNullPointerException के साथ विफल हो जाएगी। यदि आप फ्लैश होना चाहते हैं, तो आप अशक्त होने के लिए परीक्षण भी कर सकते हैं और इसके बजाय एक IllegalStateException या कुछ फेंक सकते हैं। और आप इसे एक जेनेरिक बेस क्लास तक ले जा सकते हैं, जहाँ इसका इस्तेमाल सभी बिल्डरों में किया जा सकता है।


1
मैं में 2 लाइन बदल जाएगा build()करने के लिए: this.options = new Options();। इस तरह, विकल्प उदाहरण सुरक्षित रूप से अपरिवर्तनीय होंगे, साथ ही बिल्डर एक ही समय में पुन: प्रयोज्य होगा।
नैटिक्स

5

बलोच के पैटर्न में बिल्डर का उपयोग कई बार उन वस्तुओं को उत्पन्न करने के लिए किया जा सकता है जो "ज्यादातर" समान हैं। इसके अतिरिक्त, अपरिवर्तनीय वस्तुएं (सभी फ़ील्ड अंतिम हैं और स्वयं अपरिवर्तनीय हैं) में थ्रेड सुरक्षा लाभ हैं जो आपके परिवर्तन हार सकते हैं।


0

यदि विकल्प प्रभावी रूप से क्लोन करने योग्य था (मेरा मतलब है, क्लोन करने योग्य इंटरफ़ेस की परवाह किए बिना), आप प्रोटोटाइप पैटर्न का उपयोग कर सकते हैं - बिल्डर में एक है और इसे बिल्ड () में क्लोन करें।

यदि आप क्लोन करने योग्य इंटरफ़ेस का उपयोग नहीं करते हैं, तो आपको प्रत्येक फ़ील्ड को कॉपी करना होगा, इसलिए यह एक और जगह जोड़ देगा जहां आपको इसे जोड़ने की आवश्यकता है, इसलिए कम से कम क्लोन योग्य का उपयोग करने वाले सरल फ़ील्ड वाले वर्ग के लिए अच्छा विचार होगा।

हमारी साइट का प्रयोग करके, आप स्वीकार करते हैं कि आपने हमारी Cookie Policy और निजता नीति को पढ़ और समझा लिया है।
Licensed under cc by-sa 3.0 with attribution required.