Constructeur de copie d'insertion de carte STL

Question

J'ai des objets de type MyClass stockés en tant que paires <std::string, MyClass> dans une carte STL. La chaîne std :: string est un nom unique pour chaque objet MyClass. Je veux que chaque objet MyClass soit instancié seulement une fois par nom et ainsi détruit seulement une fois à la fin dans mon application. J'essaie donc d'éviter l'invocation de constructeurs de copie ou de constructeurs par défaut, car ils pourraient invoquer la destruction. Un objet MyClass fait référence à une sorte de ressource qui doit être allouée/libérée une seule fois. J'ai essayé d'utiliser ce code pour créer des instances de MyClass, les mettre dans ma map et donner un pointeur vers l'instance nouvellement créée.

MyClass* FooClass::GetItem(std::string name) 
{ 
    MyClass* item = GetItemExists(name); 

    if (item == NULL) 
    { 
     item = &(*((this->myMap.insert(std::pair<std::string, MyClass> 
             (name, MyClass(name)))).first)).second;  
    } 

    return item; 
} 

La création et l'insertion fonctionnent de cette manière. Mais le destructeur de Class MyClass s'appelle 3! fois. Même l'instruction return item; invoque le destructeur, car il s'agit d'un pointeur ?! Je pensais que c'est impossible et doit être forcé par delete item?!

Je pensais qu'une alternative est de stocker des pointeurs MyClass * au lieu d'objets dans la carte. Ou y a-t-il une meilleure alternative? Je n'ai pas utilisé myMap[name] = MyClass(name); pour éviter la copie/destruction, mais je pense que l'insertion ne le rend pas meilleur.

Answer 1

Vous devez emplace et construire l'élément piecewise inséré:

item = &(this->myMap.emplace(std::piecewise_construct, 
          std::forward_as_tuple(name), 
          std::forward_as_tuple(name)).first->second);