Discussion :

[Hybrix]

Bonjour, bonsoir,

J'ai commencé depuis quelques temps la création d'un moteur physique 2D, j'ai donc crée mes différentes classes et autres.

Le moteur physique est composé de formes basiques ( Bloc, triangle, cercle ).
Chaque forme hérite de la classe "Listener" qui elle contient les informations nécessaires à tester les collision et autres ( Force, Velocité, position, ...).

La classe "Listener" contient aussi un pointeur appellé m_vertices, celui-ci sert à allouer dynamiquement par la suite les sommets des polygons.
Exemple : Je crée un objet de type "Cercle", je choisis le nombre de points que le cercle possedera puis celui-ci se crée.

Malheuresement cette partie la pose problème, Si je crée un premier cercle, celui-ci fonctionne correctement. Des que je souhaite en crée un autre, le programme plante, ou celui-ci se lance mais le second cercle à un point en moins.

Voici un screen montrant le problème :



Le second cercle ressemble plus à pacman et le programme plante à sa fermeture.
Je ne voit pas du tout d'ou ça vient, a chaque nouvel objet crée ( apres un premier ), les suivant on un sommet en moins ( Le premier du tableau j'ai l'impression ( m_vertices) ).

Voici le code source du moteur ( Inclus un programme de test ) :
Télécharger depuis un client SVN : svn checkout http://hx-engine.googlecode.com/svn/trunk/ hx-engine-read-only
Télécharger archive .rar : Voir le Fichier : hxengine_l6yxj7gfr7.rar

L'allocation se fait dans les classes du dossier "Formes".
J'ai inclus un programme d'exemple avec. ( Dans le dossier "Exemple" )

Merci à vous.
Hybrix.

[Mat007]

Salut,

Tu dérives des classes de Listener alors qu'elle n'a aucune méthode virtuelle : déclare le destructeur virtuel :

Code :

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

virtual ~Listener();

Ensuite ta variable est de type :

Code :

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

Vector *m_vertices;

Donc est un pointeur sur un Vector, mais tu alloues la mémoire en faisant :

Code :

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

m_vertices = new Vector[m_verticesNumber];

Donc comme un tableau de Vector...


Sinon je pense que tu devrais lire quelques bouquins ou jeter un oeil à des tutoriels parce qu'il y a quelques trucs pas géniaux, dont :
. il faut utiliser les listes d'initialisation dans les constructeurs
. il faut libérer ce que tu alloues (si tu fais un new dans un constructeur il faut faire un delete dans le destructeur)
. il vaut mieux utiliser des std::vector (par exemple) plutôt que des tableaux (en plus tu en utilises ailleurs)
. il vaut mieux passer des const & plutôt que des copies sur les types autre que de base (genre Vector).
. ça ne sert à rien de tester si un pointeur vaut 0 (ou NULL) avant de le delete (le test est de toute façon fait dans l'implémentation de delete)
. ça ne sert à rien de remettre un pointeur à 0 (ou NULL) après l'avoir libéré dans un destructeur (à la fin du destructeur l'objet n'existe plus de toute façon)
. il faut tester qu'un pointeur qui peut être nul ne l'est pas avant de la manipuler (cf. m_separationAxis dans Collision)
. il faut faire gaffe aux fuites mémoires (m_separationAxis m'a l'air écrasé à chaque appel de Collision::Search)

Après la conception est sans doute améliorable (cette classe Listener avec plein de données en protected avec leur tripotée d'accesseurs/mutateurs) mais c'est moins génant à la limite.

MAT.

[Hybrix]

Salut,

Merci beaucoup d'avoir pris le temps de regarder les sources et d'avoir répondu à mon sujet ( Et en plus rapidement ), ça fait plaisir .

J'ai suivi tes conseils dumoins certains :

Tu dérives des classes de Listener alors qu'elle n'a aucune méthode virtuelle : déclare le destructeur virtuel :

Je ne savais pas qu'il falait faire ça, je le faisait uniquement lorsque la classe était abstraite. Doit je le faire aussi pour les classes filles ?

il faut utiliser les listes d'initialisation dans les constructeurs

Le truc c'est que la liste vas être énorme pour certainnes classes ( cf. Listener.hpp), mais je l'ai fait pour les autres.

. il vaut mieux utiliser des std::vector (par exemple) plutôt que des tableaux (en plus tu en utilises ailleurs)

J'ai désormais enlever tout les tableaux et je suis passé sous les "vector", résultat : Le moteur ne plante plus et les problème d'affichages ont disparus.

il vaut mieux passer des const & plutôt que des copies sur les types autre que de base (genre Vector).

Je n'ai pas très bien compris cela, serait t-il possible d'avoir plus d'infos ?

ça ne sert à rien de tester si un pointeur vaut 0 (ou NULL) avant de le delete (le test est de toute façon fait dans l'implémentation de delete)
. ça ne sert à rien de remettre un pointeur à 0 (ou NULL) après l'avoir libéré dans un destructeur (à la fin du destructeur l'objet n'existe plus de toute façon)
. il faut tester qu'un pointeur qui peut être nul ne l'est pas avant de la manipuler (cf. m_separationAxis dans Collision)
. il faut faire gaffe aux fuites mémoires (m_separationAxis m'a l'air écrasé à chaque appel de Collision::Search)

Les "vector" ont résolus ses soucis .

Après la conception est sans doute améliorable (cette classe Listener avec plein de données en protected avec leur tripotée d'accesseurs/mutateurs) mais c'est moins génant à la limite.

Je trouve cela très propre et offre pas mal de possibilitée à l'objet.

Merci encore d'avoir répondu.

[Mat007]

Je ne savais pas qu'il falait faire ça, je le faisait uniquement lorsque la classe était abstraite. Doit je le faire aussi pour les classes filles ?

Plus d'information sur les destructeurs virtuels dans la FAQ.
Sinon en théorie non il n'est pas nécessaire de remettre le mot clef virtual dans la classe fille ("virtuelle un jour, virtuelle toujours"), mais en général on le fait quand même par soucis d'homogénéité parce que pour une méthode 'normale' on le remet systématiquement pour éviter qu'un utilisateur d'une classe fille n'ait besoin d'aller voir dans la classe mère si par hasard la méthode est virtuelle ou non.

Le truc c'est que la liste vas être énorme pour certainnes classes ( cf. Listener.hpp)

Je ne vois pas très bien le rapport, tu n'es pas obligé d'y mettre les initialisations par défaut (un std::vector typiquement).
Si tu trouves que la liste est trop longue c'est sans doute qu'il y a trop de données dans une classe...
C'est juste que plutôt qu'initialiser dans le corps du constructeur il faut faire la même chose dans la liste d'initialisation, la différence en terme de volume de code est minime...
De nouveau des informations sur ce point dans la FAQ.

Je n'ai pas très bien compris cela, serait t-il possible d'avoir plus d'infos ?

Plutôt que de faire :

Code :

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

1
2
3
4
void Listener::AddForce(Vector force)
{
  ...
}

Il vaut mieux faire :

Code :

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

1
2
3
4
void Listener::AddForce(const Vector& force)
{
  ...
}

Ca évite une copie.
Pour Vector c'est sans doute négligeable puisqu'il n'est composé que de 2 float, mais par soucis d'homogénéité on le fait en général pour tous les types qui ne sont pas de base.

MAT.

[Hybrix]

Merci beaucoup pour ces précisions, mes problèmes sont résolus.