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

[g_tarik0010]

Bonsoir,

Après le lien que t'as filé pvialatte, je sais ce que tu fais encore avec tout ces SqlObjects au lieu de profiter des conseils données dans ce bon tuto

[SaumonAgile]

Tu as changé la manière dont c'est écrit mais au final, le fonctionnement est le même, tu n'as toujours qu'une seule connexion pour tout le monde. Tes modifications n'ont servi à rien.

Et banit les try/catch avec un catch vide, c'est la pire gestion d'erreur possible. Dans ce cas, tu ferais même mieux de ne pas mettre de try/catch plutôt qu'un qui masque toutes les erreurs.

[topolino]

Merci de votre patience, j'essaye encore :


Ma factory

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
 
 public class BddConnection
    {
        public DbConnection GetConnection()
        {
            DbConnection conn = null;
            DbProviderFactory myFactory;
                try
                {
                    ConnectionStringSettings s = System.Web.Configuration.WebConfigurationManager.ConnectionStrings["BddConnectionString"];
                    myFactory = DbProviderFactories.GetFactory(s.ProviderName);
                    conn = myFactory.CreateConnection();
                    conn.ConnectionString = s.ConnectionString;
 
                }
                catch (Exception) {
                    throw;
                }
 
            return conn;
        }
    }

Utilisation

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
 
 public abstract class ClientDAO
    {
        private static SqlConnection _myConnect;
        public static SqlConnection GetConnection()
        {
            if (_myConnect == null)
            {
                BddConnection mySocle = new BddConnection();
                _myConnect = (SqlConnection)mySocle.GetConnection();
            }
 
            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;
                GetConnection().Open();
 
                int rowCount = Convert.ToInt32(oSqlCommand.ExecuteScalar());
                GetConnection().Close();
                if (rowCount > 0)
                    exist = true;
            }
            catch (Exception)
            {
 
                throw;
            }
 
            return exist;
        }
    }

Ne pas s'attarder sur le try catch, ce n'est pas encore implementé, ce qui m'interesse c la connection a la BDD.

Bon c'est mieux la ?

[lutecefalco]

J'ai jamais compris pourquoi tout le monde se prenait la tête avec ses histoires de connexion et tout.
Perso, je crée, ouvre puis ferme ma connexion à chaque requête et je laisse le framework, le driver et la base gérer comme ils le sentent le pool de connexion

[topolino]

Pas d'autres commentaires ? j'attends vos critiques

[xpander]

J'ai jamais compris pourquoi tout le monde se prenait la tête avec ses histoires de connexion et tout.
Perso, je crée, ouvre puis ferme ma connexion à chaque requête et je laisse le framework, le driver et la base gérer comme ils le sentent le pool de connexion

Idem, cela ne sert a rien de faire un singleton de ce type.

De plus quand tu utilises un membre static dans une application asp.net, il est accessible/partagé entre tous les threads, ce qui veut dire que tu es obligé de faire des lock pour être "thread safe" ...
je te conseille plutot de lire de type d'article
http://aspalliance.com/1099_Understa...ing_in_NET.all

[topolino]

Donc si j'ai bien tout compris, j'eneleve mon singleton les methodes statics.

Factory

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
    {
        public DbConnection GetConnection()
        {
            DbConnection conn = null;
            DbProviderFactory myFactory;
                try
                {
                    ConnectionStringSettings s = System.Web.Configuration.WebConfigurationManager.ConnectionStrings["BddConnectionString"];
                    myFactory = DbProviderFactories.GetFactory(s.ProviderName);
                    conn = myFactory.CreateConnection();
                    conn.ConnectionString = s.ConnectionString;
 
                }
                catch (Exception) {
                    throw;
                }
 
            return conn;
        }
 
    }

Utilisation

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

Pouvez-vous me dire votre avis ?

[xpander]

actuellement le try ne sert a rien, tu poses un try si 1/ tu es capables de faire un traitement différent en cas d'exception 2/ tu as des resources a fermer ...
dans ton cas si il y a une exception sur la ligne ExecuteScalar la connection n'est pas fermée ...

il faut plutot placer un finally

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
 public bool PseudoExist(string pseudo)
        {
		SqlConnection _myConnect = null;           
 
 
            try
            {
                _myConnect = GetConnection();
                SqlCommand oSqlCommand = new SqlCommand("ps_PseudoExist", _myConnect);
                oSqlCommand.CommandType = CommandType.StoredProcedure;
                oSqlCommand.Parameters.Add("u_pseudo", SqlDbType.VarChar, 30).Value = pseudo;
                _myConnect.Open();
 
                int rowCount = Convert.ToInt32(oSqlCommand.ExecuteScalar());
 
                if (rowCount > 0) {
			return true;
		}
 
            }
            finally 
            {
		if ( _myConnect  != null ) {
               		_myConnect.Close();
		}
            }
 
            return false;
        }
    }

[topolino]

Merci Xpander,

Comme je le disais plus haut, il ne faut pas s'attarder sur mon try catch mais plutot la facon de se connecter a la BDD et a son utilisation.

Est elle satisfaisante pour un site Web ?

[topolino]

Personne ?

Dans cette article il y a des methode static ? est ce une bonne chose ?

http://johannblais.developpez.com/tu...-donnees/#LIII

[xpander]

Oui ta méthode est bonne pour un site web ... du moins si tu utilises celle avec finally

[lutecefalco]

Dans cette article il y a des methode static ? est ce une bonne chose ?

Oui

[topolino]

Est il plus judicieux de mettre mes methodes en static ? et ds ma factory et dans ma DAL Client ?