STL Problème avec une liste d'instances de class

Version imprimable

15/02/2007, 21h15
BruceBoc

STL Problème avec une liste d'instances de class
Bonjour,
Donc voila, au départ j'utilisai une instance de class unique ( pas de liste ), et tout marchait bien. Ayant besoin d'un certain nombre de ces instances, je décide donc de les regrouper dans une "list" de la std. Mais voila, après beaucoup de tentatives et de bidouilles, rien a faire, cela ne fonctionne pas correctement et je ne comprend pas pourquoi. Actuellement pas d'erreur à la compilation, mais lorsque le programme s'exécute, l'instance qui devrait afficher une image à l'écran n'affiche rien, mais ne génére pas d'erreur lors de l'exécution du prog.

Voici un regroupement des lignes interressantes STL :
Code:

1 2 3 4 5 6 7 8 9 list <Cbowl> Lbowl; list <Cbowl>::iterator Ibowl; Cbowl A_bowl; Lbowl.push_back(A_bowl); Lbowl.begin()->initpos(0,430,1); Lbowl.begin()->draw();
Apparament, ça vient du fait que données de la class ne se modifient pas, mais je suis pas à l'aise avec les pointeurs.
15/02/2007, 21h21
bouba

Salut, as tu des pointeurs membre de ta classe Cbowl qui pointent sur des objets créé dynamiquement ? Si tu en as il faut absolument que tu définisse un constructeur de recopie qui copie correctement ces membres dynamiques !!

L'erreur que tu peux avoir serait sur le push_back.

Code:

Lbowl.push_back(A_bowl);

en fait, l'objet inséré dans ta liste n'est pas l'objet A_bowl, mais une copie de celui-ci. Cette copie doit donc être correcte !

Tu vois ou je veux en venir ? Sinon fais voir le code de Cbowl.
15/02/2007, 21h33
Bakura
Citation:
Envoyé par BruceBoc

Bonjour,
Donc voila, au départ j'utilisai une instance de class unique ( pas de liste ), et tout marchait bien. Ayant besoin d'un certain nombre de ces instances, je décide donc de les regrouper dans une "list" de la std. Mais voila, après beaucoup de tentatives et de bidouilles, rien a faire, cela ne fonctionne pas correctement et je ne comprend pas pourquoi. Actuellement pas d'erreur à la compilation, mais lorsque le programme s'exécute, l'instance qui devrait afficher une image à l'écran n'affiche rien, mais ne génére pas d'erreur lors de l'exécution du prog.

Voici un regroupement des lignes interressantes STL :

Code:

1 2 3 4 5 6 7 8 9 list <Cbowl> Lbowl; list <Cbowl>::iterator Ibowl; Cbowl A_bowl; Lbowl.push_back(A_bowl); Lbowl.begin()->initpos(0,430,1); Lbowl.begin()->draw();

Apparament, ça vient du fait que données de la class ne se modifient pas, mais je suis pas à l'aise avec les pointeurs.
Et en faisant
Lbowl.at(0)->initpos (0,430,1)
Lbowl.at(0)->draw()
?

EDIT : après reflexion, ce ne sont pas des pointeurs que tu mets dans ta liste, donc c'est
Lbowl.at(0).initpos (0, 430, 1)
Lbowl.at(0).draw ();;
15/02/2007, 21h35
Laurent Gomila

Citation:

Et en faisant Lbowl.at(0)->initpos (0,430,1) ?

Attention à ne pas confondre avec std::vector. Si on pouvait accéder aux éléments d'une liste via des indices, ça ne servirait à rien ;)
Par contre on peut utiliser Lbowl.front().initpos(...).

Je pencherai aussi pour une mauvaise gestion de la copie au niveau de la classe stockée.
15/02/2007, 21h37
Bakura

Citation:

Envoyé par Laurent Gomila

Attention à ne pas confondre avec std::vector. Si on pouvait accéder aux éléments d'une liste via des indices, ça ne servirait à rien ;)
Par contre on peut utiliser Lbowl.front().initpos(...).

Je pencherai aussi pour une mauvaise gestion de la copie au niveau de la classe stockée.

Ah je pensais que ça fonctionnait pareil... Mais je comprends pas, dans sa liste il ne met pas de pointeur, pouruquoi il utilise :

