Discussion :

[djun1]

Bonjour,

je suis un debutant en c++ et je suis entraine de faire un code pour m'entraîner. Il y a une erreur que je n'arrive pas à comprendre :

Il disent qu'a la ligne 17, il y a une faute. Mais, jai faite copier/coller de mon autre code.

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
#ifndef MONSTRE_H
#define MONSTRE_H
#include "raider.h"
 
class Monstre
{
private:
	std::string nom;
	int hp;
	int level;
	int force;
	int rapidite;
	int defense;
	bool estVivant;
 
public:
	void attaquer(Raider &ennemi);
	int getDefense() const;
	void seFaireAttaquer(int x);
	int getHp() const;
	int getLevel() const;
};
 
#endif

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
#ifndef RAIDER_H
#define RAIDER_H
#include "Monstre.h"
 
class Raider
{
private:
	std::string nom;
	int hp;
	int sp;
	int xp;
	int level;
	double argent;
	std::string classe;
	int force;
	int rapidite;
	int defense;
	std::string armure;
	std::string Arme;
	bool estVivant;
	int cadence;
	bool quelArme;
	int boost;
 
public:
	Raider();
	void seSoigner();
	void seRecuperer();
	void attaquer(Monstre &cible);
	void changerLevel();
	void attaqueSpecial(Monstre &cible);
	void changerArme();
	int getForce() const;
	void perdreSp(int x);
};
#endif

[LinuxUser]

T'as une dépendance circulaire là.
Monstre dépend de Raider et Raider dépend de Monstre.

Est-ce vraiment ce que tu voulais faire?

[jblecanard]

Hello

Dans la mesure du possible, lorsque dans un header, tu n'utilises que des pointeurs ou des références vers une classe qui n'est pas déclarée dans ce header, il convient d'utiliser une forward déclaration plutôt qu'un include. C'est on cas ici ! En clair:

Code :

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

1
2
3
4
5
#include "Montre.h" // est à remplace par
class Monstre; // <- ceci
 
#include "raider.h" // est à remplacer par
class Raider; // ceci

Voir l'entrée de la FAQ sur le sujet.. Autre remarque: le nom de ton include raider.h n'a pas de majuscule alors que Monstre.h en a une. Fait attention à garder une cohérence dans tes nommages.

[djun1]

merci