- Close() anropas av Dispose()-metoden, så så länge du använder "använder" på rätt sätt behöver du inte anropa Close() explicit.
- Om det är möjligt för en NULL att returneras, bör du leta efter det. Jag skulle råda dig att om du kontrollerar koden som får sql-anslutningen, överväger du starkt att kasta ett undantag istället för att returnera NULL. Annars kan andra utvecklare stöta på samma typ av problem. Att inte öppna en SQL-anslutning som krävs verkar för mig vara ett giltigt undantagsfall. Om det behövs kan du alltid inkludera en TryConnectOneTime för när en utvecklare vill undvika undantagshantering.
Också en annan stilanteckning - jag skulle råda dig att stärka dina användningssatser ordentligt om en extra rad läggs till och ett oväntat fel inträffar. Stilmässigt brukar jag inte göra indrag med påståenden när jag har flera påståenden tillsammans, men det beror helt på personliga preferenser.