Discussion :

[xwindoo]

Je suis en train de créer un jeu; j'utilise les gamestate, et j'ai créer un objet CMenuMrincipal, mais le problème c'est que lors de la destruction de celle-ci, le programme plante! J'ai aussi quelques problème lors de la destruction des classes CEdit et CBouton (cf.code), c'est pareil le programme plante (je crée des pointeurs de pointeurs que j'alloue 2fois de suite : une fois pour creer un tableau et une fois pour chaque objet). je vous ai mis les fichiers attaché necessaires (si il manque quelque bout de code dite le moi)
Encore une chose : est-ce que je ne fais pas trop d'allocation dynamique ?

Voila, j'attend vos réponses.

[loufoque]

Normal, tu fais free(Bouton[i]);...

Faudrait un peu apprendre à coder en C++, c'est quoi ce C partout ?

En C++ on utilise new et delete, surtout pour les objets.
Et puis même, l'allocation dynamique on évite.

[gege2061]

Bonjour,

Je suis en train de créer un jeu; j'utilise les gamestate, et j'ai créer un objet CMenuMrincipal, mais le problème c'est que lors de la destruction de celle-ci, le programme plante! J'ai aussi quelques problème lors de la destruction des classes CEdit et CBouton (cf.code), c'est pareil le programme plante (je crée des pointeurs de pointeurs que j'alloue 2fois de suite : une fois pour creer un tableau et une fois pour chaque objet). je vous ai mis les fichiers attaché necessaires (si il manque quelque bout de code dite le moi)
Encore une chose : est-ce que je ne fais pas trop d'allocation dynamique ?

Voila, j'attend vos réponses.

Si te serrais possible de cibler un peu le problème, en plus de nous simplifier la tache (après une journée à coder, on a pas forcement envie de comprendre 42Ko de texte ) et en ça peut te permettre de trouver le problème.

A part ça c'est sûr que les allocations dynamiques à la pelle c'est une source de problème, pourquoi ne pas utiliser les vecteurs ?

[Ti-R]

Smartpointer sinon !

[xwindoo]

Excusé moi, je n'ai pas précisé ou se trouvait excactement le problème :
C'est dans le fichier State.cpp dans la fonction ChangeState()

Code :

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

1
2
3
4
5
6
 
if(m_pActualState)
{
	m_pActualState->Release();
	delete m_pActualState; // <- c'est a cette ligne que le programme plante
}

Code :

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

1
2
 
m_pActualState = new CMenuPrincipal(hWnd, this, m_pDirect3D, m_pDirectMusic, m_pSocket, m_pDirectInput); //<- voila la création de m_pActualState

Désolé d'avoir mélangé le c et le c++ je modifierai ca et je vous redit si sa ne plante pas.

ti-R -> c'est quoi un smartpointer ?

[Eusebe]

Je ne sais pas si ça peut venir de là, mais je n'ai pas vu de destructeur virtuel pour CGameState.

ti-R -> c'est quoi un smartpointer ?

C'est un pointeur "intelligent". Ca sert à éviter les problèmes de pointeurs invalides...
Voir le tuto de Miles :
http://miles.developpez.com/tutoriel...ost/smartptrs/

[zais_ethael]

Les smart pointers permettent de gérer plus facilement les allocations dynamiques (en gros). Ceux de boost sont très bien mais tu peux aussi commencer par utiliser le std::auto_ptr<X> présent dans le header <memory>. C'est une classe que tu initialises en lui passant un pointeur vers un objet et qui va détruire cet objet à sa propre destruction.
Sinon, dans ce cas ci je ne vois pas de meilleure solution que tu apprenes à en utiliser. Ton code contient trop d'allocations dynamiques pour qu'on puisse te filer un coup de main, l'erreur peut très bien se trouver à 100 lieues de la ou tu penses.

[xwindoo]

gege2061 -> qu'entend tu par 'vecteur' ?
zais_ethael -> il faudrai que je ne fasse presque pas d'allocatoins dynamqiues; que CMenuPrincipal, CMTDirect3D, etc serai en dynamique et tout le reste (bouton, edit, etc) en statique; et cela éviterai peut-etre pas mal de problème de destruction de la classe CMenuPrincipal ?

Sinon pour les testes je ne pourrai que les faire vendredi

[Eusebe]

gege2061 -> qu'entend tu par 'vecteur' ?

Regarde du côté de la FAQ...
http://c.developpez.com/faq/cpp/?page=STL

[xwindoo]

J'ai fais quelques testes: le destructeur de CGameState ne change rien au problème. Après un nouveau debugage, le problème c'est ciblé sur les fonction Draw() des bouton et edit; donc je pense que ces fonctions sont appelées après la destruction des objets. Je vais par conséquent supprimer toutes les allocations dynamique de ces classes et dans un premier temps les remplacer par un tableau. Je vous redit les résultats dès que j'ai fini.

[xwindoo]

Désolé pour ce retard.
J'ai tout remplacer les edit et bouton dynamiques en statiques.
Je comprend toujours rien : maintenant, le programme plante a la fonction texte() de CMTDirect3D appelé dans la fonction Draw des edit et bouton (j'ai joins les nouveaux fichiers) plus précisément à la fonction DrawText() de l'interface ID3DXFont.
Voila, je commence a désèspérer, je sais vraiment plus quoi faire.

PS: je peu vous donner tous les fichiers du projet si vous voulez tester.