Discussion :

[syl1405]

Bonjour,

J'aimerais remplir un "vector" de pointeurs d'objets. La taille du vector dépend d'une macro, donc j'aimerais savoir comment je peux remplir le vecteur dans une boucle for intelligemment.

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
 
 
#define MAX_OBJ   10
 
vector<Obj*> v;
 
v.reserve(MAX_OBJ);
 
for(int i=0; i<MAX_OBJ;i++)
{
       v[i] = ???;
}

Si je remplace les ??? par v[i] = new Obj() j'ai bien sûr une erreur. Je ne peux pas non plus faire "Obj *ob = new Obj(); v[i] = ob" sinon je donne le même pointeur dans toutes mes "cases" ...

Une idée ?

[méphistopheles]

Bonjour.

Si je remplace les ??? par v[i] = new Obj() j'ai bien sûr une erreur. Je ne peux pas non plus faire "Obj *ob = new Obj(); v[i] = ob" sinon je donne le même pointeur dans toutes mes "cases" ...

Une idée ?

Attention ! Reservation de la mémoire ne signifie pas allocation: la place mémoire est réservée, mais il n'y a pas encore d'éléments à cet endroit.

Vector te fournit la méthode push_back() pour ajouter des éléments :

Code :

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

1
2
3
4
5
6
7
8
9
10
#define MAX_OBJ   10
 
vector<Obj*> v;
 
v.reserve(MAX_OBJ);
 
for(int i=0; i<MAX_OBJ;i++)
{
       push_back(new Obj());
}

ceci devrait marcher.

Bonne chance

[syl1405]

Code :

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

1
2
3
4
5
6
7
8
9
10
#define MAX_OBJ   10
 
vector<Obj*> v;
 
v.reserve(MAX_OBJ);
 
for(int i=0; i<MAX_OBJ;i++)
{
       push_back(new Obj());
}

ceci devrait marcher.

Oui super, ça ça marche parfaitement Merci beaucoup pour la réponse !

[Tesseract]

Tu peux aussi initialiser la taille dès la création de ton vector:

Code :

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

1
2
3
std::vector<Obj*> vec(MAX_OBJ);
for (unsigned int i = 0; i < vec.size(); ++i) 
  vec[i] = new Obj;

[méphistopheles]

Tu peux aussi initialiser la taille dès la création de ton vector:

Code :

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

1
2
3
std::vector<Obj*> vec(MAX_OBJ);
for (unsigned int i = 0; i < vec.size(); ++i) 
  vec[i] = new Obj;

C'est moins propre: en effet, on fait Max_OBJ initialisation d'objets en plus(et pour rien). par contre, on peut se contenter de faire:

Code :

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

std::vector<Obj*> vec(MAX_OBJ);

ou

Code :

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

std::vector<Obj*> vec(MAX_OBJ,Obj());//Version plus explicite

ce qui revient au mème (c'est le constructeur par défaut qui est appelé.)
Et on aura le même résultat en une ligne.

[syl1405]

Code :

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

std::vector<Obj*> vec(MAX_OBJ,Obj());

Ca ça me plait beaucoup, merci Méphistophéles !!

[koala01]

Salut,

C'est moins propre: en effet, on fait Max_OBJ initialisation d'objets en plus(et pour rien). par contre, on peut se contenter de faire:

Code :

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

std::vector<Obj*> vec(MAX_OBJ);

ou

Code :

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

std::vector<Obj*> vec(MAX_OBJ,Obj());//Version plus explicite

Malheureux, surement pas...

vec est un vecteur de... pointeurs sur Obj...

Les pointeurs doivent être:

• Soit initialisés à NULL (comportement par défaut)
• soit initialisés avec l'adresse d'objets existants par ailleurs (faites gaffe à la durée de vie de ces objets )
• soit initialisés avec l'adresse d'objets... créé dynamiquement pour l'occasion

Toute autre solution mènera à des problème sans noms (n'oubliez pas de les détruire correctement "en temps utiles" )