Moin K-H,
um Manuels Aussage noch etwas zu präzisieren:
Die Variante funktioniert zwar (wobei, wie bei DeddyH der Resourcenschutzblock für CloseKey fehlt
), aber wenn man ein Objekt als Rückgabewert einer Funktion nutzt, kann man einfach zu leicht Speicherlecks erzeugen.
Wenn man es so macht, wie Du es gezeigt hast, gehört das Try übrigens hinter den Funktionsaufruf, und nicht davor, denn wenn das Erzeugen des Objektes fehlschlägt, gibt es auch nichts freizugeben.
Die Übergabe als const-Parameter, wie bei DeddyH, ist wohl die sinnvollste Variante, da man hier am leichtesten die Übersicht waren kann, dass das Objekt auch wieder freigegeben wird, und auch nicht, wie bei var-, out- oder Wert-Parametern, versehentlich dem Parameter ein Objekt zuweisen kann.
Meiner Ansicht nach gibt es nur einen Ausnahmefall für diese Regel, nämlich wenn das Objekt von einer Methode erzeugt wird, dessen Klasse die zurückgegebenen Objekte selber verwaltet.
Als Beispiel führe ich hier gerne TListView.Items.Add an.
Diese Methode gibt ein neu erzeugtes Objekt zurück, aber die Klasse TListView verwaltet diese dann auch selber.