Probleme avec la STL(vector) avec code C

Version imprimable

05/04/2006, 16h03
Benjy

Probleme avec la STL(vector) avec code C

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:

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:

std::vector<Ma_classe*> mon_vector;

au lieu de :

Code:

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... :roll:
05/04/2006, 16h35
zul
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:

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

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:

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:

mon_vector.featcoeff;

erreur:
Code:

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

05/04/2006, 16h57
Aurelien.Regat-Barrel
Citation:

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:

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:

1 2 Maclasse v[ 10 ]; v.featcoeff;
je te laisse corriger :wink:

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

Code:

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 ?

Citation:

Envoyé par Benjy

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

Code:

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 :D

Code:

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();

05/04/2006, 18h02
Benjy

OK oui c pareil et plus court a ecrire :oops:
06/04/2006, 10h48
Aurelien.Regat-Barrel

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.
06/04/2006, 11h13
Benjy

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

Code:

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 ?
06/04/2006, 11h30
Aurelien.Regat-Barrel

Voilà, voir les liens donné ci-dessus.
06/04/2006, 11h59
Benjy

Hum j'ai fait des

Code:

delete []

avec une boucle sur tous les objets de mon vecteur avant de le clearer lui aussi mais ca me fait un segmentation fault...
06/04/2006, 12h03
Benjy

OK bah j'ai fait des delete simples et ca marche 8)
06/04/2006, 12h54
Matthieu Brucher

BEn c'est logique, un new appelle un delete et un new[] un delete[] ;)