- Close() wordt aangeroepen door de methode Dispose(), dus zolang u "using" correct gebruikt, hoeft u Close() niet expliciet aan te roepen.
- Als het mogelijk is dat een NULL wordt geretourneerd, moet u dit controleren. Ik zou adviseren dat als je de code beheert die de sql-verbinding krijgt, je sterk overweegt een uitzondering te maken in plaats van NULL terug te geven. Anders kunnen andere ontwikkelaars tegen dezelfde soort problemen aanlopen. Het niet openen van een vereiste SQL-verbinding lijkt mij een geldig uitzonderingsgeval. Indien nodig kunt u altijd een TryConnectOneTime opnemen voor wanneer een ontwikkelaar het afhandelen van uitzonderingen wil vermijden.
Ook een andere stijlopmerking - ik zou adviseren om uw gebruiksverklaringen goed te ondersteunen voor het geval er een verdwaalde extra regel wordt toegevoegd en er een onverwachte fout optreedt. Qua stijl heb ik meestal niet de neiging om uitspraken te laten inspringen als ik meerdere uitspraken bij elkaar heb, maar dat hangt allemaal af van persoonlijke voorkeur.