Neue Klasse erstellen mit 2 Klassen als Attribute

[shmia]

Zitat:

Zitat von Coffeecoder (Beitrag 1140269)

Allerdings kommt mir da eine Frage auf: Vom TComTerminal brauch ich lediglich nur eine Methode onStrReceive, weil die TComPort Klasse nur die nötigen Parameter für die Kommunikation nötig sind.
Folglich muss ich in deinem Beispiel TWettersation doch noch die TComTerminal haben.

Ich habe mich schon am Anfang gewundert, weshalb du TComTerminal noch ins Spiel bringst.
Mit TComTerminal emuliert man VT100, VT52 oder Ansi-Terminals die noch aus dem dunklen Mittelalter der Computertechnik stammen.
Da du kein Terminal hast (hoff' ich doch), brauchst du diese Klasse nicht.

[Coffeecoder]

Zitat:

Zitat von shmia (Beitrag 1140283)

Da du kein Terminal hast (hoff' ich doch), brauchst du diese Klasse nicht.

Nein das nicht :)
Ich werde mal deinen Vorschlag versuchen und Bescheid geben. Danke für deine Hilfe :thumb:

[DeddyH]

Wäre es aber in dem Fall nicht sinnvoller, gleich von TComPort abzuleiten und die paar zusätzlichen Methoden/Properties zu deklarieren? Ansonsten müsste man ja bei jedem Zugriff auf den ComPort mit Assigned() prüfen, ob er überhaupt zugewiesen wurde und kann selbst dann nicht sicher sein, dass es ihn auch wirklich noch gibt, er kann ja zugewiesen, aber trotzdem zwischendurch freigegeben, aber nicht genilt worden sein.

[Coffeecoder]

Zitat:

Zitat von DeddyH (Beitrag 1140285)

Wäre es aber in dem Fall nicht sinnvoller, gleich von TComPort abzuleiten und die paar zusätzlichen Methoden/Properties zu deklarieren?

Genau das ist mir auch gerade in den Sinn gekommen :)
Ich habe mir mal die Dokumentation für die TComport von Dejan Crnila runtergeladen und tatsächlich steht doch eine Methode zum Empfangen in der Klasse drin.

Als Resultat habe ich dann dies:

Delphi-Quellcode:

TWetterstation = class(TComport)
public
  constructor Create();
  procedure ReadWetterdaten;
  procedure SendWetterdaten(b: byte);
end;

... // ich kürze mal ab

Wir sind schon einige Schritte weiter als am Anfang. Danke für eure Hilfe :thumb:

[shmia]

Zitat:

Zitat von DeddyH (Beitrag 1140285)

Wäre es aber in dem Fall nicht sinnvoller, gleich von TComPort abzuleiten ...

Vererben oder Benutzen wäre hier also die Frage.
Ich sehe öfters mal, dass vererbt wird obwohl gar keine so enge Verwandschaft besteht.

Beispiel:
Es gibt die Klasse TField (Basisklasse) und TStringField (abgeleitete Klasse).
Hier kann man ganz klar sagen ein TStringField ist eine spezielle Variante von TField.

Bei der Wetterstation trifft dies nicht zu.
Die Wetterstation ist keine spezielle Variante von TComPort sondern
die Wetterstation benötigt ein TComPort-Objekt zur Kommunikation mit dem realen Messwertgeber.

[DeddyH]

Was mich an Deinem Vorschlag stört ist die Tatsache, dass eine Property der Klasse von außerhalb verwaltet wird. Das kann im dümmsten Fall in AVs enden (OK, nicht sehr wahrscheinlich, aber wer weiß schon immer vorher, was der Endbenutzer mit so einer Klasse anstellt, und wenn man es selbst ist?).

[Coffeecoder]

Zitat:

Zitat von shmia (Beitrag 1140344)

Ich sehe öfters mal, dass vererbt wird obwohl gar keine so enge Verwandschaft besteht.

Beispiel:

Bei der Wetterstation trifft dies nicht zu.
Die Wetterstation ist keine spezielle Variante von TComPort sondern
die Wetterstation benötigt ein TComPort-Objekt zur Kommunikation mit dem realen Messwertgeber.

Gutes Beispiel. Ich brauch die TComPort Klasse nur zur Kommunikation mit dem Messgerät/Wetterstation. Da reicht es natürlich aus, wenn ich das TComPort als Attribut deklariere und benutze. Zusätzlich brauche ich noch eine Liste um einige Korrekturen vorzunehmen bzw zu formatieren um die als Datei abzuspeichern.

Aber mal angenommen, ich hätte mehrere Wetterstationen mit unterschiedlichen Protokollen wäre es nicht sinnvoll doch zu erben? Ihr fragt euch nun was soll das mit den Protokollen nunja jede write-Methode braucht andere Zeichnen / Kommandos um mit dem entsprechenden Gerät zu kommunizieren.

[Sir Rufo]

Die Klasse sollte selber eine ComPort Instanz besitzen.

Wird von aussen eine ComPort Instanz übergeben, dann sollten von der nur die Einstellungen übernommen werden (Assign).

Das mit dem Vererben und den unterschiedlichen Protokollen kann man durch eine BasisKlasse und einer abstrakten Methode lösen.
In der konkreten Klasse wird diese Methode dann entsprechend dem Protokoll deklariert.

[Coffeecoder]

Als Zusammenfassung kann man behaupten, dass in diesem konkreten Beispiel eigentlich egal ist ob man erbt oder die Klasse ComPort als Attribut deklariert.

Doch muss ich sagen, dass meine Frage trotzdem beantwortet ist und dass ihr mit gut geholfen habt. Dafür danke ich allen :) :thumb:

Ich werde einfach mal in diesem Fall einmal mit Vererbung und einmal als Attribut die Klasse benutzen und sehen was "am besten" ist.

Danke euch :)