QList and shared_ptr

Question

Qu'en pensez-vous? Est-ce correct ou y a-t-il des fuites de mémoire?

Source:

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

Sortie:

6 
destroying A(6) 

Answer 1

Cela semble correct. Le shared_ptr de Boost est un pointeur de comptage de référence. Le comptage des références est capable de récupérer de la mémoire s'il n'y a pas de références circulaires entre les objets. Dans votre cas, les objets de classe A ne référencent aucun autre objet. Ainsi, vous pouvez utiliser shared_ptr sans soucis. De même, les sémantiques de propriété permettent d'utiliser shared_ptrs dans les conteneurs STL (et Qt).

Answer 2

Il est difficile de proposer quoi que ce soit, sans savoir pourquoi la liste des shared_ptr s de A objets existent en premier lieu.

Jetez un oeil à la ownership semantics des pointeurs intelligents. Peut-être d'aide à vous.

D'autres choses qui peuvent être améliorés:

1. utiliser des listes de initialiseur dans cteur comme:

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

2.int _tmain(int argc, _TCHAR* argv[]) n'est pas une signature standard;

Utilisez

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

ou tout simplement:

int main() 

Answer 3

Ce code semble parfaitement bien. Si vous demandez des conseils, vous pourriez fournir plus d'informations sur l'utilisation de shared_ptr avec QList, il pourrait y avoir un moyen "Qt" de le faire sans tirer sur les gros canons comme shared_ptr.

Answer 4

Si vous n'utilisez pas de pointeur intelligent, vous devez supprimer les éléments de la liste par vous-même.

Source:

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

Sortie:

6 

Pas bon.