Einzelnen Beitrag anzeigen

Benutzerbild von himitsu
himitsu

Registriert seit: 11. Okt 2003
Ort: Elbflorenz
43.177 Beiträge
 
Delphi 12 Athens
 
#7

AW: Besprechnungsanfrage (iCalendar)

  Alt 23. Nov 2013, 23:14
Du solltest drinngend nochmal einen Kurs über OOP belegen.

Wozu gibt es diese globale Variable CD?
Wenn, dann würde diese maximal als Feld in die Form gehören.
Aber eigentlich wird diese Variable nur in Button1Click verwendet, also gehört sie dort auch rein.

Und schon würde dir auffallen, daß du hier total Mist gebaut hast.
Delphi-Quellcode:
TCalendarData.CreateInvitation(IdMessage: TIdMessage);
begin
  with cd.cdBody do
Genauso wie einem etwas komisch vorkommen sollte, wenn ma die Variable Form1, bei soeinen einfachen Code aus nur einer Unit/Form, niemals irgendwo verwendet werden sollte (außer es ist wirklich notwendig, was es aber fast niemals ist)

Also, wieso wird da innerhalb der Klasse TCalendarData auf eine externe Variable (CD) zugegriffen, anstatt auf seine eigenen Felder?
Die globale Variable CD entspricht hier zwar zufällig mal der eigenen Instanz, aber das muß nicht immer so sein.

Auch wird die Variable cdBody nur in CreateInvitation verwendet, womit Diese auch nur in dieser Methode deklariert sein sollte.
Obwohl man da eigentlich auch direkt IdMessage.Body verwenden könnte.
Garbage Collector ... Delphianer erzeugen keinen Müll, also brauchen sie auch keinen Müllsucher.
my Delphi wish list : BugReports/FeatureRequests

Geändert von himitsu (23. Nov 2013 um 23:18 Uhr)
  Mit Zitat antworten Zitat