Discussion :

[LhIaScZkTer]

Bonjour à tous,

Je m'occupe de la maintenance d'une application et je suis confronté à plein de problèmes... évidemment

Je viens de trouver :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
 
            Dim myConnection As SqlConnection
            myConnection = New SqlConnection()
            myConnection.ConnectionString = System.Configuration.ConfigurationManager.AppSettings("ConnectionString")
            myConnection.Open()
            Session("myConnection") = myConnection

Cette Session("myConnection") est utilisée par des centaines de méthodes dans diverses class. J'ai tout le temps des anomalies et exception sur des datareader déjà ouvert...

Pour ma part, je trouve totalement aberrant de stocker un objet SqlConnection en session encore plus s'il est open...

Ai-je raison ou tort de penser que c'est une grave faute de conception ? Je vais devoir défendre ma thèse devant mon patron, mais les seuls arguments que j'ai sont : "Parce que c'est totalement débile..."

Vous en pensez quoi ?

Merci pour votre aide.

[Invité]

Salut,

Ai-je raison ou tort de penser que c'est une grave faute de conception ? Je vais devoir défendre ma thèse devant mon patron, mais les seuls arguments que j'ai sont : "Parce que c'est totalement débile..."

Tu as TOTALEMENT raison. ça ne fait pas parti des bonnes pratiques préconisées.

Un argument : une connexion consomme trop de ressource. Du coup ne la laissée jamais ouverte toute la durée de l'application et en plus la fermée aussitôt qu'on a fini de travailler avec pour éviter des fuites de mémoires.

[Proteus91]

Je suis d'accord. J'ai eu le meme probleme dans une de mes sociétés. Une veritable abération... Plus tard on m'a dit que le mec a été viré j'ai compris pourquoi.