क्या कन्स्ट्रक्टर प्रारंभकर्ता सूची में कोई फ़ंक्शन कॉल करना ठीक है?

Question

मेरी आंत महसूस यह नहीं है। मैं निम्नलिखित स्थिति में हूँ:

class PluginLoader 
{ 
    public: 
     Builder* const p_Builder; 
     Logger* const p_Logger; 

     //Others 
}; 

PluginLoader::PluginLoader(Builder* const pBuilder) 
    :p_Builder(pBuilder), p_Logger(pBuilder->GetLogger()) 
{ 
    //Stuff 
} 

या मैं निर्माता बदल सकते हैं और जहां PluginLoader निर्माण किया है से एक Logger* const पारित करना चाहिए?

Answer 1

यह बिल्कुल ठीक और सामान्य है। p_Builder इससे पहले शुरू किया गया था।

Answer 2

पूरी तरह से अच्छा अभ्यास। एक init समारोह में उन्हें समूह के बजाय कार्यों अपने निर्माता में बुलाया होने के

, केवल लचीलापन प्रयोजनों के लिए:

मैं इस सुझाव है (लेकिन एक विशुद्ध रूप से व्यक्तिगत स्तर पर अपनी) आप बाद में बनाने के लिए करता है, तो अन्य रचनाकार

Answer 3

आपके पास क्या ठीक है। हालांकि, मैं तो बस आप इस नहीं करने के लिए सावधान रहने की चेतावनी देने के लिए करना चाहते हैं:

class PluginLoader 
{ 
    public: 
     Logger* const p_Logger; // p_Logger is listed first before p_Builder 
     Builder* const p_Builder; 

     //Others 
}; 

PluginLoader::PluginLoader(Builder* const pBuilder) 
    :p_Builder(pBuilder), 
    p_Logger(p_Builder->GetLogger()) // Though listed 2nd, it is called first. 
             // This wouldn't be a problem if pBuilder 
             // was used instead of p_Builder 
{ 
    //Stuff 
} 

ध्यान दें कि मैं अपने कोड में 2 बदलाव किए (GMAN इस उल्लेख, मैं सिर्फ यह पूरी तरह से स्पष्ट करना चाहता था)। सबसे पहले, कक्षा परिभाषा में, मैंने p_BLilder से पहले p_Logger घोषित किया। दूसरा, मैंने पैरामीटर के बजाय, p_Logger को प्रारंभ करने के लिए सदस्य p_Builder का उपयोग किया।

या तो इनमें से कोई भी परिवर्तन ठीक होगा, लेकिन साथ में वे एक बग पेश करते हैं, क्योंकि p_Logger पहले प्रारंभ होता है, और आप इसे प्रारंभ करने के लिए अनियमित पीबीयूल्डर का उपयोग करते हैं।

बस हमेशा याद रखें कि सदस्यों को कक्षा परिभाषा में दिखाई देने के क्रम में प्रारंभ किया जाता है। और जिस क्रम में आप उन्हें अपनी प्रारंभिक सूची में डालते हैं वह अप्रासंगिक है।