Discussion :

[sylsau]

Bonjour,

J'ai un petit problème avec un vecteur de pointeurs.

Voici le code qui pose problème : (j'ai volontairement vidé les classes et mis tout ensemble pour bien isoler le problème)

Code :

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

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
 
#include <iostream>
#include <vector>
 
using namespace std;
 
class Element{
	public :
		string att1;
};
 
class Constante : public Element
{
	public :
		string att2;
};
 
 
int main()
{	
	vector<Element*>* v = new vector<Element*>;
	Element* elt = new Element();	
	v->insert(elt);
             // Constante* cte = new Constante();
             // v->insert(cte);
 
 
 
	return 0;
}

Lorsque je compile, g++ me dit en gros qu'il ne s'attend pas à avoir ce type lors de l'insertion dans le vecteur.

Pourtant mon vecteur est bien un vecteur de pointeurs sur la classe Element.

Quelqu'un saurait d'où cela peut venir ?

Ma deuxième question concerne ce que j'ai mis en commentaire dans le code. L'insertion d'un pointeur sur la classe Constante est elle possible dans le vecteur compte tenu du fait que Constante hérite de Element ?

[craryb]

peux tu copier-coller l'erreur qu'il affiche stp?

[Sylvain Togni]

La fonction insert() s'utilise avec plusieurs arguments pour insérer un ou plusieurs éléments à une certaine position dans le vecteur. Pour ajouter un élément à la fin, c'est push_back().

[sylsau]

La fonction insert() s'utilise avec plusieurs arguments pour insérer un ou plusieurs éléments à une certaine position dans le vecteur. Pour ajouter un élément à la fin, c'est push_back().

Effectivement ça marche, j'ai pas assez regardé la doc. Merci.

[Sylvain Togni]

L'insertion d'un pointeur sur la classe Constante est elle possible dans le vecteur compte tenu du fait que Constante hérite de Element ?

Oui, c'est le principe même du polymorphisme de classe, un pointeur (ou référence) vers un objet dérivé peut être utilisé partout où un pointeur (ou référence) vers un objet de base est attendu.

[rikau2]

en effet insert s'utilise avec insert(position, iteration, valeur)
dans ton cas il faudrait faire par exemple

Code :

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

v->insert(v->end(),1,elt);

[loufoque]

Les variables intermédiaires elt et cte ne servent à rien.
Ensuite, ce genre de chose reste une erreur de conception. Qui se charge de gérer la durée de vie des objets dynamiquement alloués ?

[sylsau]

Les variables intermédiaires elt et cte ne servent à rien.
Ensuite, ce genre de chose reste une erreur de conception. Qui se charge de gérer la durée de vie des objets dynamiquement alloués ?

Effectivement les variables intermédiaires ne servent à rien mais pour montrer un problème j'ai sorti l'erreur de mon code et j'ai fait en sorte de rendre ce bout de code le plus explicite possible.

Le problème avec mon vecteur c'est que je le crée à l'intérieur d'une méthode et ensuite je le renvoie.

Au départ, je ne comptais faire qu'un pointeur sur un vecteur pour pouvoir le renvoyer et ensuite y accéder mais je me suis demander si les objets que je mettrais dans ce vecteur seraient accessibles à l'extérieur si je ne mettais pas des pointeurs.

[befalimpertinent]

Au départ, je ne comptais faire qu'un pointeur sur un vecteur pour pouvoir le renvoyer et ensuite y accéder mais je me suis demander si les objets que je mettrais dans ce vecteur seraient accessibles à l'extérieur si je ne mettais pas des pointeurs.

Non pour pouvoir utiliser le polymorphisme tu dois obligatoirement passer par des pointeurs ou des références.

En revanche ce que souligne loufoque est que tu dois t'occuper de gérer correctement la mémoire (smart pointeur de Boost, compteur de référence) sous peine d'avoir des surprises si un des pointeurs de ton vecteur est supprimer en dehors de tout contrôle. Dans ce cas ta méthode pour vider ton vecteur (cf FAQ http://c.developpez.com/faq/cpp/inde...elete_sequence
risque d'avoir quelques soucis )
Si tu ne veux pas t'embêter avec tout ça tu dois au moins être sur qu'aucun delete ne sera fait sur un des éléments du vecteur avant que tu le vides avec la méthode approprié. Et que une fois vider tu sois sur de ne plus accéder à ces éléments.

[befalimpertinent]

Pourquoi un pointeur sur le vecteur ? Ne peux tu pas faire simplement :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
 
int main()
{	
	vector<Element*> v;
	Element* elt = new Element();	
	v.push_back(elt);
        Constante* cte = new Constante();
        v.push_back(cte);
 
	return 0;
}

Ma deuxième question concerne ce que j'ai mis en commentaire dans le code. L'insertion d'un pointeur sur la classe Constante est elle possible dans le vecteur compte tenu du fait que Constante hérite de Element ?

Oui, par polymorphisme cf FAQ pour plus de renseignements

[sylsau]

Pourquoi un pointeur sur le vecteur ? Ne peux tu pas faire simplement :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
 
int main()
{	
	vector<Element*> v;
	Element* elt = new Element();	
	v.push_back(elt);
        Constante* cte = new Constante();
        v.push_back(cte);
 
	return 0;
}

Oui, par polymorphisme cf FAQ pour plus de renseignements

L'exemple de code que j'ai mis est hors contexte.
Dans mon code complet, je crée ce vecteur au sein d'une méthode et je le renvoie en sortie de la méthode.
Pour pouvoir y accéder à l'extérieur il me fallait donc plutôt un pointeur.

Peut-être qu'utiliser une référence est préférable dans ce cas là ? mais ne connaissant pas trop bien le C++, j'ai mis un pointeur comme j'aurais fait en C en fait.