Discussion :

[Tipha]

Bonjour,
Voici mon problème:
J'ai une classe abstraite nommée : GraphLayout qui possède la méthode suivante: virtual void placement(Graph &graph) =0;
J'ai une des ses classe filles nommée : StraightLayout qui implémente la méthode ci-dessus.

Dans une 3ème classe, nommée Graph, j'ai en attribut : GraphLayout *layout;
Je peux récupérer ce layout avec getLayout();
Et je souhaite faire dans une de ces méthodes :

Code :

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

1
2
GraphLayout* l = this->getLayout();
l->placement(this);

Et là :erreur suivante:

Dans Graph.h

Code :

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

1
2
class GraphLayout;// =>graph.h:20: warning: forward declaration of `struct GraphLayout'
class Graph{...}

Dans Graph.cpp

Code :

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

1
2
3
l->placement(this); 
//=>graph.cpp:341: warning: invalid use of undefined type `struct GraphLayout'
//graph.cpp:341: note: neither the destructor nor the class-specific operator delete will be called, even if they are declared when the class is defined.

Pouvez-vous m'aider s'il vous plaît?? Merci beaucoup parce que je ne comprends vraiment pas le problème...

[Feriaman]

Bonjour,

je crois que la question que tu poses, sans le savoir est la suivante :
Comment créer 2 classes qui font référence l'une à l'autre ?

Il se trouve que ça tombe bien, deux excellents développeurs y ont déjà répondu.

Bonne chance.

[Tipha]

Je te remercie beaucoup en effet, c'était ça et je ne le savais pas...
En plus, la solution est hyper simple : faire une inclusion dans le .cpp...
J'ai bien fait de poster sur le forum, j'aurais jamais trouvé sinon. Encore merci.

[Tipha]

Sauf que maintenant j'ai un autre problème...
Comme vous avez vu, j'ai la signature suivante: void placement(Graph &graph);
En effet, je veux que mon objet de type Graph soit modifier donc je le passe par référence..

Cependant, quand je fais dans ma classe Graph: l->placement(this);
J'obtiens l'erreur suivante:

Code :

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

1
2
graph.cpp:104: error: no matching function for call to `GraphLayout::placement(Graph* const)'
graphlayout.h:11: note: candidates are: virtual void GraphLayout::placement(Graph&)

Donc j'ai bien compris que je ne luis passe pas ce qu'il faut..mais je ne sais pas ce qu'il faut lui passer...
Please, help!

[stillman]

En C++, this est un pointeur, en l'occurence un Graph *. Or, ta méthode attend une référence vers un Graph, il faut donc que tu lui passes une instance de Graph. Pour ce faire, il faut que tu déréférences le pointeur this :

Code :

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

l->placement(*this);

[Tipha]

Ok. C'est bon maintenant.
Je savais pas que this c'était un pointeur.
Merci beaucoup. et désolée parce que j'imagine, que ce genre d'erreur, n'enrichira pas beaucoup les connaissances des plus adeptes de ce forum.. Encore merci quand même