Membre de classe/vector

Version imprimable

Bonjour, j'ai un problème, je bloque dessus depuis plusieurs heures déjà et je n'en vois pas le bout...

J'ai uen classe Perso :

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
struct position {
	int para;
	int meri;
};
 
 
class Perso {
  public:
          Perso();
          Perso(string n, int v,int e, Sac s, int p, int m);
          ~Perso();
 
          string getNom() const ;
          int getVitesse() const ;
          void setVitesse(int v);
          int getEnergie() const;
          void setEnergie(int e);
          int getParallele() const;
          int getMeridien() const;
          void setPosition(int p, int m);
	  Sac getSac() const;
          void frapper(Perso p);
 
          virtual char typePerso() { return 'p'; }
 
 
  protected:
          string nom;
          int vitesse,energie;
          Sac s;
          position pos;
          friend ostream &operator << (ostream &os, const Perso &p);
};

Et j'aimerai afficher juste le parallele du perso avec un vector :

Code:

1
2
3
4
5
6
7
8
9
void Carte::afficherTousPerso() {
  vector<Perso> * p;
  vector<Perso>::iterator i;
  for (i = (*p).begin(); i != (*p).end(); i++)
    {
	int para = (*p)[0].getParallele();
	cout << para << endl;
    }
}

Il me sort un seg fault, au niveau du (*p)[0]...
Il y a-t-il une autre facon d'acceder a un membre d'une classe ou ma méthode est bonne, juste mal codé ?

Salut,

Si tu ne précises pas de quel Perso il s'agit, le compilateur ne va pas l'inventer pour toi :
Code:

1 2 3 4 5 6 7 8 9 void Carte::afficherTousPerso() { vector<Perso> * p; vector<Perso>::iterator i; for (i = (*p).begin(); i != (*p).end(); i++) { int para = (*p)[0].getParallele(); cout << para << endl; } }
MAT.

16/06/2008, 22h03
poukill

De plus, tu définis un itérateur que tu n'utilises pas... :aie:

Ton code est moyen moyen ça c'est certain, par contre je comprend pas pourquoi ça plante, ton vecteur étant vide il ne devrait même pas passer dans la boucle.... :koi:
17/06/2008, 01h20
Mat007

Citation:

Envoyé par poukill

par contre je comprend pas pourquoi ça plante, ton vecteur étant vide il ne devrait même pas passer dans la boucle....

Ca plante parce que p n'est pas initialisé, et déréférencer un pointeur non initialisé c'est le plantage assuré.
Bon ok ma première réponse était au mieux énigmatique et au pire partiellement fausse, c'est sûr... :roll:

MAT.

Déréférencer un pointeur null, on va droit dans le mur, ça tout le monde est d'accord.
Ce qui me dérange, c'est

Citation:

Il me sort un seg fault, au niveau du (*p)[0]...

Chez moi ce code pourri fonctionne (Visual 2005) et ne rentre pas dans la boucle.
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 #include <iostream> #include <vector> struct Perso { int getParallele() const { return 1; } }; using namespace std; int main() { vector<Perso> * p; vector<Perso>::iterator i; for (i = (*p).begin(); i != (*p).end(); i++) { int para = (*p)[0].getParallele(); cout << para << endl; } return 0; }

17/06/2008, 10h47
BainE
Visual c'est mal, c'est comme les char étoiles.

Peut etre en initialisant le vector a NULL :
Code:

1 2 vector<Perso> * p = NULL;
la sa va peut etre claquer
17/06/2008, 10h50
Black_Lagoon

Mais alors y a t il un autre moyen d'acceder a getparallele avec les vecteurs ?
17/06/2008, 10h57
BainE

heu, non getParalelle() tu y accede avec ta classe Perso.
ton vecteur stock des perso, et permet de les parcourir (notement avec les iterator).

il faut commencer par obtenir une des instance de ta classe perso stockée dans ton vecteur.

Mais la ton vecteur est NULL, il n'y a aucun objet dedans donc tu ne peux rien en faire.

D'ailleurs pourquoi utiliser un pointeur sur vecteur ?
Un vecteur tout simple sera tout aussi efficace ici.
17/06/2008, 11h00
Black_Lagoon

Le pointeur est la car ca faisait 2-3h que j'étais la dessus, et ca commencé a m'enerver, je testai plein de solution possible et inimaginable, et qui faisait tout le temps planté le programme lol

je viens de faire ca vite fais

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
 
static int count = 0;
 
Perso::Perso()
{
    ++count;
    this->energie = 42;
    this->nom = "marvin";
    this->vitesse = count;
}
 
int main( int argc, char** argv )
{
	vector<Perso> p;
    Perso un;
    Perso deux;
    Perso trois;
 
    p.push_back( un );
    p.push_back( deux );
    p.push_back( trois );
 
	vector<Perso>::iterator i;
	for (i = p.begin(); i != p.end(); i++)
	{
		Perso tmp = *i;
        cout << tmp.getNom() << " : " << tmp.getVitesse() << endl;
	}
 
	return 0;
}

voila la version pointeur

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
 
int main( int argc, char** argv )
{
    // Et oui si c est un pointeur faut l allouer
    // Et apres tester que l allocation c'est bien passé mais la pas le temps :P
	vector<Perso>* p = new vector< Perso >();
    Perso un;
    Perso deux;
    Perso trois;
 
    p->push_back( un );
    p->push_back( deux );
    p->push_back( trois );
 
	vector<Perso>::iterator i;
	for (i = p->begin(); i != p->end(); i++)
	{
		Perso tmp = *i;
        cout << tmp.getNom() << " : " << tmp.getVitesse() << endl;
	}
 
	return 0;
}

17/06/2008, 11h52
Black_Lagoon

Alléluia BainE, tu es un dieu xD
Suite aux commentaires laissés, je commençais a partir dans la même direction que toi, mais bon tu m'as quand même bien aidé.
Merci beaucoup !!
17/06/2008, 11h54
BainE

De rien, ce fut un plaisir, par contre tu peux flaguer le post a :resolu: