- Close() est appelé par la méthode Dispose(), donc tant que vous utilisez "using" correctement, vous n'avez pas besoin d'appeler Close() explicitement.
- S'il est possible qu'un NULL soit renvoyé, vous devez le vérifier. Je conseillerais que si vous contrôlez le code qui obtient la connexion sql, vous envisagez fortement de lancer une exception plutôt que de renvoyer NULL. Sinon, d'autres développeurs peuvent rencontrer les mêmes types de problèmes. Le fait de ne pas ouvrir une connexion SQL qui est requise me semble être un cas exceptionnel valable. Si nécessaire, vous pouvez toujours inclure un TryConnectOneTime lorsqu'un développeur souhaite éviter la gestion des exceptions.
En outre, une autre note de style - je vous conseillerais de bien préparer vos instructions d'utilisation au cas où une ligne supplémentaire parasite serait ajoutée et qu'une erreur inattendue se produirait. En ce qui concerne le style, je n'ai généralement pas tendance à utiliser des déclarations en retrait lorsque j'ai plusieurs déclarations ensemble, mais tout dépend de mes préférences personnelles.
