Discussion :

[HK1989]

Bonsoir,
Je voudrais utiliser un conteneur d'objet en utilisant vector mais
quand je lui ajoute plus d'un objet le code se compile mais il y a un bug
lors de l'exécution du programme.

C'est un petit programme qui utilise Direct3D et l'erreur se produit au
moment du rendu. Voici la méthode appelée lors du rendu :

Code :

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

1
2
3
4
5
6
7
 
void My_object :: Render ()
{
    m_sprite -> Begin ( D3DXSPRITE_ALPHABLEND );
    m_sprite -> Draw ( m_texture , NULL , NULL , & m_position , D3DCOLOR_XRGB ( 255 , 255 , 255 ) );
    m_sprite -> End ();
}

La prochaine instruction exécutée est "m_sprite -> Begin ( D3DXSPRITE_ALPHABLEND );".


Pour créer le conteneur j'utilise "std :: vector<My_object> mo;" et j'appelle la méthode void My_object :: Render ()
à chaque rendu comme ça : mo [ 0 ] . Render ();

[JolyLoic]

Je parierais que l'objet My_object ne possède pas de constructeur de copie ni d'opérateur= qui gère correctement la ressource m_sprite de l'objet.
Or, quand on met un objet dans un vector, il va être copié.

[Florian Goo]

Bonjour,

Une petite assertion serait judicieuse ici :

Code :

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

1
2
3
4
5
6
7
8
9
10
 
#include <cassert>
 
void My_object :: Render ()
{
    assert(m_sprite);
    m_sprite -> Begin ( D3DXSPRITE_ALPHABLEND );
    m_sprite -> Draw ( m_texture , NULL , NULL , & m_position , D3DCOLOR_XRGB ( 255 , 255 , 255 ) );
    m_sprite -> End ();
}

Ça teste si m_sprite n'est pas nul (pense bien à l'initialiser à 0 dans ton constructeur), ou autrement dit qu'il pointe bien vers un objet !
Ça ne va peut-être pas te donner la solution, mais c'est au moins une bonne habitude à prendre.

[koala01]

Salut, et bienvenue sur le forum.

Visiblement, m_sprite est... un pointeur...

Or, il faut savoir, ainsi que l'a dit JolyLoic, lorsque tu place un objet dans un vecteur, l'objet est copié, mais aussi que, si tu ne défini pas ton constructeur par copie, le comportement implémenté par le compilateur consiste à copier le contenu de l'objet... membre à membre.

Cela signifie que l'adresse pointée par m_sprite sera recopiée "telle quelle" de ton My_object d'origine vers le My_object "copié" qui sera placé dans le vecteuur...

Cela implique aussi que, lorsque ton instance My_object d'origine sera détruite (parce que tu sort de la portée dans laquelle la variable est déclarée), le destructeur sera appelé, et, si tu as placé delete m_pointer dedans, la mémoire allouée au m_sprite qui est partagé par deux My_object (celui d'origine et celui qui se trouve dans le vecteur) sera libérée...

Dés lors, toute tentative d'acces à cette mémoire échouera lamentablement en faisant "planter" l'application (dans le meilleur des cas)

Plusieurs solutions s'offrent à toi, et elles sont toutes bien plus facile à expliquer qu'à implémenter en exemple:

• Soit, tu change m_sprite en un objet "simple" (sans allocation dynamique de la mémoire)
• Soit, tu définis le constructeur par copie (et l'opérateur d'affectation) de telle manière à ce que le premier effectue une nouvelle allocation de mémoire pour m_sprite et fasse une copie du contenu
• Soit enfin, tu retire la responsabilité de la gestion de la mémoire de m_sprite à My_object, ce qui implique qu'il faudra sans doute la donner à quelqu'un d'autre (le pattern observer ou mediator pourrait te venir en aide sur ce coup )
• peut être y a-t-il d'autres possibiltiés auxquelles je n'ai pas pensé

[HK1989]

En fait le problème, c'était que je ne savais pas du tout ce qu'était un constructeur de copie. J'ai alors fait mon propre constructeur de copie tout en laissant m_sprite pointeur de LPD3DXSPRITE et c'est certain que maintenant mon programme marche beaucoup mieux.
Merci à vous trois.