Discussion :

[Core8]

Bonjour à tous,

C'est la première fois que je fais de l'héritage en C++ et j'ai un problème au niveau des include des .hh.
J'ai quatre classes (avec pour chaque un .cc et .hh (dans le .hh il y a les déclarations des attributs de classes et des méthodes)).
La première est une classe "conteneur" Ecole qui a un tableau de Personne.
La seconde est la classe Personne est la classe mère des deux dernières classes Professeur et Etudiant.

Jusque là ça va mais après pour les include je recontre des problèmes de redéfinition de la classe Personne.
Mon erreur est la suivante :

Code :

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

1
2
3
4
5
In file included from Etudiant.hh:1,
                 from Ecole.hh:2,
                 from Ecole.cc:6:
Personne.hh:5: error: redefinition of `class Personne'
Personne.hh:5: error: previous definition of `class Personne

Dans mes fichiers .hh j'avais comme include :

Code :

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

1
2
3
4
5
6
7
8
9
Ecole.hh
#include "Etudiant.hh"
#include "Professeur.hh"
 
Professeur.hh
#include "Personne.hh"
 
Etudiant.hh
#include "Personne.hh"

Avec ce code c'est sûr que j'ai une redéfinition mais en essayant d'autres façons je suis parvenu à trouver quelque chose qui marche mais qui n'est sûrement pas propre. J'ai dû mettre le contenu (déclarations des attributs de classes et des méthodes) des .hh des classes Professeur et Etudiant dans le .hh de Personne. J'ai supprimé les .hh de Professeur et Etudiant. Donc finalement je n'ai plus que le #include "Personne.hh" dans la classe Ecole.

J'aurais besoin de votre aide pour résoudre ce problème, si vous avez des suggestions n'hésiter pas.

Merci de vos réponses

[zais_ethael]

Tu as bien mis dans chaque fichier .hh les directives:

Code :

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

1
2
3
4
#ifndef ON_SEN_FOUT_TANT_QUE_CEST_UNIQUE
#define ON_SEN_FOUT_TANT_QUE_CEST_UNIQUE
...//tout le reste
#endif

?

[Core8]

Merci zais_ethael de ta réponse.

Non, je n'ai pas mis dans chaque fichier .hh les directives car on ne m'a jamais dit qu'il fallait en mettre que ça soit en C ou en C++.

Donc je vais essayer et je dirais si ça marche.

[Core8]

Je viens de tester et ça marche.

Merci beaucoup de ton aide zais_ethael.

[zais_ethael]

Erreur de débutant, c'est pas grave.
Il faut toujours en mettre partout que ce soit en C ou en C++. Si tu es à cours d'idée pour le nom de la constante on utilise généralement un mot style _NOM_DU_FICHIER_HH_, ça évite les redondances.

[Jean-Marc.Bourguet]

Erreur de débutant, c'est pas grave.
Il faut toujours en mettre partout que ce soit en C ou en C++. Si tu es à cours d'idée pour le nom de la constante on utilise généralement un mot style _NOM_DU_FICHIER_HH_, ça évite les redondances.

On évite d'utiliser des noms qui commencent par _, pas mal sont réservés à l'implémentation de même que ceux qui contiennent deux __.