- Close() kaldes af Dispose()-metoden, så så længe du bruger "bruger" korrekt, behøver du ikke at kalde Close() eksplicit.
- Hvis det er muligt for en NULL at blive returneret, bør du tjekke for det. Jeg vil anbefale, at hvis du styrer koden, der får sql-forbindelsen, overvejer du kraftigt at smide en undtagelse frem for at returnere NULL. Ellers kan andre udviklere løbe ind i de samme slags problemer. Undladelse af at åbne en SQL-forbindelse, der er påkrævet, forekommer mig at være et gyldigt undtagelsestilfælde. Hvis det er nødvendigt, kan du altid inkludere en TryConnectOneTime, når en udvikler vil undgå undtagelseshåndtering.
Også en anden stilbemærkning - jeg vil råde dig til, at du bruger sætninger ordentligt, hvis der tilføjes en ekstra streg, og der opstår en uventet fejl. Stilmæssigt plejer jeg normalt ikke at indrykke ved hjælp af udsagn, når jeg har flere udsagn sammen, men det afhænger helt af personlige præferencer.