Discussion :

[djun1]

Bonjour,

Je sais que le titre n'est pas vraiment claire. Mais, j'ai un petit probleme. Je me demande: Est-ce qu'on peut mettre un objet comme attribut?

Jai essayé avec ce code mais ça n'a pas marcher(ligne 21):

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
#include <iostream>
#include <string>
 
class Guerrier
{
private:
	int hp;
	int sp;
	int xp;
	int level;
	int argent;
	std::string classe;
	std::string nom;
	std::string technique1;
	std::string technique2;
	std::string technique3;
	int force;
	int rapidite;
	int energie;
	int defense;
	Arme monArme;
public:
	Guerrier();
	int getForce(int &force);
	void recoveryHP(int &hp);
	void recoverySP(int &sp);
	void gagnerXp(int &xp);
	void changerLVL(int &xp);
	void reatsu(std::string &technique1);
	void reveille(std::string &technique2);
	void bankai(std::string &technique3);
	void seFaireFrapper(int &defense);
	bool vivant() const;
	void dommage();
};
 
class Arme
{
private:
	std::string nomArme;
	int forceArme;
	int poid;
 
public:
	void changerArme(std::string nouveauNom, int nouveauForce, int nouveauPoid,std::string &nomArme , int &forceArme, int &poid);
	Arme();
};

[Bousk]

Bonjour,

heureusement que oui..!
"ça ne marche pas" n'est pas un message d'erreur. Sois plus précis à l'avenir.
Tout objet doit être déclaré avant d'être utilisé.

[koala01]

Salut,

La réponse courte d'abord:"oui, c'est possible".

La réponse longue:

Seulement, il faut savoir que le compilateur parcoure ton code un peu à la manière dont tu lis un livre : de la première ligne de la première page, à la dernière ligne de la dernière page.

Tout comme toi qui ne sais pas ce qui va se passer à la page 17 quand tu n'es qu'à la page 10, le compilateur n'a connaissance que de ce qu'il a déjà rencontré.

Quand il arrive à ta ligne 21, tu lui parle d'un type nommé "Arme", mais, comme il n'a encore rencontré ni une déclaration ni (encore moins) une définition de ce type, il te fait la seule réponse qu'il peut donner dans ce cas: désolé, je ne sais pas de quoi tu parles avec ton type "Arme".

Et pour cause, il ne saura que "Arme" est une classe qu'à la ligne 37.

La solution "de simplicité" consiste à, simplement, remonter la définition de "Arme" avant celle de Guerrier, mais...

Je vois venir le coup gros comme une maison: tôt ou tard, tu voudras créer des spécialisations de Armes (faire la distinction entre les haches à une main, les flamberges, les marteaux de guerre et les haches à deux mains).

Et comme ce seront à chaque fois des armes, tu envisageras, à juste titre, de faire hériter ces différentes armes de ta classe Arme, et tu voudras très certainement profiter du polymorphisme.

Seulement, voilà, le polymorphisme ne peut fonctionner que sur des pointeurs (de préférence intelligents) ou sur des références, pas sur des instances "classiques".

L'avantage, c'est que le compilateur sait parfaitement quelle taille font les pointeurs (car ce n'est jamais que adresse mémoire à laquelle on va trouver l'objet du type indiqué) et les références (car, au niveau du processeur, ca fonctionne comme un pointeur), et, surtout, que le compilateur n'a réellement besoin de connaitre le type (non seulement de savoir qu'il existe, mais aussi de quoi il est composé) que lorsqu'on essaye d'accéder à une partie de son contenu.

Tu pourrais donc aussi envisager de:

1. créer un fichier d'en-tête séparé pour Arme (car tu auras sans doute à connaitre Arme sans avoir forcément besoin de connaitre Guerrier )
2. Effectuer une déclaration anticipée de Arme dans le fichier d'en-tête de Guerrier
3. fournir un pointeur (de préférence intelligent) sur Arme à ton guerrier
4. Si ton guerrier manipule directement l'arme dans l'une de ses fonctions, inclure le fichier d'en-tête de Arme dans le fichier d'implémentation de Guerrier

[djun1]

