Forward Deklaration trotz include?

Simon2

edwood

hmm....ich verwende eh header guards (wollts nur hier beim posten ersparen da es sonst noch unübersichtlicher wird), von daher kanns ja gar nicht sein dass sich die header untereinander mehrmals einbinden, oder?

ich versteh im moment nur nicht wieso m_client einen unvollständigen typen haben soll (Client), ich binde ja genau in diesem file client.h ein und mach acuh zusätzlich die forward deklaration vom typ Client, also wie gibts das dass anscheinend der Typ Client nicht erkannt wird?

Simon2

edwood schrieb:

...mach acuh zusätzlich die forward deklaration vom typ Client, ...

Die würde ich erstmal weglassen ... hilft hier sowieso nichts (weil bei einem Client-Member eine vollständige Deklaration vorliegen muss).

Ansonsten tippe ich auf einen Fehler bei den include-guards... (Vertipper oder so).

Gruß,

Simon2.

edwood

Simon2 schrieb:

Die würde ich erstmal weglassen ... hilft hier sowieso nichts (weil bei einem Client-Member eine vollständige Deklaration vorliegen muss).

das ist ja eins von den merkwürdigen problemen, wenn ich die forward deklaration weglasse kommt wieder die fehlermeldung: "ISO-C++ verbietet Deklaration von »Client« ohne Typ". was ich ja nicht versteh weil ich eh den header Client.h einbinde, somit müsste es ja reichen und der Typ "Client" bekannt sein.

hab auch (leider) keine fehler in den guards finden können...

drakon

Also mich persönlich irritiert noch das:

#include "Client.h"
Client::Client() {
   m_load = new LoadImage(this);
}

Warum übergibs du hier einen Zeiger der Klasse selbst? (btw: da würde noch das dazu passende delete fehlen..)

Simon2

edwood schrieb:

Simon2 schrieb:

Die würde ich erstmal weglassen ... hilft hier sowieso nichts (weil bei einem Client-Member eine vollständige Deklaration vorliegen muss).

das ist ja eins von den merkwürdigen problemen, wenn ich die forward deklaration weglasse kommt wieder die fehlermeldung: "ISO-C++ verbietet Deklaration von »Client« ohne Typ". ...

Nö ... das ist im Prinzip dieselbe Fehlermeldung bzw. hat dieselbe Ursache: Wenn der Compiler zu den Zeilen

Client *getClient();
// ...
Client m_client;

kommt, liegt ihm noch keine Klassendefinition von Client vor. Mit der forward-Deklaration bringst Du ihn zwar über der ersten Zeile vorläufig zum Schweigen, aber spätestens bei der zweiten geht's einfach nicht mehr weiter.

Versuch doch mal "brute force" und kopiere den Inhalt von Client.h in die Stelle des #includes ...

Gruß,

Simon2.

P.S.: Vielleicht ist auch mit Deinen include-Pfaden was schräg...

edwood

drakon schrieb:

Warum übergibs du hier einen Zeiger der Klasse selbst? (btw: da würde noch das dazu passende delete fehlen..)

wieso irritiert dich das? ich möchte in der klasse LoadImage eine Methode von Client aufrufen (z.b. m_client->methode()), ist das so unüblich? (bin aber leider nicht so der C++-Profi wie man sieht...), btw: delete mach ich eh im destruktor, aber nachdem das nichts mit meinem aktuellen problem zu tun hat hab ichs hier nicht dazu geschrieben.

übrigens steht Client in einem eigenen Namespace (XY), könnte das zu einem Problem führen? hab aber eh überall XY::Client geschrieben, auch bei den forward deklarationen darauf geachtet: "namespace XY { class Client; }". wollts hier nicht extra dazuposten um es etwas übersichtlicher zu lassen und ich nicht da das problem vermute.

nurf

Meiner Meinung nach ergeben sich die Probleme schon durch den falschen Entwurf ( Konzept ).

1. Wenn jede Klasse jede andere kennt, wozu brauchst du dann überhaupt noch verschiedene Klassen ?

2. Sei froh, du hast hier nur 2 Klassen, die Probleme werden verzwickter je mehr Klassen involviert sind --> zyklische Inkludierungen

Lösung :

1. Das Image weiß nichts vom Client !
2. Alle Aktionen gehen vom Client aus.

PseudoCode :

Client::Client()
{
m_Image = new LoadImage();
m_Image->SetValues(...);
}

Gruß

nurF

edwood

nurf schrieb:

Meiner Meinung nach ergeben sich die Probleme schon durch den falschen Entwurf ( Konzept ).

1. Wenn jede Klasse jede andere kennt, wozu brauchst du dann überhaupt noch verschiedene Klassen ?

ok, danke, das war jetzt der hint. habs jetzt hinbekommen, hab einfach die load-image-klasse weggeschmissen und in die client klasse integriert. jetzt hab ich allerdings ne wirklich große (unübersichtliche) client-klasse aber es funktioniert, und das reicht vorerst.
danke nochmals an die leute die mir weitergeholfen haben.

Fellhuhn

Das Problem war übrigens die Ringverkettung. Wenn du in beiden Headern jeweils den anderen inkludierst kann es nicht funktionieren.

Denn er liest A ein und bekommt da ein Include auf B das ein Include auf A hat, dies aber ignoriert dank Includeguards. Nun hast du ein A bei denen nur die ersten Zeilen eingelesen wurden und ein B bei dem die Klassen aus A unbekannt sind (da er nicht soweit kam beim Parsen).

Hätte also wohl reichen dürfen die Definitionen aus den Headern zu entfernen und mit Forward Declarations zu arbeiten.