Discussion :

[topolino]

Bonjour,

Je cherche la meilleur solution pour ma connexion à ma Bdd.
Que pensez vous de mon code ? est bien ?

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
32
33
34
35
36
37
38
39
40
41
42
43
 
public class SqlDbConnection
    {
        private SqlConnection oSqlConnection = null;
        private static object _instanceLocker = new object();
 
        private static SqlDbConnection _instanceSqlDbConnection;
        public static SqlDbConnection InstanceSqlDbConnection
        {
            get
            {
                lock (_instanceLocker)
                {
                    if (_instanceSqlDbConnection == null)
                    {
                        _instanceSqlDbConnection = new SqlDbConnection();
                    }
                    return _instanceSqlDbConnection;
                }
            }
        }
 
        //Creation connexion et la renvoi
        public SqlConnection GetSqlConnection()
        {
            oSqlConnection = new SqlConnection("Data Source=localhost;Initial Catalog=maBdd;Integrated Security=True");
            //oSqlConnection = new SqlConnection(ConfigurationManager.ConnectionStrings["DBConnectionString"].ToString());
 
            if (oSqlConnection.State == System.Data.ConnectionState.Closed)
                oSqlConnection.Open();
 
            return oSqlConnection;
        }
 
        public void CloseSqlConnection()
        {
            if (oSqlConnection != null)
            {
                if (oSqlConnection.State == System.Data.ConnectionState.Open)
                    oSqlConnection.Close();
            }
        }
    }

Appel de la classe :

Code :

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

1
2
 
SqlConnection cnx = SqlDbConnection.InstanceSqlDbConnection.GetSqlConnection();

En faite je fais le meme systeme pour toutes classes de ma DAL, BLL...

Des reactions ?

Merci

[Philippe Vialatte]

euh...

j'ai un doute affreux, mais tu veux une seule connexion pour toute ton appli et tous tes utilisateurs ?

passque a vue de nez, c'est ce que tu fais, non ?

Et ca, ca ne me semble pas etre une bonne idee a moyen terme...quid de l;utilisateur2 qui doit faire une rerquete pendant que l'utilisateur1 en fait deja une ????


Pour une appli asp.net, essaye plutot de pooler tes connections

[topolino]

autais tu un exemple pour moi que je puisse me servir ?

[topolino]

Je cherche une class, que je puisse integrer a mes projets.

merci

[Nip]

Je rejoins Philippe. Tu peux lire http://msdn.microsoft.com/en-us/libr...aseconnections histoire d'y voir plus clair

Et puis t'as deja des trucs tout pret:
http://msdn.microsoft.com/en-us/library/cc309504.aspx
http://aspnet.4guysfromrolla.com/articles/030905-1.aspx

[Philippe Vialatte]

l'idee de depart est pas top mal (en dehors du singleton)

A la limite, fait plutot qq chose comme ce qui est decrit ici :
http://johannblais.developpez.com/tu...-donnees/#LIII

(je vais finir par le mettre en signature )

Sinon, si tu veux des trucs plus "packages", regarde du cote des entreprise libraries, ou de Subsonic (j'aiiiime subsonic... )

[topolino]

j'ai retravaillé mon code. Pouvez-vous me dire si c'est bon :


Connection à la Bdd

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
 
public class BddConnection
    {
        private SqlConnection oSqlConnection = null;
 
        public SqlConnection GetSqlConnection()
        {
            try
            {
                ConnectionStringSettings s = System.Web.Configuration.WebConfigurationManager.ConnectionStrings["BddConnectionString"];
                oSqlConnection = new SqlConnection(s.ConnectionString.ToString());
 
                if (oSqlConnection.State == System.Data.ConnectionState.Closed)
                    oSqlConnection.Open();
 
            }
            catch (Exception)
            {
 
            }
            return oSqlConnection;
        }
    }

Utilisation : ClientDAO

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
32
33
34
35
36
37
 
 public abstract class ClientDAO
    {
        private static SqlConnection _myConnect;
        public static SqlConnection GetConnection()
        {
            if (_myConnect == null)
            {
                BddConnection mySocle = new BddConnection();
                _myConnect = (SqlConnection)mySocle.GetSqlConnection();
            }
 
            return _myConnect;
        }
 
        public static bool PseudoExist(string pseudo)
        {
            bool exist = false;
            try
            {
                SqlCommand oSqlCommand = new SqlCommand("ps_PseudoExist", GetConnection());
                oSqlCommand.CommandType = CommandType.StoredProcedure;
                oSqlCommand.Parameters.Add("u_pseudo", SqlDbType.VarChar, 30).Value = pseudo;
 
                int rowCount = Convert.ToInt32(oSqlCommand.ExecuteScalar());
                if (rowCount > 0)
                    exist = true;
            }
            catch (Exception)
            {
 
                throw;
            }
 
            return exist;
        }
    }

Merci à vous