Probleme de Delete

Version imprimable

Probleme de Delete

Bonjour,

J'ai un problème de pointeur je pense. Et avec mon niveau moyen en C++, je n'arrive pas à trouver d'où viens le problème.

Donc j'ai une classe InputManager qui gère la gestion du clavier dans une application que je développe. C'est une classe de type Singleton.
Je créer mon application une première fois avec cette classe et tout marche impeccable.
Le problème vient ensuite. J'ai besoin de TOUT détruire (y compris l'InputManager) pour recréer une fenêtre quand l'utilisateur choisi une nouvelle résolution de jeu ou des trucs du genre.
Donc je détruit le pointeur sur mon InputManager, et quand je le relance la fonction addKeyListener, mon application plante sur mKeyListener.find(instancename).

J'espère que vous pourrez m'aider, car je nage dans la farine là ...

Merci de votre aide, ci joint la classe InputManager.

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
192
193
194
195
196
197
198
199
200
201
202
203
204
205
206
207
208
209
210
211
212
213
214
215
216
217
218
219
220
221
222
223
224
225
226
227
228
229
230
231
232
233
234
235
236
#include "InputManager.h"
 
InputManager *InputManager::mInputManager=NULL;
 
InputManager::InputManager( void ) : mMouse( 0 ), mKeyboard( 0 ), mInputSystem( 0 ) 
{
}
 
InputManager::~InputManager( void ) 
{
    if( mInputSystem ) 
	{
        if( mMouse ) 
		{
            mInputSystem->destroyInputObject( mMouse );
            mMouse = 0;
        }
 
        if( mKeyboard ) 
		{
            mInputSystem->destroyInputObject( mKeyboard );
            mKeyboard = 0;
        }
 
        OIS::InputManager::destroyInputSystem(mInputSystem);
        mInputSystem = 0;
 
	mKeyListeners.clear();
        mMouseListeners.clear();
    }
}
 
void InputManager::initialise( Ogre::RenderWindow *renderWindow ) 
{
    if( !mInputSystem ) 
	{
        // Setup basic variables
        OIS::ParamList paramList;    
        size_t windowHnd = 0;
        std::ostringstream windowHndStr;
 
        // Get window handle
        renderWindow->getCustomAttribute( "WINDOW", &windowHnd );
 
        // Fill parameter list
        windowHndStr << windowHnd;
        paramList.insert( std::make_pair( std::string( "WINDOW" ), windowHndStr.str() ) );
		paramList.insert(std::make_pair(std::string("w32_mouse"), std::string("DISCL_FOREGROUND" )));
		paramList.insert(std::make_pair(std::string("w32_mouse"), std::string("DISCL_NONEXCLUSIVE")));
 
        // Create inputsystem
        mInputSystem = OIS::InputManager::createInputSystem( paramList );
 
        mKeyboard = static_cast<OIS::Keyboard*>( mInputSystem->createInputObject( OIS::OISKeyboard, true ) );
        mKeyboard->setEventCallback( this );
        mMouse = static_cast<OIS::Mouse*>( mInputSystem->createInputObject( OIS::OISMouse, true ) );
        mMouse->setEventCallback( this );
 
        // Get window size
        unsigned int width, height, depth;
        int left, top;
        renderWindow->getMetrics( width, height, depth, left, top );
 
        // Set mouse region
        this->setWindowExtents( width, height );
    }
}
 
void InputManager::capture( void ) 
{
    // Need to capture / update each device every frame
    if( mMouse )	mMouse->capture();
    if( mKeyboard ) mKeyboard->capture();
}
 
void InputManager::addKeyListener( OIS::KeyListener *keyListener, const std::string& instanceName ) 
{
    if( mKeyboard ) 
	{
        // Check for duplicate items
        itKeyListener = mKeyListeners.find( instanceName );
 
        if( itKeyListener == mKeyListeners.end() )	mKeyListeners[ instanceName ] = keyListener;
		else {	/* Duplicate Item */	}
    }
}
 
void InputManager::addMouseListener( OIS::MouseListener *mouseListener, const std::string& instanceName ) 
{
    if( mMouse ) 
	{
        // Check for duplicate items
        itMouseListener = mMouseListeners.find( instanceName );
        if( itMouseListener == mMouseListeners.end() )	mMouseListeners[ instanceName ] = mouseListener;
        else {	/* Duplicate Item */	}
    }
}
 
void InputManager::removeKeyListener( const std::string& instanceName ) 
{
    // Check if item exists
    itKeyListener = mKeyListeners.find( instanceName );
    if( itKeyListener != mKeyListeners.end() ) {
        mKeyListeners.erase( itKeyListener );
    }
    else {
        // Doesn't Exist
    }
}
 
