Ich stehe gerade vor folgendem Problem.
Ich habe zwei Klassen "TSCEngine" und "TSCNet"

sc_engine.h:

#ifndef sc_engineH
#define sc_engineH

#include "sc_net.h"

class TSCEngine : public TObject
{
  private : TSCNet *Net;
  ...
}
#endif

sc_net.h:

#ifndef sc_netH
#define sc_netH

class TSCEngine;

class TSCNet : public TObject
{
  ...
  public : TSCNet(TSCEngine *Owner);   
}

#endif

Wenn ich das richtig verstanden habe, sollte

class TSCEngine;

als Forward-Include TSCEngine vorab bekannt machen. Die tatsächliche Klasse wird durch das includen der "sc_engine.h" in der "sc_net.cpp" realisiert.

Leider funktioniert das nicht.

Zwar scheint TSCEngine in der "sc_net.h" bekannt zu sein, dafür wirft mir der Compiler haufenweise Fehler in der "sc_net.cpp" und sagt mir, dass die Methoden, die eigentlich zu TSCNet gehören sollten nicht zu dieser Klasse gehören:

[C++ Error] sc_net.cpp(13): E2316 '_fastcall TSCNet::{(int)' is not a member of 'TSCNet'
[C++ Error] sc_net.cpp(25): E2316 '_fastcall TSCNet::{(int,AnsiString)' is not a member of 'TSCNet'
[C++ Error] sc_net.cpp(52): E2316 'RecieveText' is not a member of 'TCustomWinSocket'
[C++ Error] sc_net.cpp(56): E2379 Statement missing ;
[C++ Error] sc_net.cpp(59): E2089 Identifier 'Write' cannot have a type qualifier
[C++ Error] sc_net.cpp(125): E2139 Declaration missing ;
[C++ Error] sc_net.cpp(125): E2134 Compound statement missing }

Was mache ich hier falsch?

eine "forward declaration" ersetzt nicht in allen Fällen die vollständige Deklaration !
Sobald Du die konkrete Deklaration einsetzt (z.B. beim Zugriff auf Member), brauchst Du sie auch.
Oder anders ausgedrückt: Für einen Zeiger oder eine Referenz reicht eine "forward declaration" ... für mehr eigentlich nicht.
Aus so einer "Falle" kommst Du, in dem Du die Implementierung der Memberfunktionen von der Klassendeklaration trennst:

// A.h
class B;

class A {
   B& meinB;
public:
   A(B& b);
   void do();
};

// A.cpp
#include "B.h" // erst hier includieren

A::A(B& b) : meinB(b) {}
void A::do() { meinB.doMich(); }

Ebenso andersherum: A.h nur in B.cpp inkludieren.
Schwierig wird's, wenn eine von der andern erben muss (aber da es keine "gegenseitige Vererbung" geben kann, muss dann eben doch das eine .h im Anderen inkludiert werden)

Gruß,

Simon2.

Ich habe nur leider diese Querabhängigkeit meiner zwei Klassen.
Wie bekomme ich das hin, dass das trotzdem funktioniert?

In der "sc_net.cpp" wird ja im Übrigen die "sc_engine.h" includiert.
Das sollte doch dann genügen, oder irre ich mich?

Hallo zusammen!

als Forward-Include TSCEngine vorab bekannt machen. Die tatsächliche Klasse wird durch das includen der "sc_engine.h" in der "sc_net.cpp" realisiert.

Leider funktioniert das nicht.

Zwar scheint TSCEngine in der "sc_net.h" bekannt zu sein, dafür wirft mir der Compiler haufenweise Fehler in der "sc_net.cpp" und sagt mir, dass die Methoden, die eigentlich zu TSCNet gehören sollten nicht zu dieser Klasse gehören:

[C++ Error] sc_net.cpp(13): E2316 '_fastcall TSCNet::{(int)' is not a member of 'TSCNet'
[C++ Error] sc_net.cpp(25): E2316 '_fastcall TSCNet::{(int,AnsiString)' is not a member of 'TSCNet'
[C++ Error] sc_net.cpp(52): E2316 'RecieveText' is not a member of 'TCustomWinSocket'
[C++ Error] sc_net.cpp(56): E2379 Statement missing ;
[C++ Error] sc_net.cpp(59): E2089 Identifier 'Write' cannot have a type qualifier
[C++ Error] sc_net.cpp(125): E2139 Declaration missing ;
[C++ Error] sc_net.cpp(125): E2134 Compound statement missing }

Was mache ich hier falsch?

Binde mal in der sc_net.cpp auch die komplette sc_engine.h ein:
#include "sc_engine.h"

Ansonsten verstehe ich die Fehlermeldung "'TSCNet::{(int)' is not a member of 'TSCNet'" nicht, daher wäre es für mich sinnvoll den Code mal zu sehen.

mein geändertes Post gesehen (s.o.) ?

Gruß,

Simon2.

sc_engine.h:

#ifndef sc_engineH
#define sc_engineH

class TSCNet;

class TSCEngine : public TObject
{
  private : TSCNet *Net;
  ...  

}
#endif

sc_engine.cpp

#include <vcl.h>
#pragma hdrstop

