Discussion :

[takout]

Bonjour à tous,

Après avoir codé une classe shape, j'ai écrit dans un autre fichier source des fonctions qui permettent d'afficher ces shapes. Je souhaiterai qu'il y ait un lien d'amitié entre les fonctions d'affichage et la classe shape.

J'ai donc fait la chose suivante :
-shape.hh

Code :

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

1
2
3
4
5
6
7
8
9
 
#include "plot.hh"
 
class Shape
{
   ....
  friend void plot( Shape const& s);
  ....
};

-plot.hh

Code :

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

1
2
3
4
5
 
 
class Shape;
 
void plot(Shape const& s);

-plot.cc

Code :

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

1
2
3
4
5
6
7
 
#include "plot.hh"
 
void plot(Shape const& s)
{
...
}

malgré le fait que dans plot.hh j'ai déclaré la classe Shape j'obtiens comme message d'erreur : invalid use of incomplete type ‘const struct Shape’.
Ce message est du au fait que j'utilise dans la fonction plot un acces direct aux attributs de s. Pourtant ma fonction plot est amie à la classe shape.

Pouvez-vous m'aidez svp ?

[Neckara]

Bonjour,

Tu as oublié d'include Shappe.hh dans Plot.cc
Tu n'as fait que déclarer la classe Shappe dans Plot.hh mais tu ne l'a pas défini.

[GeantVert13]

Je pense que ça marchera si tu fais #include shape.h dans ton plot.cc

Edit : pffff grillé par neckara... Cela dit je pense qu il vaut mieux l inclure dans le cc. Ça évitera l inclusion cyclique.

[Neckara]

Je pense que ça marchera si tu fais #include shape.h dans ton plot.cc

Edit : pffff grillé par neckara... Cela dit je pense qu il vaut mieux l inclure dans le cc. Ça évitera l inclusion cyclique.

En effet je me suis trompé en écrivant
Je corrige cela tout de suite.

[takout]

J'ai testé l'inclusion de l'header plot.h dans shape.cc mais j'ai toujours le même message d'erreur.

[Neckara]

J'ai testé l'inclusion de l'header plot.h dans shape.cc mais j'ai toujours le même message d'erreur.

Peux-tu nous montrer la ligne de l'erreur ?

[GeantVert13]

J'ai testé l'inclusion de l'header plot.h dans shape.cc mais j'ai toujours le même message d'erreur.

Hmmm relis les réponses. C shape.h qu il faut inclure dans plot.cc

[takout]

Euh oui dsl c'est bien shape.h que j'ai mis dans plot.c.

voici le message d'erreur en entier :

Code :

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

1
2
3
4
 
Shapes/source/plot.cc: In function ‘void plot(const Shape&, int, bool)’:
/Shapes/source/plot.cc:8:5: erreur: invalid use of incomplete type ‘const struct Shape’
compilation terminated due to -Wfatal-errors.

j'ai mis un namespace dans Shape.cc et Shape.hh serait ce la cause de mon problème ?

[GeantVert13]

Peux-tu nous montrer la ligne de l'erreur ?

Ainsi que le code complet. Pour voir ou tu en es.

[takout]

Le code complet ça sera difficile car j'utilise beaucoup de méthode et de librairies. J'ai simplifié pour que vous puissiez comprendre mon problème.

[GeantVert13]

Euh oui dsl c'est bien shape.h que j'ai mis dans plot.c.

voici le message d'erreur en entier :

Code :

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

1
2
3
4
 
Shapes/source/plot.cc: In function ‘void plot(const Shape&, int, bool)’:
/Shapes/source/plot.cc:8:5: erreur: invalid use of incomplete type ‘const struct Shape’
compilation terminated due to -Wfatal-errors.

j'ai mis un namespace dans Shape.cc et Shape.hh serait ce la cause de mon problème ?

Ça dépend si plot est dans le même namespace c est bon. Sinon il faut spécifier avec using namespace tonNS; ou utiliser tonNS::Shape quand tu veux acceder à ta classe shape

[takout]

Merci pour votre aide, c'est le namespace qui causait le message d'erreur.

[takout]

J'aurais une autre question, pour protéger des inclusions cycliques il ne suffit pas de mettre les directives de preprocessing #ifndef ... , #define ..., #endif dans le header ?

[GeantVert13]

J'aurais une autre question, pour protéger des inclusions cycliques il ne suffit pas de mettre les directives de preprocessing #ifndef ... , #define ..., #endif dans le header ?

oui, ça protègera des inclusions cycliques mais il n'y en avait pas dans ton code. De plus, si tu peux éviter des include, autant le faire, ça peut faire gagner du temps de compilation (en utilisant ls forward declaration)