- Close() viene chiamato dal metodo Dispose(), quindi fintanto che si utilizza correttamente "using" non è necessario chiamare Close() in modo esplicito.
- Se è possibile restituire un NULL, dovresti verificarlo. Consiglierei che se controlli il codice che ottiene la connessione sql, consideri fortemente di lanciare un'eccezione piuttosto che restituire NULL. Altrimenti altri sviluppatori potrebbero incontrare lo stesso tipo di problemi. La mancata apertura di una connessione SQL richiesta mi sembra un caso eccezionale valido. Se necessario, puoi sempre includere un TryConnectOneTime per quando uno sviluppatore vuole evitare la gestione delle eccezioni.
Inoltre, un'altra nota di stile:consiglierei di rinforzare correttamente le tue istruzioni using nel caso in cui venga aggiunta una linea extra vagante e si verifichi un errore imprevisto. Per quanto riguarda lo stile, di solito non tendo a rientrare usando le dichiarazioni quando ho più affermazioni insieme, ma tutto dipende dalle preferenze personali.
