Đó có phải là một rò rỉ bộ nhớ để push_back một con trỏ vào một vector của con trỏ?

Question

Trong lớp học của tôi, tôi có một biến thành viên std::vector<node*> children

Chức năng thành viên của lớp sau có tạo ra rò rỉ bộ nhớ không?

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

Các vector tạo một bản sao của con trỏ và tôi có hai con trỏ vào bộ nhớ cùng, và sau đó con trỏ gốc đi ra khỏi phạm vi, phải không?

Điều này có thể đơn giản và rõ ràng, nhưng tôi chỉ muốn xác nhận giả định của mình.
cảm ơn

Answer 1

Nó chưa bị rò rỉ ... Tuy nhiên, nếu số vector vượt quá phạm vi hoặc bạn erase, pop_back hoặc làm điều gì đó khác để xóa các phần tử khỏi vectơ, mà không cần delete nhập phần tử mà bạn đang xóa, bạn sẽ bị rò rỉ trên tay.

Cách đúng để thực hiện việc này là thay đổi từ việc sử dụng vector<node *> thành vector<unique_ptr<node>>. Mã của bạn sẽ thay đổi thành

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

Hoặc sử dụng boost::ptr_vector<node> nếu bạn có thể sử dụng Boost.

Answer 2

Chỉ rò rỉ bộ nhớ nếu bạn quên chia nhỏ nút con khi lớp có chứa hàm hủy của vectơ được gọi.

Answer 3

Nó không phải là rò rỉ bộ nhớ. Bạn vẫn có một con trỏ trong vectơ và bạn sẽ có thể giải phóng bộ nhớ khi cần thiết.

Answer 4

Khi vectơ đi ra khỏi phạm vi, đó là hàm hủy không phá hủy đối tượng được trỏ tới. Nó phá hủy con trỏ - không làm gì cả.

Mã của bạn đã tạo đối tượng đó qua new. Mã của bạn chịu trách nhiệm xóa đối tượng đó. Nếu bạn không làm như vậy, bạn bị rò rỉ. Nếu bạn làm như vậy để sớm, tức là, trước khi loại bỏ con trỏ khỏi vectơ, bạn thậm chí còn có vấn đề lớn hơn.

Answer 5

Giả sử children là một thành viên của lớp, bạn chỉ cần xóa tất cả các mục trong vectơ trên bộ giải mã lớp.

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); 

Nếu bạn sử dụng một vector<Foo>child thay vào đó, mỗi push_back trên vector sẽ tạo ra một bản sao của đối tượng gốc phải được lưu trữ, nhưng kể từ khi bạn đang sử dụng con trỏ, trong trường hợp này cấp phát trên heap bởi new không có đối tượng bản sao được tạo ra, con trỏ trỏ đến một đối tượng dài hạn được lưu trữ đơn giản, có nghĩa là có, bạn phải xóa nó sau này.