- Ich würde Server und Desktop als lokale Referenzen halten bis klar ist, dass nichts schiefgelaufen ist. Erst dann den Klassenfeldern zuordnen.
Dann spiele ich mal FxCop/StyleCop für Delphi:
- Top-Level Exceptions sollte man nicht fangen, ausser man re-raised.
- Die Codeformatierung entspricht keiner gängigen Richtlinie.
- Hardcodierte Strings.
- isCellSelected Funktion in globalem Scope; Könnte man als Class-Helper gestalten oder sonst in eine Hilfsklasse tun.
- isCellSelected: Du weist dem Resultat mehrfach etwas zu. Du könntet das if weglassen und eine direkte Zuweisung machen.
- ConnectOpOff: Du weist dem Resultat mehrfach etwas zu.
- (angenommen ConnectOpOff ist nicht private): Du validierst stringgrid als Referenz nicht, bevor du sie verwendest.