Lbowl.begin()->initpos(0,430,1);
Lbowl.begin()->draw();

begin() renvoi un pointeur vers le premier élément ?
15/02/2007, 22h00
loufoque

Citation:

begin() renvoi un pointeur vers le premier élément ?

Un itérateur, qui se comporte comme un pointeur.
15/02/2007, 22h01
Bakura

Citation:

Envoyé par loufoque

Un itérateur, qui se comporte comme un pointeur.

Et pourquoi il n'utilise pas son iterateur (Ibowl) pour ça ?

J'ai changé le "->" par "." et sa ne change rien.
Voici le code de la class :

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
 
class Cbowl
{
 
	private:
 
		GLuint surface;
 
		SDL_Rect rectsrc;
		SDL_Rect rectdest;
 
		int size_X;
		int size_Y;
		int frames_X;
		int frames_Y;
		int way;
		int pos_X;
		int pos_Y;
		int amplitude;
		float step;
		float angle;
 
	public:
 
		FSOUND_SAMPLE* hit;
 
		void load(char* filename, int sizeX, int sizeY, int framesX, int framesY);
		void draw();
		void initpos();
		void update();
		void changeway();
 
		int returnposX(){ return pos_X;};
		int returnposY(){ return pos_Y;};
		int returnsizeX(){ return rectsrc.w;};
		int returnWay(){ return way;};
 
};

Je ne me sert pas encore de l'itérateur, car déja à la base le code ne marche déja pas, et pas besoin de cela puisque la liste ne comporte pour l'instant qu'un seul élément.

15/02/2007, 22h38
olive_le_malin

Pourquoi stocker des objets et plutot des références ou des pointeurs ?
Ca éviterait les copies couteuses ...

La fonction initpos de la classe ne prend pas d'arguments.
Il n'y a pas de constructeur de recopie.
Que sont les GLuint, et SDL_rect ?
15/02/2007, 23h06
BruceBoc

Citation:

Pourquoi stocker des objets et plutot des références ou des pointeurs ?

Je veux bien passer des références ou des pointeurs, mais j'ai déja essayé et je n'arrive pas à compiler, il faudrait que tu me montre le code pour je l'essaye ( je ne suis pas à l'aise avec ces 2 notions ).

Citation:

La fonction initpos de la classe ne prend pas d'arguments.

En fait si, simple erreur de recopiage.

Citation:

Il n'y a pas de constructeur de recopie.

Je ne sai pas à quoi sa sert, peut-tu préciser l'intérêt.

Citation:

Que sont les GLuint, et SDL_rect ?

GLuint : variable OpenGL pour stocker une texture
SDL_rect : variable SDL qui est un tableau de positions ( x, y ,h, w ).
15/02/2007, 23h35
loufoque

Citation:

En fait si, simple erreur de recopiage.

Comment tu veux qu'on t'aide si tu fais des erreurs de recopiage ?
Fournis un exemple minimal réduit qu'on peut compiler et qui produit ton problème.
16/02/2007, 09h04
olive_le_malin
Citation:

Envoyé par BruceBoc

Je veux bien passer des références ou des pointeurs, mais j'ai déja essayé et je n'arrive pas à compiler, il faudrait que tu me montre le code pour je l'essaye ( je ne suis pas à l'aise avec ces 2 notions ).
Code:

1 2 list <Cbowl*> Lbowl; LBowl.push_back(new CBowl());
Il serait bien aussi qu'il y ait un constructeur ...

Citation:

Je ne sai pas à quoi sa sert, peut-tu préciser l'intérêt.

Ben vu que tu veux stocker des objets, il faut un constructeur de copie puisque tu passes des objets par valeur.
Le compilo t'en génères un par défaut, qui peut trés bien aller si ton objet n'utilises que des données "simples" (pas d'allocation, objets ayant déjà un constructeur de copie ...).
Si tu les passes par réf ou pointeur, il n'y aura pas besoin de faire de copie.
[/QUOTE]
16/02/2007, 14h12
BruceBoc
Merci beaucoup Olive le malin, maintenant sa fonctionne à merveille.
Voici donc le bon code à utiliser :
Code:

1 2 3 4 5 list <Cbowl*> Lbowl; Lbowl.push_back(new CBowl()); Lbowl.front()->initpos(0,430,1); ...