Xóa véc tơ của con trỏ

Question

Tôi cần tạo con trỏ của các cá thể của một lớp và chương trình không biết thời gian biên dịch sẽ tạo bao nhiêu con trỏ. Để xóa, tôi đã xem xét lưu trữ các con trỏ trong một vectơ, và sau đó xóa từng con một. Việc sử dụng các con trỏ thông minh có phải là cách dọn dẹp không? Và nếu không muốn sử dụng con trỏ thông minh, liệu việc sử dụng vectơ này có được coi là sạch sẽ không?

đang tối thiểu:

#include <vector> 
using namespace std; 

class Foo { 
public: 
    Foo(); 
}; 
Foo::Foo(){} 
void createFooVector(int nb, std::vector<Foo*> &v){ 
    for(int i=0;i<nb;i++){ 
     Foo* f = new Foo(); 
     v.push_back(f); 
    } 
} 
int main(int argc, char *argv[]){ 
    std::vector<Foo*> v; 
    createFooVector(5,v); 
    while (!v.empty()){ 
     Foo* f = v.back(); 
     v.pop_back(); 
     delete f; 
    } 
} 

Answer 1

tôi sẽ đề nghị một trong hai cách sử dụng một boost::pointer_vector, một std::vector<std::unique_ptr<Foo>>, hoặc tung ra riêng lớp Foo quản lý của bạn mà giữ một vector<Foo*> và chăm sóc xóa trong constructor (bạn nên coi đây là giải pháp "chuyên gia", và chỉ có nỗ lực nếu bạn hoàn toàn hiểu được sự an toàn ngoại lệ). Bạn không muốn thực hiện xóa theo cách thủ công, điều đó có thể dễ dẫn đến lỗi.

Answer 2

Mã của bạn là tốt. Tuy nhiên, việc sử dụng con trỏ thông minh nên là lựa chọn ưu tiên (ít mã hơn để viết và ít cơ hội hơn cho các lỗi bộ nhớ).

Answer 3

Bạn có nên sử dụng con trỏ thông minh để dọn dẹp không?

Có.

Và nếu không muốn sử dụng con trỏ thông minh, liệu việc này có được coi là sạch sẽ không?

Tôi không có ý tưởng, tại sao ai đó không muốn sử dụng con trỏ thông minh trong C++, nếu nó không phải là bài tập về nhà ... Nhưng, tôi nghĩ, nó tốt hơn để sử dụng một cái gì đó giống như boost::pointer_containers trong trường hợp này.

Answer 4

Nếu bạn không sử dụng các lớp học có nguồn gốc từ Foo và Foo tương đối rẻ để sao chép cấu trúc, chỉ cần sử dụng vector<Foo>.

Nếu trình biên dịch của bạn hỗ trợ ngữ nghĩa di chuyển thì không có vấn đề gì.