void InputManager::removeMouseListener( const std::string& instanceName ) 
{
    // Check if item exists
    itMouseListener = mMouseListeners.find( instanceName );
    if( itMouseListener != mMouseListeners.end() )	mMouseListeners.erase( itMouseListener );
    else 
	{
        // Doesn't Exist
    }
}
 
void InputManager::removeKeyListener( OIS::KeyListener *keyListener ) 
{
    itKeyListener    = mKeyListeners.begin();
    itKeyListenerEnd = mKeyListeners.end();
    for(; itKeyListener != itKeyListenerEnd; ++itKeyListener ) {
        if( itKeyListener->second == keyListener ) {
            mKeyListeners.erase( itKeyListener );
            break;
        }
    }
}
 
void InputManager::removeMouseListener( OIS::MouseListener *mouseListener )
{
    itMouseListener    = mMouseListeners.begin();
    itMouseListenerEnd = mMouseListeners.end();
    for(; itMouseListener != itMouseListenerEnd; ++itMouseListener ) {
        if( itMouseListener->second == mouseListener ) {
            mMouseListeners.erase( itMouseListener );
            break;
        }
    }
}
 
void InputManager::removeAllListeners( void ) 
{
    mKeyListeners.clear();
    mMouseListeners.clear();
}
 
void InputManager::removeAllKeyListeners( void ) 
{
    mKeyListeners.clear();
}
 
void InputManager::removeAllMouseListeners( void ) 
{
    mMouseListeners.clear();
}
 
void InputManager::setWindowExtents( int width, int height ) 
{
    // Set mouse region (if window resizes, we should alter this to reflect as well)
    const OIS::MouseState &mouseState = mMouse->getMouseState();
    mouseState.width  = width;
    mouseState.height = height;
}
 
OIS::Mouse* InputManager::getMouse( void ) 
{
    return mMouse;
}
 
OIS::Keyboard* InputManager::getKeyboard( void ) 
{
    return mKeyboard;
}
 
bool InputManager::keyPressed( const OIS::KeyEvent &e ) {
    itKeyListener    = mKeyListeners.begin();
    itKeyListenerEnd = mKeyListeners.end();
    for(; itKeyListener != itKeyListenerEnd; ++itKeyListener ) {
        itKeyListener->second->keyPressed( e );
    }
 
    return true;
}
 
bool InputManager::keyReleased( const OIS::KeyEvent &e ) {
    itKeyListener    = mKeyListeners.begin();
    itKeyListenerEnd = mKeyListeners.end();
    for(; itKeyListener != itKeyListenerEnd; ++itKeyListener ) {
        itKeyListener->second->keyReleased( e );
    }
 
    return true;
}
 
bool InputManager::mouseMoved( const OIS::MouseEvent &e ) {
    itMouseListener    = mMouseListeners.begin();
    itMouseListenerEnd = mMouseListeners.end();
    for(; itMouseListener != itMouseListenerEnd; ++itMouseListener ) {
        itMouseListener->second->mouseMoved( e );
    }
 
    return true;
}
 
bool InputManager::mousePressed( const OIS::MouseEvent &e, OIS::MouseButtonID id ) {
    itMouseListener    = mMouseListeners.begin();
    itMouseListenerEnd = mMouseListeners.end();
    for(; itMouseListener != itMouseListenerEnd; ++itMouseListener ) {
        itMouseListener->second->mousePressed( e, id );
    }
 
    return true;
}
 
bool InputManager::mouseReleased( const OIS::MouseEvent &e, OIS::MouseButtonID id ) {
    itMouseListener    = mMouseListeners.begin();
    itMouseListenerEnd = mMouseListeners.end();
    for(; itMouseListener != itMouseListenerEnd; ++itMouseListener ) {
        itMouseListener->second->mouseReleased( e, id );
    }
 
    return true;
}
 
InputManager* InputManager::getSingletonPtr( void ) {
    if( !mInputManager ) {
        mInputManager = new InputManager();
    }
 
    return mInputManager;
}

Pour creer mon inputManager, je fais :

Code:

1
2
3
4
mInputMgr = InputManager::getSingletonPtr();
mInputMgr->initialise( mRenderWindow );
mInputMgr->addKeyListener( this, "GameManager" );
mInputMgr->addMouseListener( this, "GameManager" );

Pour le détruire, je fais :

Code:

delete mInputMgr;

28/11/2009, 12h47
screetch

et si tu fais

Code:

delete mInputManager; mInputManager = 0;
28/11/2009, 14h21
Morrigan