Une erreur : fichier include trop nombreux.

Arme.h

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
#include "Arme.h"
 
class Arme
{
private:
	std::string nomArme;
	int forceArme;
	int poid;
 
public:
	void changerArme(std::string nouveauNom, int nouveauForce, int nouveauPoid,std::string &nomArme , int &forceArme, int &poid);
	Arme(Guerrier &classe);
};

Personnage.h

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
#include "Arme.h"
class Guerrier
{
private:
	int hp;
	int sp;
	int xp;
	int level;
	int argent;
	std::string classe;
	std::string nom;
	std::string technique1;
	std::string technique2;
	std::string technique3;
	int force;
	int rapidite;
	int energie;
	int defense;
	Arme monArme;
 
public:
	Guerrier();
	int getForce(int &force);
	void recoveryHP(int &hp);
	void recoverySP(int &sp);
	void gagnerXp(int &xp);
	void changerLVL(int &xp);
	void reatsu(std::string &technique1);
	void reveille(std::string &technique2);
	void bankai(std::string &technique3);
	void seFaireFrapper(int &defense);
	bool vivant() const;
	void dommage();
};

body.cpp

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
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
#include <iostream>
#include <string>
#include "Personnage.h"
using namespace std;
 
void Guerrier::recoveryHP(int &hp)
{
	hp += 20;
	cout << "Hp recovery";
}
 
void Guerrier::recoverySP(int &sp)
{
	sp += 20;
	cout << "Sp recovery";
}
 
bool Guerrier::vivant() const
{
	if(hp > 0)
	{
		return true;
	}
	else
	{
		return false;
	}
}
 
Guerrier::Guerrier()
{
	hp = 100;
	sp = 100;
	xp = 0;
	level = 1;
	argent = 8000;
	force = 250;
	rapidite = 250;
	energie = 250;
	defense = 250;
	cout << "Son nom : " << endl;
	getline(cin,nom);
	cout << "Job : Soldat, Raider, Civile, Policier, Agent" << endl;
	cin >> classe;
	if(classe == "Soldat")
	{
	force += 125;
	rapidite += 125;
	energie += 125;
	defense += 125;
	}
	else if(classe == "Raider")
	{
	force += 125;
	rapidite += 125;
	energie += 125;
	}
	else if(classe == "Policier")
	{
	rapidite += 125;
	}
	else if(classe == "Agent")
	{
	rapidite += 125;
	energie += 125;
	}
	else
	{
	 classe = "Civile";
	}
	cout << endl << endl << "Information a propos de " << nom << "  : " << endl << "Nom : " << nom << endl << "Hp : " << hp << endl << "Sp : " << sp << endl << "Xp : " << xp << endl << "Niveau : " << level << endl << "Monnaie : " << argent << endl << endl << "Info a propos de la class " << classe << " : " << endl << "Force : " << force << endl << "Rapidite : " << rapidite << endl << "Energie : " << energie << endl << "Defense : " << defense << endl << endl;
	system("PAUSE");
}
 
void Arme::changerArme(std::string nouveauNom, int nouveauForce, int nouveauPoid, std::string &nomArme, int &forceArme, int &poid)
{
	nomArme = nouveauNom;
	forceArme = nouveauForce;
	poid = nouveauPoid;
}
 
int main()
{
	Guerrier Souwn;
}

[koala01]

Voyons!!!!

Regarde un tout petit peu la première ligne de ton fichier Arme.h, et interroges toi (à moins que ce ne soit une erreur de typo )

Ceci dit, lorsque tu travailles avec plusieurs fichiers d'en-tête, il est toujours intéressant de veiller à placer des gardes anti inclusion

[djun1]

Même avec cela. Ça ne fonctionne pas.

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
#include "Personnage.h"
 
class Arme
{
private:
	std::string nomArme;
	int forceArme;
	int poid;
 
public:
	void changerArme(std::string nouveauNom, int nouveauForce, int nouveauPoid,std::string &nomArme , int &forceArme, int &poid);
	Arme(Guerrier &classe);
};

[Ehonn]

Regarde le constructeur de Arme. Pourquoi prend-t-il un Guerrier en paramètre ?