Guter Stil oder 'Wie programmiere ich vernünftig?'

[Lemmy1]

Übrigens: Zum Thema "guter" Stil gehört auch, nicht JEDE Exception abzufangen, sondern nur spezifisch die, die Du abfangen willst. Dann kann man auch meist das try...except...end weiter nach aussen schieben und für die gesamte Funktion auf einmal durchführen...

Böße:

Delphi-Quellcode:

try
  ... some code that goes wrong all the time ...
except
  ... handle here
end;

Gut:

Delphi-Quellcode:

try
  ... some code that goes wrong all the time ...
except
  on E: FileNotFoundException do
  begin
    ... handle here
  end
end;

Man sollte sich auch immer überlegen, welche Exceptions man abfangen sollte. Eine ArgumentNullException ist zum Beispiel immer ein Programmierfehler, die in keinen try-except Block gehört. Eine FileNotFoundException hingegen könnte auf Fehlerhafte Config etc hindeuten, also gehört die da rein. In Java werden diese beiden Typen von Exceptions sehr schön getrennt. Die zweite (FileNotFoundException, ComputerTurnedOffException etc) Kategorie MUSS man dann sogar abfangen.

[Lemmy1]

Zitat:

Zitat von TPok

Alles klar. Auf den Blick in die Borland.Delphi.System hätte ich auch kommen können. Das heißt also, wenn das Objekt IDisposable nicht implementiert, ist ein Free überflüssig. Ansonsten sollte man es aufrufen. So dachte ich mir das auch.

Ich werde mich mal an die Umstrukturierung meines Queltextes machen. Bei neuen Frage, melde ich mich bestimmt wieder.

Bis dahin,
Stephan

Denk aber dran, dass Delphi.Net "destructor Destroy; override" automatisch in IDisposable konvertiert.