QList i shared_ptr

Question

Co myślisz? Czy to prawda, czy są wycieki pamięci?

Źródło:

#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; 
} 

wyjściowa:

6 
destroying A(6) 

Answer 1

Wygląda na to, że jest poprawny. Boost's shared_ptr jest referencyjnym wskaźnikiem zliczającym. Licznik odwołań jest w stanie odzyskać pamięć, jeśli nie ma żadnych odwołań kołowych między obiektami. W twoim przypadku obiekty klasy A nie odwołują się do żadnych innych obiektów. W ten sposób możesz używać shared_ptr bez obaw. Ponadto semantyka własności zezwala na używanie shared_ptrs w kontenerach STL (i Qt).

Answer 2

Trudno sugerować niczego nie wiedząc dlaczego lista shared_ptr s od A obiekty istnieją w pierwszej kolejności.

Spójrz na ownership semantics inteligentnych wskaźników. Może ci pomóc.

Kilka innych rzeczy, które można poprawić:

1. list Wykorzystanie initializer w ctor jak:

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

2.int _tmain(int argc, _TCHAR* argv[]) podpis nie jest standardowa;

Zastosowanie

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

lub po prostu:

int main() 

Answer 3

Ten kod wygląda perfekcyjnie.

Jeśli chcesz uzyskać poradę, możesz podać więcej informacji na temat używania shared_ptr z QList, może to być metoda "Qt" bez ciągnięcia dużych broni, takich jak shared_ptr.

Answer 4

Jeśli nie używasz inteligentnego wskaźnika, musisz sam usunąć elementy listy.

Źródło:

#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; 
} 

wyjściowa:

6 

nie jest dobra.