Discussion :

[soufix]

Bonjour a tous, je bloque un peu dans un programme qui créé un addin excel (via excel-DNA, mais normalement ca ca va), pour aller requeter sur une DB. je souhaite utiliser le pattern du singleton pour faire mes connecxons, mais je ne vois pas comment structurer mon code (ou placer la connection). Si vous pouviez m'aider ca serait vraiment cool.
Voila ma fonction d'appel :

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
namespace ClassLibrary
{
    public class MyFunctions
        {
 
        [ExcelFunction(Description = "fonction de recherche")]
        public static double id(string _isin)
            //module excel
        {
 
            double res;
 
            //res= "id= " + B.getID(_isin);
            res = DBSQL.Instance.getID(_isin);
 
            return res;
        }
    }
 
}

et voila la class ou je bug :

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
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
 
using System;
using System.Collections.Generic;
using System.Linq;
using System.Text;
using System.Data.SqlClient;
 
 
 
 
 
namespace AccesDB
{
    public class DBSQL : iAccesDB1
    {
        private SqlConnection MaConn = new SqlConnection("ezklhfgpiqhgbfozhg");
 
        private static volatile DBSQL instance;
        private static object syncRoot = new Object();
 
        private DBSQL() {}
 
        public static DBSQL Instance
        {
            get
            {
                if (instance == null)
                {
                    lock (syncRoot)
                    {
                        if (instance == null)
 
                            instance = new DBSQL();
                    }
                }
                return instance;
             }
 
        }
 
 
        public void Connection()
        {
            MaConn.Open();
        }
 
        public void CloseConnection()
        {
            MaConn.Close();
        }
 
 
        public double getID(String _isin)
        {
            SqlDataReader rd;
            double res = -9999;
 
 
            SqlCommand cd = new SqlCommand("select cpn from tbassetOLD where isin='" + _isin + "'", MaConn);
 
            try
            {
                rd = cd.ExecuteReader();
 
                if (rd.HasRows)
                {
                    while (rd.Read())
                        res = double.Parse(rd["cpn"].ToString());
                }
 
 
            }
            catch (Exception ex)
            {
                throw new Exception("1000: " + ex.Message);
            }
 
            return res;
 
        }
 
 
    }
}

Merci a tous !

[katkiller]

Salut,

Je ne vois pas précisément ton problème donc petit exemple.

Ma classe qui implémente le "pattern singleton" :

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
    public class TestSingleton
    {
        public int Compteur { get; private set; }
 
        private static TestSingleton current;
 
        public static TestSingleton Current
        {
            get
            {
                return current ?? (current = new TestSingleton());
            }
        }
 
        private TestSingleton()
        {
            Compteur = 0;
        }
 
        public void AddCompteur()
        {
            Compteur++;
        }
    }

Mon programme qui l'utilise :

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
            var testSingleton1 = TestSingleton.Current;
            Console.WriteLine("testSingleton1.AddCompteur()");
            testSingleton1.AddCompteur();
            var testSingleton2 = TestSingleton.Current;
            Console.WriteLine("testSingleton1.Compteur : {0}", testSingleton1.Compteur);
            Console.WriteLine("testSingleton2.Compteur : {0}", testSingleton2.Compteur);
            Console.WriteLine("testSingleton2.AddCompteur()");
            testSingleton2.AddCompteur();
            Console.WriteLine("testSingleton1.Compteur : {0}", testSingleton1.Compteur);
            Console.WriteLine("testSingleton2.Compteur : {0}", testSingleton2.Compteur);
            Console.ReadLine();
 
            //Résultat :
            //testSingleton1.AddCompteur()
            //testSingleton1.Compteur : 1
            //testSingleton2.Compteur : 1
            //testSingleton2.AddCompteur()
            //testSingleton1.Compteur : 2
            //testSingleton2.Compteur : 2

[soufix]

Je comprends à peu près ton exemple (je suis pas très au point), tu créé ton singleton et tu le rappelle dans ton programme.
Sur mon exemple, j'appelle une fonction, qui est dans une classe ou il y a le singleton et la connexion à la DB. En fait je crois que je ne comprends pas ou je dois placer les procédures de connection: a part, après le res=instance ?
Je ne vois pas pourquoi mon truc ne marche pas en fait et c'est horrible :-)

[katkiller]

Je vois quelques petites "faille"...

Par exemple dans ta méthode "getID" je ne te vois jamais ouvrir ta connexion.
N'hésite pas à dire ce qu'il ne te va pas...


Une connexion doit être ouverte la plus tard possible et fermée le plus tôt possible... L'idéal est de l'ouvrir juste avant d'effectuer ta requête ! et la refermé juste après...
Par exemple je pense que c'est à ta classe d'ouvrir et fermer les connexions...

[soufix]

merci pour ta réponse. alors en fait c'est un peu ca mon problème : je ne vois pas quand établir ma connection :je la met en dehors du pattern ? ou je la met juste avant ma fonction excel... En fait je ne comprends pas vraiment ce système de singleton.
Concernant le moment de la connexion, vu qu'il peut y avoir 100 fonctions excel (la j'en ai mis qu'une seule), je me dis que ouvrir ma connection, puis balancer mes requètes (quitte à mettre un pool plus tard s'il y a trop d'utilisateurs), puis fermer la connection ca peut aller.
Après je vais tenter de gérer le cache pour éviter des requètes inutiles.
Du coup tu mettrais ou la connection toi ?
Merrci de ton aide en tous cas, ca m'aide à comprendre.

[katkiller]

Si on applique le principe Objet : SRP (Single responsibility principle) qui est : une classe doit avoir une et une seule responsabilité
On peut dire que ta classe "DBSQL" a pour responsabilité de lire les données en base. C'est donc à elle d'ouvrir et fermer les connexions.
J'espère que tu comprends l'idée... Car résumé des principes en quelques lignes n'est pas évident...

Voici un petit tuto vidéo qui utilise ADO.NET/Singleton (La partie 1 & 2 pourrait t'intersser) : Accéder aux données ADO.NET, Entity Framework et WCF

Voici ta classe avec quelques modifications :
- using sur SqlCommand (libérer la mémoire [Appelle de la méthode "Dispose" de SqlCommand ).
- Ouverture et fermeture de connexion en "privé".
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
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
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
    public class DBSQL : iAccesDB1
    {
        private SqlConnection _maConn;
        private static volatile DBSQL instance;
 
        private DBSQL()
        {
            _maConn = new SqlConnection("ezklhfgpiqhgbfozhg");
        }
 
        public static DBSQL Instance
        {
            get
            {
                return instance ?? (instance = new DBSQL());
            }
        }
 
        private void Connection()
        {
            _maConn.Open();
        }
 
        private void CloseConnection()
        {
            _maConn.Close();
        }
 
 
        public double getID(String _isin)
        {
            SqlDataReader rd;
            double res = -9999;
 
 
            using (SqlCommand cd = new SqlCommand("select cpn from tbassetOLD where isin='" + _isin + "'", _maConn))
            {
                try
                {
                    Connection();
                    rd = cd.ExecuteReader();
 
                    if (rd.HasRows)
                    {
                        while (rd.Read())
                            res = double.Parse(rd["cpn"].ToString());
                    }
                    CloseConnection();
 
                }
                catch (Exception ex)
                {
                    throw new Exception("1000: " + ex.Message);
                }
                finally
                {
                    CloseConnection();
                }
            }
 
            return res;
 
        }
    }