Problème d'affichage :/

Bonsoir,

Bon je ne suis pas sur que le titre correspond vraiment mais je ne sais pas trop comment définir mon problème :

En gros j'ai une classe joueur contenant ces champs public

Code:

---

1 2 3 4 5

int id; bool humain; int score; Joueur(int,bool); void affiche();

---

Et voici les fonctions constructeur et affichage

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15

Joueur::Joueur(int i,bool h) { id = i; humain = h; score = 0; }   void Joueur::affiche() { if (humain) cout<<"Joueur "; else cout<<"Ordinateur "; cout << id << endl << "- Score : "<< score << endl; }

---

une classe jeu contenant pour le moment ces champs, toujours publics

Code:

---

1 2 3 4

public:Joueur** joueurs; int nbjoueurs; Jeu(int,int);

---

Et le voici son constructeur

Code:

---

1 2 3 4 5 6 7 8 9 10 11

Jeu::Jeu(int nbhumains,int nbordi) { nbjoueurs=0; joueurs = new Joueur*[10]; for (int i=0;i<nbhumains;i++){ Joueur j = Joueur(nbjoueurs+1,true); joueurs[nbjoueurs] = &j;nbjoueurs++; }for (int i=0;i<nbordi;i++){ Joueur o = Joueur(nbjoueurs+1,false); joueurs[nbjoueurs] = &o;nbjoueurs++; }//ctor }

---

Et voici mon problème : DAns le main lorque je défini un joueur dans le main puis que j'appelle la fonction affiche sur ce joueur ça fonctionne bien

ex :

Code:

---

1 2	Joueur j = Joueur(1,true); j.affiche();

---

Code:

---

1 2	Joueur 1 - Score : 0

---

Par contre si je crée un jeu puis que j'appelle la fonction affiche sur un des joueurs du jeu, l'affichage n'est pas celui attendu

ex :

Code:

---

1 2 3

Jeu jeu = Jeu(2,2); jeu.joueurs[0]->affiche();

---

donne

Code:

---

1 2	Joueur 954220448 - Score : 0

---

au lieu de 1 comme ca devrait l'être normalement. :/
Sachant que le nombre "954220448" n'est pas fixe, et je ne vois vraiment pas d'où ça vient.



Une idée?

Merci. :)

Bonsoir.

Citation:

---

Envoyé par Senkai

Code:

---

1 2 3 4 5 6 7 8 9 10 11

Jeu::Jeu(int nbhumains,int nbordi) { nbjoueurs=0; joueurs = new Joueur*[10]; for (int i=0;i<nbhumains;i++){ Joueur j = Joueur(nbjoueurs+1,true); joueurs[nbjoueurs] = &j;nbjoueurs++; }for (int i=0;i<nbordi;i++){ Joueur o = Joueur(nbjoueurs+1,false); joueurs[nbjoueurs] = &o;nbjoueurs++; }//ctor }

---

---

Je mise sur ceci :
La variable j est une variable locale au constructeur Jeu::Jeu(int, int), quand le constructeur a fini de s'exécuter, la variable j est détruite et jeu.joueurs[0] pointe vers une variable qui n'existe plus, sur une zone mémoire qui peut contenir tout et n'importe quoi.

La solution serait d'utiliser un tableau de Joueur (voire un std::vector<Joueur>) au lieu d'un tableau de pointeur de Joueur, ce qui est, en l'occurence, bien plus simple.

C'est ce que je voulais faire au départ, mais le problème c'est qu'un simple tableau de joueur m'oblige à avoir un constructeur Joueur() pour l'initialisation, mais bon si c'est la seule solution je m'en contenterai.

Merci en tout cas. :)

Un std::vector<Joueur> ne souffre pas de ce problème.

Bon j'ai testé et ça fonctionne bien avec un simple tableau de joueurs (si un rajoute un constructeur Joueur())
Par contre j'ai essayé avec un vecteur en modifiant le constructeur de jeu en conséquent

Code:

---

1 2 3 4 5 6 7 8 9

Jeu::Jeu(int nbhumains,int nbordi) { vector<Joueur> joueurs; for (int i=0;i<nbhumains;i++){ joueurs.push_back(Joueur(joueurs.size()+1,true)); }for (int i=0;i<nbordi;i++){ joueurs.push_back(Joueur(joueurs.size()+1,false)); } }

---

Et cette fois ci

Code:

---

1 2	Jeu jeu = Jeu(2,2); jeu.joueurs[0].affiche();

---

me donne une erreur de segmentation...Où ai je fais une erreur?

J'ai l'impression que tu mélanges la variable membre joueurs de ta classe Jeu, et la variable locale joueurs du constructeur de celle ci. Si ta donnée membre est du type vector<joueur>, tu n'as pas besoin de vector<Joueur> joueurs; dans ton constructeur

Citation:

---

Envoyé par JolyLoic

Un std::vector<Joueur> ne souffre pas de ce problème.

---

Il faut utiliser emplace_back() (c++11) à la place de push_back(), non?

---

@Senkai Ton vector<Joueur> doit être la variable membre, et non pas une variable locale.
Aussi, tu devrais faire plus de contrôles: Utilise at() au lieu de [] pour indexer ton vecteur, ça évitera les segfaults au profit de vraies exceptions C++.

Dans les deux cas, le constructeur par défaut ne sera pas requis. emplace_back construira directement au bon endroit. push_back construira en dehors, puis utilisera le constructeur de copie (ou de déplacement, selon les cas) pour mettre dans le tableau.
Mais pour quelqu'un qui débute, pour l'instant, je préfère encore parler de push_back, c'est disponible partout.

Effectivement, parler de emplace_back() à un débutant me parait un peu trop tôt et peut amener de la confusion. Entre le besoin conceptuel de comprendre la sémantique de mouvement, et la syntaxe qui n'est pas la même (on passe uniquement les paramètres du constructeur à la fonction emplace_back()), ça me parait compliqué pour des choses qui ne sont pas essentielles.

Cela dit tu as raison Médinoc, il faut privilégier le emplace_back.

Il faut dire que je n'ai pas l'habitude des objets qui sont copiables mais pas défaut-constructibles.