boost :: sémantique shared_ptr (copie)

Question

Je voulais juste avoir une nouvelle paire d'yeux que le code ci-dessous est correct en ce que:

Les pointeurs contenus dans le trifoo d'objet (stocké dans un ptr_vector) sont Partagée pointeurs f, g, h.

Aussi, quel est le résultat de la copie shared_ptr dans le constructeur de trifoo; est-ce la méthode correcte de 'partage' shared_ptr, en veillant à ce que le nombre de références soit augmenté etc. Tous mes autres doutes ont pu être vérifiés, mais je ne suis pas sûr de pouvoir vérifier cela (correctement). Toute critique est la bienvenue aussi.

#include <boost/ptr_container/ptr_vector.hpp> 
#include <boost/shared_ptr.hpp> 

class foo { 
    int a, b; 
public: 
    foo(int A, int B) : a(A), b(B) {} 
}; 

typedef boost::shared_ptr<foo> foo_ptr; 

class trifoo { 
    foo_ptr c, d, e; 
public: 
    trifoo(const foo_ptr& C, const foo_ptr& D, const foo_ptr& E) : c(C), d(D), e(E) {} 
}; 

int main() { 
    for (int i = 0; i < 5000000; i++) { 
     foo_ptr f(new foo(1,2)); 
     foo_ptr g(new foo(2,3)); 
     foo_ptr h(new foo(4,5)); 

     boost::ptr_vector<trifoo> tris; 

     tris.push_back(new trifoo(f, g, h)); 
    } 

    return 0; 
} 

Remarque: la boucle inutile consistait à tester les fuites de mémoire, dont aucune ne s'est produite.

Answer 1

Le code semble être techniquement correct.

La sémantique de la copie d'un shared_ptr, cependant, est que le nombre de références de l'objet référencé est augmenté. Cela fonctionne juste. Pas d'inquiétudes à avoir.

Certaines questions de style, cependant:

• Passing shared_ptr par référence ou déclarant const, n'a pas de sens. C'est parce qu'il peut toujours être copié. Il suffit de passer ces shared_ptr par valeur.

• Utilisez les listes d'initialisation de constructeur à la place des affectations lorsque cela est pratiquement possible. Avoir les trois new dans les expressions différentes est très bon. Cela évite un écueil de sécurité exceptionnel. Mais encore mieux, mettez cette logique de création dans une fonction d'usine.

Vive & HTH.,