Merci ca marche niquel (t'imagine pas le soulagement après avoir galérer deux jours sur ce problème).

Donc si ca peux aider plus vu que je suis dans la partie débutant, j'ai passé le :
static InputManager *mInputManager;
en public (il était en privé encore que en protected ca passerait mieux) en l'appelant comme ça :
delete InputManager::mInputManager; InputManager::mInputManager = 0;

Par contre, si tu pouvais m'expliquer pourquoi ça bugait éventuellement ? D'un point de vue pointeur ca faisait quoi ? Car je deletais tout le reste sauf le mInputManager.
28/11/2009, 15h40
screetch

lorsque tu appelles delete sur un pointeur, tu rends ce sur quoi ca pointe invalide. mais, un des défauts du C++ (je trouve ca un peu con quelque fois :( ) c'est que lorsque tu rends invalide ce sur quoi le pointeur pointe, le pointeur continue de pointer dessus.

Dans ton cas, tu detruisais ce sur quoi le pointeur pointait mais le pointeur lui-même ne changeait pas. Ton application continuait a penser que l'objet etait vivant, meme apres que tu l'aies tué.

Une pratique conseillée est de remettre a 0 un pointeur apres avoir detruit le contenu
28/11/2009, 16h50
Morrigan

Très sympa, merci pour ton aide et pour tes éclaircissements.

A+, Olivier.
28/11/2009, 17h04
screetch

ah oui sinon, un cleanup un peu plus correct serait que :
* le constructeur affecte le pointeur mInputManager
* le destructeur le remet a 0

ainsi tu évites le probleme de devoir le mettre en public, et tu garantis que le pointeur correspondra exactement a la durée de vie de l'objet (et ce quelle que soit la facon dont tu t'y prend pour le tuer : le singleton sera toujours réinitialisé)

Salut,

Citation:

Envoyé par screetch

ah oui sinon, un cleanup un peu plus correct serait que :
* le constructeur affecte le pointeur mInputManager
* le destructeur le remet a 0

ainsi tu évites le probleme de devoir le mettre en public, et tu garantis que le pointeur correspondra exactement a la durée de vie de l'objet (et ce quelle que soit la facon dont tu t'y prend pour le tuer : le singleton sera toujours réinitialisé)

Et c'est d'autant préférable que, typiquement, l'utilisateur n'est pas sensé obtenir l'instance du singleton autrement que par la fonction instance ou équivalente (getSingletonPtr dans le cas présent) ni pouvoir le détruire autrement que par la fonction destroy ou similaire...

Toutes deux devant être... des fonctions statiques (les constructeurs et destructeurs devant être privé, le constructeur par copie et l'opérateur d'affectation devant être déclaré en privé mais non défini), cela va de soit, afin d'assurer l'existance d'une instance unique ;)

[EDIT]Ceci dit, il n'est pas *forcément* nécessaire de détruire un singleton afin de le purger (et d'aucuns sur le forum estiment d'ailleurs, avec raison) que l'utilisateur ne devrait jamais avoir à appeler une fonction impliquant la destruction d'un singleton...

Peut etre devrais tu "simplement" envisager de rajouter un service à ton singleton dont le but serait de... libérer correctement l'ensemble des ressources qu'il gère afin de "faire de la place" pour les nouvelles à venir.

En effet, tu peux te mettre dans une situation assez embarrassante si tu écris un code proche de
Code:

1 2 3 4 5 6 mInputManager =InputManager::getSingletonPtr(); /*... on l'utilise, sans problème */ InputManager::destroy(); // on le détruit, mais, mince, on oublie // de mettre mInputManager à NULL... /*... on fait autre chose */ mInputMgr->addMouseListener( this, "GameManager" ); // Aiiie...
Si tu prévois une fonction "clear", qui libère correctement les ressources et vide la collection d'objets, tu évite ce risque car on se trouve alors, quoi qu'il arrive, dans une situation proche du code suivant:
Code:

1 2 3 4 5 6 7 mInputManager =InputManager::getSingletonPtr(); /*... on l'utilise, sans problème */ mInputManager->clear(); // mInputManager est un pointeur toujours // valide sur notre singleton :D /*... on fait autre chose */ mInputMgr->addMouseListener( this, "GameManager" ); //aucun problème, le // pointeur étant encore valide :D

28/11/2009, 23h42
Morrigan

Oui, c'est vrai que si je passe mon instance en public ca perd l'une des utilités du singleton ...

Donc j'ai repassé mon instance en private, créé une méthode clear(), qui "tue" non plus mon singleton, mais seulement les mInputSystem, mMouse, mKeyboard et réinitialise mes std::map.

Donc ça marche niquel aussi et c'est syntaxiquement beaucoup plus propre :)

Merci beaucoup :)