Saubere Programmierung mit Delphi

[sx2008]

Ich würde die Klasse TBackground nicht mit einem Formular in Verbindung bringen, sondern nur mit einem Canvas. Die Größe des Canvas wird dann über 2 Properties Width & Height mitgeteilt.
Vorteil: die Klasse arbeitet auch mit Printer.Canvas und TImage.Canvas zusammen.

Ausserdem sollten die beiden Farben auch als Property freigelegt werden; die Übergabe im Konstruktor ist so ja ziemlich restriktiv.

Ausserdem sollten deutlich mehr Background-Muster (Karos, Linien, Dreiecke, Sechsecke) programmiert werden, damit sich der Aufwand für eine Klasse überhaupt lohnt.

[Christian Seehase]

Moin Sebastian,

Zitat von Phoenix:

Des öfteren Sucht man eine Deklaration um schnell dort hin zu gelangen. Ich würde dann z.B. immer nach "FColor:" suchen.

Um eine Deklaration zu finden genügt es doch die entsprechende Variable mit STRG anzuklicken.

[Muetze1]

Zitat von Christian18:

Muetze1, ich habe da mal ne Frage. Warum machst du alle Eigenschaften der Klasse TBackground über property öffentlcih und die Form nicht?

Wie könnte ich denn, wo mein Vorschlag die Form nicht in der Klasse hält? Die Klasse malt was auf eine Form - warum sollte ich die Klasse auf eine Instanz festlegen und somit beschränken? Somit übergebe ich einfach die Form in den beiden Routinen. Damit hat die TBackground Instanz immer die Farb- bzw. Hintergrundeinstellungen und kann dann auch x beliebige Formulare angewendet werden.

Den Constructor habe ich entfernt, weil er zu restriktiv ist, wie auch schon sx2008 geschrieben hat. Du kannst die Klasse anlegen und die farblichen Einstellungen jederzeit ändern bzw. anpassen.

@sx2008: Das mit dem Canvas war auch mein erster Ansatz, aber er nutzt die Color Eigenschaft der Form, was auch entsprechend Windows als Window Color mitgeteilt wird. Von daher konnte ich mich nicht auf ein TCanvas beschränken.