Supprimer des objets d'une classe singleton

Question

Je souhaite créer une classe Singleton simple en C++ qui contient des pointeurs vers d'autres types.

Le singleton deviendra le propriétaire de ce pointeur à un moment donné.

Donc, c'est un bon moyen de supprimer l'objet de type Test et de libérer sa mémoire sur Singleton destructor ?.

class Test{...}; 
class Singleton 
{ 
    public: 
    static Singleton& getInstance() 
    { 
     static std::unique_ptr<Singleton> instance(new Singleton()); 
     return *instance.get(); 
    } 

    void SetTest(Test* test); 
    Test* GetTest() const; 
    ... 
    private: 
     Singleton(){} 
     ~Singleton(){ if(test) delete test;} // ?? 
     Test* test; 
    ... 
}; 

PS: je ne peux pas utiliser de pointeurs intelligents pour l'objet de test.

Answer 1

Pour donner cette question une conclusion: Oui, il est généralement bon de supprimer l'objet comme vous le proposez. Cependant, s'il vous plaît l'esprit les mises en garde suivantes, qui ont déjà été évoqués dans les commentaires:

• Assurez-vous d'initialiser test-nullptr dans le constructeur du singleton. Sinon, vous pourriez finir par essayer de supprimer une adresse mémoire aléatoire. (Certains programmeur mec)
• Vous n'avez pas besoin de vérifier nullptr dans le destructeur. La suppression d'un nullptr est généralement sûre et ne fera rien. (Massimiliano Janes et here)
• Il est probablement possible d'utiliser un pointeur intelligent dans votre cas, ce qui serait préférable. (Quelque programmeur mec, Massimiliano Janes)