Discussion :

[koukou0687]

Bonsoir,
j'essaie de supprimer un enregistrement dans une base de donnée à partir d'un click sur bouton dont voilà le code

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
 
 
protected void OnItemCommand(Object sender, RepeaterCommandEventArgs e)
        {
 
            SqlCommand myCommand;
            SqlConnection cnx = (SqlConnection)Session["cnx"];
            string SQL = "DELETE FROM Commande WHERE ID_Commande = "+e.CommandArgument;
            myCommand = new SqlCommand(SQL, cnx);
            myCommand.CommandType = CommandType.Text;
            myCommand.Connection = cnx;
            cnx.Open();
            myCommand.ExecuteNonQuery();
            cnx.Close();
 
        }

mais il me sort un erreur en me disant :

La connexion n'est pas fermée. L'état actuel de la connexion est ouvert.

alors que c'est pas le cas
Merci de m'aider à trouver le bug

[Nicolas Esprit]

Bonsoir,

Le fait que tu stockes ta connexion en Session a de très grande chance d'être la source du problème. Ce n'est pas une bonne pratique. Le mieux serait que ta couche d'accès aux données ouvre puis ferme une connexion le plus rapidement possible. Tu peux aussi regarder du côté du pooling.

Autres remarques qui te seront utiles :

• Pense à utiliser l'instruction using afin de mieux gérer la mémoire (par exemple sur tes objets SqlConnection, SqlCommand)
  
• L'accès aux donnés ne devrait pas se faire dans ta page ASP.NET. C'est une très mauvaise pratique.
• La construction de ta requête SQL est dangereuse d'un point de vue sécurité. Qui sait ce que peut contenir le CommandArgument ? Utilise des SqlParameters.

En espérant t'avoir aidé.