- Close() je volána metodou Dispose(), takže pokud používáte "používáte" správně, nemusíte Close() volat explicitně.
- Pokud je možné vrátit hodnotu NULL, měli byste ji zkontrolovat. Doporučil bych, že pokud ovládáte kód, který získá připojení sql, důrazně zvažte vyvolání výjimky spíše než vrácení NULL. V opačném případě mohou ostatní vývojáři narazit na stejný druh problémů. Selhání při otevření požadovaného SQL připojení se mi jeví jako platný výjimečný případ. V případě potřeby můžete vždy zahrnout TryConnectOneTime, když se vývojář chce vyhnout zpracování výjimek.
Také další poznámka ke stylu - doporučoval bych správně ztuhnout vaše příkazy using pro případ, že by byl přidán zbloudilý řádek navíc a dojde k neočekávané chybě. Pokud jde o styl, obvykle nemám tendenci odsazovat příkazy, když mám více příkazů pohromadě, ale to vše závisí na osobních preferencích.