Discussion :

[User Name]

Bonjour a tous,

Il m'est demande de programmer un objet Echiquier qui contient une fonction charger(), censee lire un fichier .txt de 8 lignes de 8 caracteres.

En fonction des caracteres, je remplis mon objet Echiquier avec des Pieces. L'Echiquier contient un attribut qui est un vecteur de Pieces, Pieces etant un objet abstrait avec comme filles concretes Pion, Roi et Cavalier.

Voici la fonction charger que j'ai faite:

Echiquier.cxx

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
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
 
void Echiquier::charger (std::string nomfichier)
{
	std::ifstream fichier (nomfichier.c_str()); //Fallait le savoir!!!
 
	if (fichier)
	{
		std::string contenu;
		int num_ligne = 0;
		int num_colonne = 0;
		while (std::getline(fichier,contenu))
		{
			std::string::iterator it;
			num_colonne=0;
			for (it=contenu.begin(); it!=contenu.end(); it++)
			{
				std::cout<<"num_ligne="<<num_ligne<<", num_colonne="<<num_colonne<<" et *it="<<(*it)<<std::endl;
				if ((*it)=='.')
				{
 
				}
				if ((*it)=='p')
				{
					static Pion pi = *(new Pion("blanc",*(new Position(num_ligne,num_colonne))));
					static Piece* p = &pi;
					ajouter(p);
				}
				if ((*it)=='c')
				{
					static Cavalier pi = *(new Cavalier("blanc",*(new Position(num_ligne,num_colonne))));
					static Piece* p = &pi;
					ajouter(p);
				}
				if ((*it)=='r')
				{
					static Roi pi = *(new Roi("blanc",*(new Position(num_ligne,num_colonne))));
					static Piece* p = &pi;
					ajouter(p);
				}
				if ((*it)=='P')
				{
					static Pion pi = *(new Pion("noir",*(new Position(num_ligne,num_colonne))));
					static Piece* p = &pi;
					ajouter(p);
				}
				if ((*it)=='C')
				{
					static Cavalier pi = *(new Cavalier("noir",*(new Position(num_ligne,num_colonne))));
					static Piece* p = &pi;
					ajouter(p);
				}
				if ((*it)=='R')
				{
					static Roi pi = *(new Roi("noir",*(new Position(num_ligne,num_colonne))));
					static Piece* p = &pi;
					ajouter(p);
				}
				num_colonne++;
			}
			num_ligne++;
		}
		fichier.close();
	}
	else
	{
		std::cout<<"Impossible d'ouvrir le fichier"<<std::endl;
	}
}

Probleme: Evidemment, des que la fonction se termine, les Pieces disparaissent donc mon echiquier est vide...

Le rajout du mot static m'a permis de voir que c'est bien ca le probleme, mais comme vous pouvez vous en douter, ca ne resout pas le probleme. Je crois qu'il faille que je declare un vecteur<Piece*> comme variable globale, mais je ne sais pas ou le mettre et comment le faire.

Si quelqu'un a une idee, merci d'avance.

[Xtrem_Voyageur]

Tu as un affichage graphique à côté de ton échiquier?
Sinon, en effet, il me parait raisonnable que les pièces soient des attributs de ta classe Echiquier, tu ne devrais pas avoir à les créer dans la méthode charger. Elles devraient l'être en même temps que la création d'une instance de ta classe.

[User Name]

Non. L'exercice impose que le constructeur de l'échiquier crée un échiquier vide. Son vecteur de pièce est donc initialisé, mais vide, après la construction de l'échiquier.

Par contre, les pièces sont des structures à part, et me servent à remplir l'échiquier dans le main en utilisant des pointeurs du type

Code :

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

Piece* = &PionBlanc;

Où PionBlanc est un objet "Pion", une classe fille de Piece (Piece est abstraite).

La méthode charger, quant à elle, doit servir à remplir l'échiquier de pièces en fonction du contenu du fichier.

Sauf que je n'arrive pas à créer de pièces dedans de façon qu'elles survivent à la fin de la fonction (mon échiquier est bien rempli dans charger, mais se vide quand elle se termine, ce qui est logique puisque les pointeurs sont "flingués").

Remède?

[Xtrem_Voyageur]

D'ailleurs pourquoi nommer tes pièces dans ta méthode charger?
Tu peux faire directement

Code :

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

1
2
 
ajouter(new Pion(...)), ajouter(new Cavalier(...))

Quoiqu'il en soit, c'est créé sur le tas donc tant que ta mémoire n'est pas désallouée explicitement, elles seront en mémoire, il faut juste veiller à ne pas "perdre" les pointeurs sur ces zones mémoire, donc ici c'est à ta méthode ajouter de faire le nécessaire.

Car toi, tu fais quelque chose d'étrange, tu crées tes objets Pion, Cavalier et tu utilises une affectation pour qu'ils aient la valeur que tu souhaites leur donner à partir du contenu du même objet crée dynamiquement... tes variables sont donc des variables automatiques locales à ta méthode et sont détruites à la fin de la méthode. A AUCUN MOMENT TU N'AS CONSERVE UN POINTEUR EN MEMOIRE en faisant Pion p = *(new Pion(...)), je pense que c'est la que tu pêches. Tu réalises une copie d'un objet créé dynamiquement avec le new (dont tu n'as gardé aucune référence) et cet objet est copié dans la variable locale p que tu crées. Attention donc à ca.
A la limite, fais Piece* p = new Pion(..);