संकेत का एक वेक्टर समाशोधन

Question

मान लें मैं इस तरह एक वर्ग को परिभाषित किया है:

class foo { 
private: 
    std::vector< int* > v; 
public: 
    ... 
    void bar1() 
    { 
     for (int i = 0; i < 10; i++) { 
     int *a = new int; 
     v.push_back(a); 
     } 
    }; 

    void bar2() 
    { 
     std::vector<int>::iterator it = v.begin(); 
     for (; it != v.end(); it++) 
     std::cout << (*it); 
     v.clear(); 
    } 
}; 

संक्षेप में, मैं एक सदिश में कुछ संकेत दिए गए पीछे धकेलने, बाद में मैं वेक्टर साफ़ करें। सवाल यह है कि क्या इस कोड में स्मृति रिसाव है? मेरा मतलब है वेक्टर को साफ़ करके, क्या पॉइंटर्स ठीक से हटाए गए हैं?

Answer 1

हां, कोड मेमोरी लीक है जब तक आप पॉइंटर्स को हटा नहीं देते। यदि foo कक्षा पॉइंटर्स का मालिक है, तो उन्हें हटाना उनकी ज़िम्मेदारी है। आपको वेक्टर को साफ़ करने से पहले ऐसा करना चाहिए, अन्यथा आप उस स्मृति को हैंडल खो देते हैं जिसे आपको हटाने की आवश्यकता है।

for (std::vector<int>::iterator it = v.begin() ; it != v.end(); ++it) 
    { 
    delete (*it); 
    } 
    v.clear(); 

आपके पास कोई उपयुक्त smart pointer के std::vector का उपयोग करके पूरी तरह स्मृति प्रबंधन समस्या से बचने के सकता है।

Answer 2

नहीं, आप केवल वेक्टर स्टोरेज को साफ़ करते हैं। 'नई' के साथ आवंटित स्मृति अभी भी वहां है।

for (int i =0; i< v.size();i++) 
    { 
    delete (v[i]); 
    } 
    v.clear(); 

Answer 3

आप for_each उपयोग कर सकते हैं:

std::vector<int*> v; 

template<typename T> 
struct deleter : std::unary_function<const T*, void> 
{ 
    void operator() (const T *ptr) const 
    { 
    delete ptr; 
    } 
}; 

// call deleter for each element , freeing them 
std::for_each (v.begin(), v.end(), deleter<int>()); 
v.clear(); 

Answer 4

मुझे लगता है कि कम से कम और स्पष्ट समाधान होगा:

std::vector<Object*> container = ... ; 
for (Object* obj : container) 
    delete obj; 
container.clear();