- Close() wird von der Dispose()-Methode aufgerufen, solange Sie also "using" richtig verwenden, müssen Sie Close() nicht explizit aufrufen.
- Wenn es möglich ist, dass NULL zurückgegeben wird, sollten Sie dies überprüfen. Ich würde raten, dass Sie, wenn Sie den Code steuern, der die SQL-Verbindung erhält, dringend erwägen, eine Ausnahme auszulösen, anstatt NULL zurückzugeben. Andernfalls könnten andere Entwickler auf die gleichen Probleme stoßen. Das Versäumnis, eine erforderliche SQL-Verbindung zu öffnen, scheint mir ein gültiger Ausnahmefall zu sein. Bei Bedarf können Sie immer eine TryConnectOneTime einschließen, wenn ein Entwickler die Ausnahmebehandlung vermeiden möchte.
Außerdem noch ein Stilhinweis - ich würde Ihnen raten, Ihre using-Anweisungen richtig zu verklammern, falls eine verirrte zusätzliche Zeile hinzugefügt wird und ein unerwarteter Fehler auftritt. Was den Stil betrifft, neige ich normalerweise nicht dazu, mit Anweisungen einzurücken, wenn ich mehrere Anweisungen zusammen habe, aber das hängt alles von meinen persönlichen Vorlieben ab.
