Discussion :

[dharkan]

Bonjours,

concernant les transaction gérer du coté de C# j ai trouver cette méthode, mais est-elle correct ?

celle-ci

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
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
 
using (SqlConnection cn = new SqlConnection(ChaineConnexion))
            {
                using (SqlCommand com = new SqlCommand("SP_AjoutCarte", cn))
                {
                    SqlTransaction transaction;
 
                    com.CommandType = CommandType.StoredProcedure;
 
                    com.Parameters.Add("@nomCarte", SqlDbType.VarChar).Value = nomCarte;
                    com.Parameters.Add("@idType", SqlDbType.Int).Value = idType;
                    com.Parameters.Add("@nbCarte", SqlDbType.Int).Value = nbCarte;
                    com.Parameters.Add("@rareteCarte", SqlDbType.VarChar).Value = rarete;
                    com.Parameters.Add("@idSousType", SqlDbType.Int).Value = idSousType;
                    com.Parameters.Add("@idEdition", SqlDbType.Int).Value = idEdition;
                    com.Parameters.Add("@idCouleur", SqlDbType.Int).Value = idCouleur;
 
                    cn.Open();
                    transaction = cn.BeginTransaction();
                    com.Transaction = transaction;
                    try
                    {
                        com.ExecuteNonQuery();
                        com.Transaction.Commit();
                    }
                    catch(Exception e)
                    {
                        com.Transaction.Rollback();
                    }
                }
            }

[infosam76]

Bonjour,

ca me parait correct, je rajouterais peut-être un finaly à votre classe pour fermer la connexion que vous ouvrer dans le bloc try et libéré les différences ressources (Dispose,...)

[youtpout978]

Le faite qu'il utilise un using fait qu'il n'a pas besoin de libérer sa connexion, ça sera fait automatiquement.

[infosam76]

Juste j'avais loupé cette ligne autant pour moi