Discussion :

[blueLight]

Bonjour tout le monde,
J'ai un problème j'utilise ce code pour libérer mes pointeurs :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
template <typename B>
    static void SafeDelete(B & pointer)
    {
        if( pointer )
        {
            delete pointer ;
            pointer = nullptr ;
        }
    }

SafeDelete est une fonction extérieure aux classes, et le problème est que si un pointeur à déjà été libéré, avec le code ci-dessus, le programme va quand même dans le if, et plante.
Quelqu'un pourrait il m'aider ?
J'ai également une question : doit on libérer dans le destructeur, les smart pointeurs s'ils ne sont pas encore libérés ?

[koala01]

Salut,

Tu devrais jeter un oeil sur la classe std::unique_ptr, proposée depuis 2011 par la bibliothèque standard

A priori, elle répond exactement à ton problème, même si tu devras, en plus, t'intéresser un peu à la sémantique de déplacement, si tu souhaites -- par exemple -- pouvoir maintenir des collections de pointeurs en mémoire.

L'idée de base de cette classe est très simple: on applique le principe du RAII (Ressource Acquisition Is Initialization) pour la gestion du pointeur sous-jacent (celui qui pointe effectivement vers la ressource dont on a besoin), de manière à ce que le destructeur de cette classe puisse appeler delete sur le pointeur sous-jacent dés que l'on quitte la portée dans laquelle l'instance de la classe a été déclarée.

Ah, et, une dernière chose: de manière générale, l'utilisation des pointeurs représente la toute dernière possibilité à utiliser, celle vers laquelle on ne devrait se tourner que lorsque toutes les autres possibilités s'avèrent impraticables