- Close() се извиква от метода Dispose(), така че докато използвате правилно „using“, не е необходимо да извиквате Close() изрично.
- Ако е възможно NULL да бъде върнат, трябва да го проверите. Бих ви посъветвал, ако контролирате кода, който получава sql връзката, силно обмислете хвърлянето на изключение, вместо да връщате NULL. В противен случай други разработчици могат да се натъкнат на същите проблеми. Неуспешното отваряне на необходимата SQL връзка ми се струва валиден изключителен случай. Ако е необходимо, винаги можете да включите TryConnectOneTime, когато разработчикът иска да избегне обработката на изключения.
Също така, още една бележка за стила - бих ви посъветвал правилно да оформите вашите изрази за използване в случай, че бъде добавен страничен допълнителен ред и възникне неочаквана грешка. Що се отнася до стила, обикновено не съм склонен да използвам изрази с отстъп, когато имам няколко израза заедно, но всичко зависи от личните предпочитания.
