Connection Bdd

Version imprimable

Connection Bdd

Bonjour,

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

Code:

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:

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

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

Des reactions ?

Merci

25/09/2008, 18h06
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 ;)
25/09/2008, 18h19
topolino

autais tu un exemple pour moi que je puisse me servir ?
25/09/2008, 18h47
topolino

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

merci
25/09/2008, 21h51
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
25/09/2008, 21h56
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 :mouarf:)

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

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

Connection à la Bdd

Code:

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:

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

26/09/2008, 06h17
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
26/09/2008, 08h39
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.

Merci de votre patience, j'essaye encore :

Ma factory

Code:

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:

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 ?

26/09/2008, 10h20
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 :mrgreen:
26/09/2008, 12h07
topolino

Pas d'autres commentaires ? j'attends vos critiques
26/09/2008, 13h13
xpander

Citation:

Envoyé par 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 :mrgreen:

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

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

Factory

Code:

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:

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 ?

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:

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;
        }
    }

26/09/2008, 14h45
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 ?
26/09/2008, 17h04
topolino

Personne ?

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

http://johannblais.developpez.com/tu...-donnees/#LIII
26/09/2008, 17h15
xpander

Oui ta méthode est bonne pour un site web ... du moins si tu utilises celle avec finally
26/09/2008, 17h15
lutecefalco

Citation:

Envoyé par topolino

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

Oui
26/09/2008, 17h57
topolino

Est il plus judicieux de mettre mes methodes en static ? et ds ma factory et dans ma DAL Client ?
27/09/2008, 14h23
xpander

des methodes oui, des champs / variables non