QList và shared_ptr

Question

Bạn nghĩ sao? Điều này có đúng hay có rò rỉ bộ nhớ không?

Nguồn:

#include <QList.h> 
#include <boost/shared_ptr.hpp> 
#include <iostream> 

class A { 
private: 
    int m_data; 
public: 
    A(int value=0) { m_data = value; } 
    ~A() { std::cout << "destroying A(" << m_data << ")" << std::endl; } 
    operator int() const { return m_data; } 
}; 

int _tmain(int argc, _TCHAR* argv[]) 
{ 
    QList<boost::shared_ptr<A> > list; 
    list.append(boost::shared_ptr<A>(new A(6))); 
    std::cout << int(*(list.at(0))) << std::endl; 
    return 0; 
} 

Output:

6 
destroying A(6) 

Answer 1

Có vẻ như chính xác. Shared_ptr của Boost là một con trỏ đếm tham chiếu. Việc đếm tham chiếu có thể đòi lại bộ nhớ nếu không có tham chiếu vòng tròn giữa các đối tượng. Trong trường hợp của bạn, các đối tượng của lớp A không tham chiếu đến bất kỳ đối tượng nào khác. Vì vậy, bạn có thể sử dụng shared_ptr mà không phải lo lắng. Ngoài ra, ngữ nghĩa quyền sở hữu cho phép shared_ptrs được sử dụng trong các container STL (và Qt).

Answer 2

Rất khó để đề nghị bất cứ điều gì mà không biết lý do tại sao danh sách các shared_ptr s của A đối tượng tồn tại ở nơi đầu tiên.

Hãy xem ownership semantics con trỏ thông minh. Có thể giúp bạn.

Một số thứ khác mà có thể được cải thiện:

1. danh sách Sử dụng initializer trong ctor như:

class A { 
private: 
    int m_data; 
public: 
    A(int value=0) : m_data (value) {} 
// .... 

2.int _tmain(int argc, _TCHAR* argv[]) không phải là một chữ ký tiêu chuẩn;

Sử dụng

int main(int argc, char* argv[]) 

hoặc chỉ:

int main() 

Answer 3

Mã này trông hoàn toàn tốt đẹp.

Nếu bạn đang tìm kiếm lời khuyên, bạn có thể cung cấp thêm thông tin về mục đích sử dụng shared_ptr với QList, có thể có cách "Qt" để thực hiện việc này mà không cần kéo những khẩu súng lớn như shared_ptr.

Answer 4

Nếu bạn không sử dụng con trỏ thông minh, bạn phải tự xóa các phần tử danh sách.

Nguồn:

#include <QList.h> 
#include <boost/shared_ptr.hpp> 
#include <iostream> 

class A { 
private: 
    int m_data; 
public: 
    A(int value=0) { m_data = value; } 
    ~A() { std::cout << "destroying A(" << m_data << ")" << std::endl; } 
    operator int() const { return m_data; } 
}; 

int _tmain(int argc, _TCHAR* argv[]) 
{ 
    QList<A *> list; 
    list.append(new A(6)); 
    std::cout << int(*(list.at(0))) << std::endl; 
    return 0; 
} 

Output:

6 

Không tốt.