#include "sc_engine.h"
#include "sc_net.h"

sc_net.h

#ifndef sc_netH
#define sc_netH

#include "scktcomp.hpp"
#include "coords.h"

class TSCNet : public TObject
{
  ...
  public : 
    __fastcall TSCNet(TSCEngine *owner, int Port);
    __fastcall TSCNet(TSCEngine *owner, int Port, String Host);
};
#endif

sc_net.cpp

#include <vcl.h>
#pragma hdrstop

#include "sc_engine.h"

Jetzt meckert er, dass die zwei Konstruktoren von TSCNet keine Member der Klasse TSCNet sind und
"Undefined symbol TObject" in einer Zeile, in der es um TObjectTCustomWinSocket geht.

Achja, ist zwar OT, aber wenn du designtechnisch dazu was hören willst, sag bescheid

sc_net.cpp

#include <vcl.h>
#pragma hdrstop

#include "sc_engine.h"

Fehlt da nicht noch ein #include "sc_net.h" ?

hmm.. diese includeanweisung steht ja jetzt in der sc_engine.cpp.
Oder hab ich das von oben jetzt komplett falsch verstanden?

@filmor
da lasse ich mich gerne belehren!
das __fastcall steht da, um die Konstruktoren der TObject Klasse überschreiben zu können. oder wäre das hier unnötig gewesen?

die genauen Fehlermeldungen:

[C++ Error] 
sc_net.cpp(14): E2316 '_fastcall TSCNet::{(int)' is not a member of 'TSCNet'
[C++ Error] sc_net.cpp(26): E2316 '_fastcall TSCNet::{(int,AnsiString)' is not a member of 'TSCNet'
[C++ Error] sc_net.cpp(36): E2451 Undefined symbol 'TObject'

Die Codestellen dazu:

Zeile 14:

__fastcall TSCNet :: TSCNet(int Port)
{
...

Zeile 26:

__fastcall TSCNet :: TSCNet(int Port, String Host)
{
...

Zeile 36:

void __fastcall TSCNet::OnClientRead(TObject *Sender, TCustomWinSocket *Socket)
{
...

In Zeile 36 wird das TCustomWinSocket angemeckert

Edit:
Gut... nach langem Überlegen und Kopf freibekommen bin ich gerade darauf gestoßen. Es tat richtig Weh:

Headerdatei:

class TSCNet : public TObject
{
  public : __fastcall TSCNet(TSCEngine *owner, int Port);
}

Cpp- Datei:

__fastcall TSCNet::TSCNet(int Port) 
{
...

Danke trotzdem an alle, die mitgeholfen haben.
Insbesondere Simon2, für die tolle Erklärung!

da lasse ich mich gerne belehren!

Na dann ...
1. Präfixe sind unpraktisch. Benutz stattdessen Namensräume. Präfixe, die die "Art" des Namens angeben find ich persönlich unpraktisch, das ist aber mehr Geschmackssache.
2. Es gibt sehr selten triftige Gründe, /jede/ Klasse von einer bestimmten Klasse (bei dir TObject) abzuleiten. Haben wirklich alle Objekte bei dir in irgendeinem Bereich gleiche Eigenschaften?
3. Benutz' keine compiler- oder plattformspezifischen Schlüsselworte wenn du nicht musst. Bist du dir sicher, dass du weißt, was __fastcall bedeutet? Bist du dir sicher, dass du es brauchst? Wenn nicht, dann weg damit. Lass die Optimierungen mal ruhig von deinem Compiler machen.

Damit könnte dein Code in etwa so aussehen:

// sc/engine.hpp
#ifndef SC_ENGINE_HPP // .h benutzt man auch für C, hpp oder hxx ist eindeutiger
#define SC_ENGINE_HPP // Groß geschrieben, um nicht mit anderen Bezeichnern
                      // ins Gehege zu kommen

namespace sc
{
    class Net;

    class Engine
    {
    private:
        Net* _net;
        ...
    };
}
#endif

// engine.cpp
#include <vcl.h>
#pragma hdrstop

#include "engine.hpp"
#include "net.hpp"

namespace sc
{
   ...
}

// net.hpp
#ifndef SC_NET_HPP
#define SC_NET_HPP

#include "socketcomp?.hpp" // Du kannst ruhig längere Namen verwenden, 8.3 ist
#include "coords.hpp"      // nicht mehr erforderlich ;)

namespace sc
{

    class Net
    {
        ...
    public:
        Net(Engine* owner, int Port);
        Net(Engine* owner, int Port, String Host);
    };
}

#endif

// net.cpp

#include <vcl.h>
#pragma hdrstop

#include "engine.hpp"

namespace sc
{
    ...
}

Danke!!
Dadurch schaut das gleich um 150% besser aus!

Nachmal wegen dem __fastcall:

Ich will, dass durch ein Event eine bestimmte Funktion aufgerufgen wird und weise zu :

Client->OnRead = this->OnRead;

Dabei muss meine Funktion TSCNet::OnRead die Direktive __fastcall haben, da ich sie sonst nicht zuweisen kann. Ansonsten hätte ich das so auch nicht gemacht...