- Close() jest wywoływana przez metodę Dispose(), więc tak długo, jak poprawnie używasz "używania", nie musisz jawnie wywoływać Close().
- Jeśli możliwe jest zwrócenie wartości NULL, należy to sprawdzić. Radziłbym, że jeśli kontrolujesz kod, który uzyskuje połączenie sql, zdecydowanie rozważ rzucenie wyjątku zamiast zwracania wartości NULL. W przeciwnym razie inni programiści mogą napotkać ten sam rodzaj problemów. Wydaje mi się, że nieudane otwarcie połączenia SQL, które jest wymagane, jest prawidłowym wyjątkowym przypadkiem. Jeśli to konieczne, zawsze możesz dołączyć TryConnectOneTime, gdy programista chce uniknąć obsługi wyjątków.
Również inna uwaga dotycząca stylu - radziłbym odpowiednio usztywnić instrukcje using na wypadek dodania zabłąkanej dodatkowej linii i wystąpienia nieoczekiwanego błędu. Jeśli chodzi o styl, zwykle nie używam wcięć, gdy mam kilka zdań razem, ale to wszystko zależy od osobistych preferencji.
