Probleme avec la STL(vector) avec code C

**Benjy** · 05/04/2006, 16h03

Bonjour,

Partant d'un code initial en C, j'ai voulu ecrire une classe C++ afin de pouvoir empiler des objets de celle-ci dans un vector de la STL.
Jusqu'ici ca va. j'ai bien lu la FAQ et j'ai reussi a cree mon vecteur et a faire des push_back de mes objets.
Mais la ou ca coince c'est pour les elements de ma classe que j'ai repris tels quels d'un code C, et qui sont des pointeurs sur structure. Un collegue m'a suggere que quand je faisais un push_back type:

Code :

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

mon_vector.push_back(Ma_classe(mes_parametres));

ce que faisait la STL c d'abord appeler normalement le constructeur pour mon nouvel objet, puis tenter de copier son contenu dans le vector, et que dans cette operation les variables passeraient bien mais pas les pointeurs sur structure que j'ai, qui se metteraient a deconner.
Il m'a alors conseille de faire un vecteur non pas de la classe mais de pointeurs de la classe, c-a-d:

Code :

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

std::vector<Ma_classe*> mon_vector;

au lieu de :

Code :

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

std::vector<Ma_classe> mon_vector;

comme je fais actuellement.

Le probleme, c'est que j'ai essaye, mais qu'en faisant comme conseille je n'arrive plus a acceder aux variables des objets crees dans mon vector, ca plante a la compilation. Alors que dans le deuxieme cas (vecteur de classe normal) ce n'est pas a la compilation, mais a l'execution que je me paie un segmentation fault parce que les pointeurs n'ont pas (ou mal) ete recopies...

Merci d'avance de votre aide, jsuis completement bloque, ce genre de pb me depasse... c dur de concilier C et C++ parfois...

**zul** · 05/04/2006, 16h35

Dans le premier cas, il te faut un constructeur par copie correcte.
Dans le deuxieme cas, il faudrait nous monter ton code mais a priori, ca doit fonctionner

Code :

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

1
2
3
 
Maclasse * tmp = new MaClasse(mes parametre)
mon_vector.push_back(tmp);

**Benjy** · 05/04/2006, 16h46

Pour le deuxieme cas, c-a- d ce que je fais actuellement et qui plante a l'execution sur les pointeurs, j'ecris pour ma part directement:

Code :

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

mon_vecteur.push_back(Maclasse(mesparametres));

ce qui doit revenir au meme avec ce que tu as ecris, mais pour le premier cas je ne comprends pas ton conseil, comment pourrais-je fais un constructeur par copie propre ?

Voila l'erreur de compilation que je me paie quand je suis dans le cas d'un vecteur de pointeurs vers la classe (1er cas) et que j'essaie d'acceder a un parametre short tout con, appele "featcoeff", c-a-d:

Code :

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

mon_vector.featcoeff;

erreur:

Code :

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

1
2
3
melfilt.cxx:1595: error: request for member `featcoeff' in `(mf + 
   460)->std::vector<_Tp, _Alloc>::operator[] [with _Tp = Scdnet*, _Alloc = 
   std::allocator<Scdnet*>](i)', which is of non-class type `Scdnet*'

Scdnet etant ma classe.
mf est un struct contenant mon vecteur

**Aurelien.Regat-Barrel** · 05/04/2006, 16h57

Envoyé par Benjy

Voila l'erreur de compilation que je me paie quand je suis dans le cas d'un vecteur de pointeurs vers la classe (1er cas) et que j'essaie d'acceder a un parametre short tout con, appele "featcoeff", c-a-d:

Code :

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

mon_vector.featcoeff;

là tu essais d'accéder au membre featcoeff de std::vector, qui n'existe pas.
http://c.developpez.com/faq/cpp/?page=STL#STL_vector
c'est comme si tu écrivais:

Code :

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

1
2
Maclasse v[ 10 ];
v.featcoeff;

je te laisse corriger

**Benjy** · 05/04/2006, 17h29

Bon , alors maintenant ca marche apres que j'ai fait ca:

Code :

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

1
2
3
4
5
6
7
8
9
10
 
std::vector<Ma_classe*> mon_vector;
 
Ma_classe *tmp = new Ma_classe(parametres);
mon_vector.push_back(tmp);
 
int i=0; // premier objet ajoute
Ma_classe* tmpnet = mon_vector[i]
short mon_parametre = (*tmpnet).featcoeff;
(*tmpnet).ma_fonction_perso();

C'est bon non ?

**bigboomshakala** · 05/04/2006, 17h51

Envoyé par Benjy

Bon , alors maintenant ca marche apres que j'ai fait ca:

Code :

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

1
2
3
4
5
6
7
8
9
10
 
std::vector<Ma_classe*> mon_vector;
 
Ma_classe *tmp = new Ma_classe(parametres);
mon_vector.push_back(tmp);
 
int i=0; // premier objet ajoute
Ma_classe* tmpnet = mon_vector[i]
short mon_parametre = (*tmpnet).featcoeff;
(*tmpnet).ma_fonction_perso();

C'est bon non ?

c'est pas très joli

Code :

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

1
2
3
4
5
6
7
 
std::vector<Ma_classe*> mon_vector;
mon_vector.push_back(new Ma_classe(parametres)); // là j'ai un petit doute ?
 
int i=0; // premier objet ajoute
short mon_parametre = mon_vector[i]->featcoeff;
mon_vector[i]->ma_fonction_perso();

**Benjy** · 05/04/2006, 18h02

OK oui c pareil et plus court a ecrire

**Aurelien.Regat-Barrel** · 06/04/2006, 10h48

Pense au delete aussi.
http://c.developpez.com/faq/cpp/?pag...lete_array_dim
Ou n'utilise pas de pointeur, ou des pointeurs intelligents:
http://c.developpez.com/faq/cpp/?pag...S_intelligents
car ça devient vite complexe si tu veux quelque chose de robuste.

**Benjy** · 06/04/2006, 11h13

Pour l'instant pour supprimer je ne fais qu'un

Code :

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

mon_vecteur.clear();

Mais il parait que comme ca, ca tue bien le vecteur mais ca ne libere pas la memoire allouee pour les differents objets que j'ai ajoute dedans. Comment faire alors ? Une boucle sur les elements et faire du delete ?