सी ++: क्या मुझे पॉइंटर सदस्यों को प्रारंभ करना चाहिए जिन्हें कन्स्ट्रक्टर बॉडी में न्यूल में असाइन किया गया हो?

Question

मान लीजिए मैं:

// MyClass.h 
class MyClass 
{ 
    public: 
    MyClass(); 

    private: 
    Something *something_; 
} 

// MyClass.cpp 
MyClass::MyClass() 
{ 
    something_ = new Something(); 
} 

मैं MyClass निर्माता के निर्माता प्रारंभ सूची में नल (या 0) को something_ प्रारंभ करना चाहिए? या यह जरूरी नहीं है क्योंकि मैं इसे कन्स्ट्रक्टर के शरीर में सौंप रहा हूं? अनुशंसित अभ्यास क्या है?

Answer 1

आम तौर पर आप केवल एक बार, निर्दिष्ट, आरंभीकरण सूची या शरीर में जब तक शरीर प्रारंभ या ऐसा नहीं हो सकता है, या शर्त कोड है:

MyClass::MyClass() 
{ 
    //this code must happen first 
    // _now_ max is known 
    something_ = new Something(max); 
} 

MyClass::MyClass() 
{ 
    if (max) 
     something_ = new Something(max); 
    else 
     something_ = NULL; 
} 

MyClass::MyClass() 
    : something_(new Something()) 
//as pointed out in the comments, use smart pointers 
{ 
} 

Answer 2

आम तौर पर बोलते हुए - नहीं। लेकिन यदि आपके कंस्ट्रक्टर पॉइंटर में कहीं भी इसे प्रारंभ करने से पहले उपयोग किया जाता है तो आपको अपरिभाषित व्यवहार मिलता है। हालांकि, आपके पास सबसे बड़ी समस्या यह है - अगर कन्स्ट्रक्टर में अपवाद फेंक दिया जाता है, तो इसके विनाशक को नहीं कहा जाता है। तो कल्पना करें कि आपके पास ऑब्जेक्ट्स के दो पॉइंटर्स हैं और पहली ऑब्जेक्ट की आवंटन सफल होती है जबकि दूसरा आवंटन विफल रहता है, उस स्थिति में आप संसाधन रिसाव के साथ समाप्त होते हैं। इसे "स्मार्ट" पॉइंटर्स का उपयोग करके हल किया जा सकता है। लेकिन उस स्थिति में उन्हें प्रारंभिक सूची में प्रारंभ किया जाएगा, और यदि आप उन्हें दो बार मूल्य असाइन नहीं करना चाहते हैं तो आप कन्स्ट्रक्टर बॉडी के बजाय वहां स्मृति को आवंटित करते हैं।

Answer 3

नहीं आवश्यक है, खासकर अगर आप तुरंत में प्रारंभ कन्स्ट्रक्टर बॉडी, हालांकि अगर प्रारंभिकता इतना स्पष्ट नहीं है तो क्यों नलिका इसे अनियमित चर के आकस्मिक पहुंच से बचने के लिए नहीं है।

• न्यूनतम प्रदर्शन भूमि के ऊपर
• पागल बग शिकार के साथ इतना कुछ डिबगिंग/समस्या निवारण समय