İşaretçiyi bir işaretçi vektörüne itmek için bir bellek sızıntısı mı? Benim sınıfımda

Question

, ben üye değişkeni std::vector<node*> children

Aşağıdaki sınıf üyesi işlevi bellek sızıntısı oluşturuyor mu var?

//adds a child node 
{ 
    node* child = new node("blah","blah","blah"); 
    child->Set_Parent(this); 
    children.push_back(child); //<- Is this ok? 
} 

vektör pointer bir kopyasını oluşturur ve aynı bellek, iki işaretçiler ve daha sonra orijinal işaretçi doğru, kapsam dışına gider?

Bu basit ve açık olabilir, ancak yalnızca varsayımımı onaylamak isterim.
teşekkürler

Answer 1

Bu bir sızıntı değil ... henüz. Bununla birlikte, vector kapsam dışına çıkarsa veya erase, pop_back veya vektörlerden öğeleri kaldıran başka bir şey yaptıktan sonra, kaldırdığınız öğenin ilk kez delete elinizde sızıntısı olur.

Bunu yapmanın doğru yolu, vector<node *> ile vector<unique_ptr<node>> arasında değişiklik yapmaktır. Kodunuz

//adds a child node 
{ 
    node* child = new node("blah","blah","blah"); 
    child->Set_Parent(this); 
    children.push_back(std::unique_ptr<node>(child)); 
} 

değiştirmek Yoksa Boost'u kullanabilirsiniz eğer boost::ptr_vector<node> kullanacaktır.

Answer 2

Bu vektörün yıkıcı içeren sınıf çağrıldığında Eğer çocuk düğümü ayırması unutursanız sadece bir bellek sızıntısı var.

Answer 3

Bu bir bellek sızıntısı değil. Vektörde hala bir işaretçiniz var ve gerektiğinde hafızayı kaldırabileceksiniz.

Answer 4

Vektör kapsam dışına çıktığında, yıkıcı sivri nesneyi yok etmiyor. İşaretçiyi yok eder - hiçbir şey yapmaz.

Kodunuz, new aracılığıyla bu nesneyi oluşturdu. Bu nesneyi silmek için kodunuz sorumludur. Eğer bunu yapmazsan, bir sızıntın var. Bunu erken yapmak için yaparsanız, yani, işaretçiyi vektörden çıkarmadan önce, daha da büyük sorunlarınız vardır.

Answer 5

children'un bir sınıf üyesi olduğunu varsayarak, sınıf deconstructor'daki vektörteki tüm öğeleri silebilirsiniz.

struct Foo{}; 

class Bar 
{ 
public: 
    Bar(){}; 
    ~Bar() 
    { 
     for(vector<Foo*>::iterator it = children.begin(); it != children.end(); ++it) 
     { 
      SAFE_DELETE((*it)); //use your own macro/template or use delete if you don't have one 
      delete (*it); 
      (*it) = NULL; 
     } 
    } 
    vector<Foo*>children; 
} 

Foo* p = new Foo(); 
children.push_back(p); 

Bir new hiçbir nesne öbek üzerinde tahsis bu durumda, vector<Foo>child yerine, özgün nesnenin bir kopyasını yaratacak vektör üzerinde her push_back saklanmasına, ancak işaretçileri kullanıyorsanız beri kullandıysanız kopya oluşturulur, uzun ömürlü bir nesneye işaret eden işaretçi basitçe saklanır, yani evet, daha sonra silmeniz gerekir.