Bien formater une classe avec fonction sql

**daviddu54** · 05/06/2010, 16h13

Bonjour tous le monde,

Je crée un site web sans utiliser les providers de microsoft car je dois importer une grosse base de données avec une structure vraiment trop differente de celle d'asp.net.

J'en suis donc a la construction de mes classes et je voudrais donc savoir comment bien structurer ma classe et ne pas ecrire comme un bourrin.

Voici mon code, est'il bien structurer ou est que vous auriez fait autrement?

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
 
 
Imports Microsoft.VisualBasic
Imports System.Data.SqlClient
 
Public Class Iseekyou_Users
    Private _sqlConnectionString As String
 
 
    Public Sub New()
        _sqlConnectionString = ConfigurationManager.ConnectionStrings("bdd_users").ConnectionString
    End Sub
 
 
    Public Function UserNameExist(ByVal UserName As String) As Integer
        Try
            Dim conn As SqlConnection = Nothing
            Dim cmd As SqlCommand = Nothing
            Dim myreader As SqlDataReader = Nothing
            Dim cpt As Integer = 0
            Try
                conn = New SqlConnection(_sqlConnectionString)
                conn.Open()
                cmd.Connection = conn
                cmd.CommandText = "SELECT COUNT(*) FROM Users WHERE UserName='" & UserName & "'"
                myreader = cmd.ExecuteReader
 
                While myreader.Read
                    cpt = myreader.GetValue(0)
                End While
 
                Return cpt
            Finally
                If Not IsNothing(cmd) Then cmd.Dispose()
                If Not IsNothing(conn) Then conn.Close() : conn = Nothing
            End Try
 
        Catch ex As Exception
            Throw ex
        End Try
    End Function
 
End Class

Par ailleur je n'arrive pas a acceder a la fonction "UserNameExist" car ca me genere une erreur , est ce que je dois mettre cette fonction en shared pour pouvoir y aceder depuis mon formulaire d'inscription?
Est ce que je dois creer un namespace?

Merci de partager votre connaissance et expérience avec moi.

**Immobilis** · 05/06/2010, 19h12

Salut,

Envoyé par daviddu54

Je crée un site web sans utiliser les providers de microsoft car je dois importer une grosse base de données avec une structure vraiment trop differente de celle d'asp.net.

Provider Microsoft? Grosse base de données différente de ASP.Net?? Ca veut rien dire ça... ASP.Net n'est pas une base de données.

Sinon, non, c'est pas super bien codé. Ta requête SQL est ouverte aux injections SQL (utiliser des procédures stockées ou du LINQ), inutile de faire un try...catch si c'est juste pour jeter l'erreur. Autant la laisser remonter toute seule. Les objets disposable comme les connections SQL doivent être dans des USING. Exemples ici: http://dotnet.developpez.com/faq/asp...onet_procstock

A+

**Philippe Vialatte** · 05/06/2010, 21h31

je dirais même plus, il faudrait lire cet article :

http://johannblais.developpez.com/tu...acces-donnees/

**tomlev** · 06/06/2010, 02h47

Envoyé par Immobilis

Salut,Provider Microsoft? Grosse base de données différente de ASP.Net?? Ca veut rien dire ça... ASP.Net n'est pas une base de données.

Je pense que daviddu54 fait référence aux Membership Providers qui permettent de gérer les utilisateurs du site, pas aux providers ADO.NET

**Immobilis** · 06/06/2010, 10h29

"shared" te permet d'accéder à ta classe sans l'instancier. Ca n'a pas de rapport avec sa visibilité ou portée. Si ça te convient c'est bien. Il n'y a pas d'obligation. Si c'est pour le web par contre il faut faire attention au variables "shared" qui peuvent être partagées par tous les utilisateurs de l'application. Perso, je ne le mettrai pas.

C'est d'autant plus intéressant de ne pas le faire que ta classe accède à des ressources externe via une connexion SQL. Si tu fais plusieurs appels à "Iseekyou_Users.UserNameExist" tu va créer à chaque fois une instance de connexion. D'un point de vu performances c'est loin d'être acceptable. Tu peux atteindre la limite de connections par défaut du pool. En général cela indique un défaut de conception. Pour éviter cela:

Laisse ta classe non "shared"
Passe tes objets de connexion et de commande en private accessible dans toute la classe
Implémente l'interface IDisposable sur ta classe.

De plus, si la classe est "shared" toutes les méthodes de la classe doivent l'être aussi.

Un namespace permet de bien ranger tes classes.

**daviddu54** · 06/06/2010, 15h09

Salut tous le monde,

tomlev-> Oui exactement ça les providers membershisp,user,profil et role, ne me convienne pas, pour plusieurs raisons:
* La structure de la base de données que je devrais importées est trop différentes.
*La base de données "aspnetdb.mdf" crée par les providers .net fait 10 mo vide, or la base de données a importer fait 45 mo et mon futur hebergeur n'autorise que 4 base e 50 mo.
*Le mot de passe utilisateur de la base de données a importée est au format md5,je n'ai pas reussi a reproduire le meme type de mot de passe avec le fichier "web.config" .

Donc au final pour pouvoir importer ma base de données j'aurai du surcharger les fonction des 4 classe de gestion des utilisateurs, et si c'est pour faire de la surcharge a tour de bras j'ai autant ecrire mes propre classe.

Philippe-> Merci ,excellent article ,mais avant d'implementer cette classe generique je trouver réponse à certaines questions.

Immobilis->

Laisse ta classe non "shared"
Passe tes objets de connexion et de commande en private accessible dans toute la classe
Implémente l'interface IDisposable sur ta classe.

Tu a mis dans le mille,le but de mon post est justement d'obtenir réponse a ces 3 suggestions.

Voici a quoi ressemble mon code aprés avoir suivi vos conseils

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
Imports System.Data.SqlClient
Imports System.Data
 
Public Class Iseekyou_Users
 
    Public Shared Function UserNameExist(ByVal UserName As String) As Integer
        ' Chaîne de connexion
        Dim _sqlConnectionString As String = ConfigurationManager.ConnectionStrings("bdd_users").ConnectionString
        ' Objet connection
        Dim connection As New SqlConnection(_sqlConnectionString)
        ' Ouverture
        connection.Open()
        ' Objet Command
        Dim command As New SqlCommand("SELECT COUNT(UserName) AS cpt FROM Users WHERE UserName=@UserName", connection)
        ' Paramètres
        command.Parameters.Add(New SqlParameter("@UserName", SqlDbType.VarChar, 30))
        command.Parameters("@UserName").Value = UserName
 
        ' Object datareader
        Dim reader As SqlDataReader = command.ExecuteReader()
        Dim cpt As Integer = 0
        While reader.Read()
            cpt = reader.GetValue(0)
        End While
        ' Fermeture reader
        reader.Close()
        ' Fermeture base
        connection.Close()
        Return cpt
    End Function
 
End Class

Comment je dois modifier mon code pour implementer les suggestions que tu ma soumises?

**Immobilis** · 06/06/2010, 16h00

Envoyé par Immobilis

Laisse ta classe non "shared"
Passe tes objets de connexion et de commande en private accessible dans toute la classe
Implémente l'interface IDisposable sur ta classe.

Les méthodes ne devraient pas être "shared" pour préserver l'instance. Externalise tes objets d'accès aux données en dehors des méthodes pour pouvoir les réutiliser sans créer de nouvelles instances, implémente l'interface IDisposable. Voici un exemple:

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
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
using System;
using System.Collections.ObjectModel;
using System.Configuration;
using System.Data;
using System.Data.Common;
 
namespace DataAccessLayer
{
    public class SeekUser : IDisposable
    {
        #region Private members
 
        private DbProviderFactory _fcty;
        private IDbCommand _cmd;
        private IDbConnection _cnn;
        private string _connectionString;
        private bool _disposed = false; // Variable to track if Dispose has been called
        private Collection<IDataParameter> _parameters = new Collection<IDataParameter>();
        private string _provider;
        private IDbTransaction _trans;
 
        #endregion
 
        #region Constructor
 
        /// <summary>
        /// 
        /// </summary>
        public SeekUser()
        {
            _fcty = DbProviderFactories.GetFactory(ConfigurationManager.ConnectionStrings[1].ProviderName);
            _cnn = _fcty.CreateConnection();
            _cmd = _fcty.CreateCommand();
 
            _cnn.ConnectionString = _cnn.ConnectionString;
            _cnn.Open();
            _cmd.Connection = _cnn;
            _cmd.CommandTimeout = 60;
        }
 
        #endregion
 
        #region IDisposable
 
        /// <summary>
        /// <c>SeekUser</c> creates members of the following IDisposable types: 
        /// <c>DBFactory</c>. If <c>SeekUser</c> has previously shipped,
        /// adding new members that implement IDisposable to this 
        /// type is considered a breaking change to existing consumers.
        /// </summary>
        public void Dispose()
        {
            // Check if Dispose has already been called 
            if (!_disposed)
            {
                // Call the overridden Dispose method that contains common cleanup code
                // Pass true to indicate that it is called from Dispose
                Dispose(true);
                // Prevent subsequent finalization of this object. This is not needed 
                // because managed and unmanaged resources have been explicitly released
                GC.SuppressFinalize(this);
            }
            _disposed = true;
        }
 
        /// <summary>
        /// Implement a finalizer by using destructor style syntax.
        /// Use of this destructor will be done by GC.
        /// </summary>
        ~SeekUser()
        {
            // Call the overridden Dispose method that contains common cleanup code
            // Pass false to indicate the it is not called from Dispose
            Dispose(false);
        }
 
        /// <summary>
        /// Implement the override Dispose method that will contain common cleanup functionality
        /// </summary>
        /// <param name="disposing">This parameter indicates whether disposition of object is managed by user's coding or GC cleanup.</param>
        private void Dispose(bool disposing)
        {
            if (disposing)
            {
                try
                {
                    if (_trans != null)
                    {
                        _trans.Rollback();
                        _trans.Dispose();
                        _trans = null;
                    }
                }
                catch { }
                finally
                {
                    if (_cmd != null)
                    {
                        _cmd.Dispose();
                        _cmd = null;
                    }
 
                    if (_cnn != null)
                    {
                        _cnn.Dispose();
                        _cnn = null;
                    }
 
                    _fcty = null;
                    _parameters = null;
                    _provider = null;
                    _connectionString = null;
                }
            }
        }
 
        #endregion
    }
}

Assez précis je dois dire...
A utiliser ainsi:

Code :

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

1
2
3
4
using (SeekUser su = new SeekUser())
{
    su.UserNameExist("toto");
}

C'est du C#. A toi de le comprendre et l'adapter.

A+

**daviddu54** · 06/06/2010, 18h02

Cool exactement ce qu'il me fallait,toutefois une question me vient a l'esprit.

Est ce que je dois rajouter mes fonctions dans cette classe ou est ce que je peut importer cette classe dans les class que j'ecris?

pour l'instant j'ai essayer de rajouter ma fonction dans la class que tu ma donner,mais ca ne marche pas, voici le 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
 
 
        Private Function UserNameExist(ByVal UserName As String) As Integer
 
            _cmd.CommandText = "SELECT COUNT(UserName) AS cpt FROM Users WHERE UserName=@UserName"
            _cmd.Parameters.Add(New SqlParameter("@UserName", SqlDbType.VarChar, 30))
            _cmd.Parameters("@UserName").Value = UserName
 
            ' Object datareader
            Dim reader As SqlDataReader = _cmd.ExecuteReader()
            Dim cpt As Integer = 0
            While reader.Read()
                cpt = reader.GetValue(0)
            End While
            ' Fermeture reader
            reader.Close()
            Dispose()
            Finalize()
 
            Return cpt
        End Function

Allez immobilis sois pas radin sur les exemples d'implementation ca fait que 2 semaine que je me suis mis a l'asp

**Philippe Vialatte** · 06/06/2010, 18h58

Commence par enlever les appels à Dispose et Finalise dans ta fonction

La, tu lui demande de s'auto-détruire

Deuxio, qu'est-ce que tu entends par :

j'ai essayer de rajouter ma fonction dans la class que tu ma donner,mais ca ne marche pas

**Immobilis** · 06/06/2010, 19h46

Envoyé par daviddu54

Allez immobilis sois pas radin sur les exemples d'implementation ca fait que 2 semaine que je me suis mis a l'asp

????? Pas radin???? Tu me vexe là...

**daviddu54** · 06/06/2010, 19h52

Commence par enlever les appels à Dispose et Finalise dans ta fonction

La, tu lui demande de s'auto-détruire

C'est pas ce qu'il faut faire après avoir l'utiliser ?

Deuxio, qu'est-ce que tu entends par :

j'ai essayer de rajouter ma fonction dans la class que tu ma donner,mais ca ne marche pas

Et bien tout simplement que je n'ai pas la moindre idée de comment utiliser la classe que tu ma donnée .

La question que je pose actuellement c'est : est ce que je dois importer la classe "seekuser" dans ma classe "iseekyou_users" ,mais dans ce cas la je n'arrive pas acceder a rien puisque tout est en private.

Ou alors je rajoute directement ma fonction dans la classe "seekuser" comme je viens de faire mais je suis obliger de la mettre en shared pour pouvoir y acceder.

ps: je n'ai aucune connaissance des classe en .net,j'en ai juste fait quelque une php c'est tout

**daviddu54** · 07/06/2010, 17h27

Salut à tous,

Bon ça avance, tout fonctionne,aprés avoir réparer un oublie dans la classe "SeekUser"

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
 
        public SeekUser()
        {
            _fcty = DbProviderFactories.GetFactory(ConfigurationManager.ConnectionStrings[1].ProviderName);
            _cnn = _fcty.CreateConnection();
            _cmd = _fcty.CreateCommand();
 
            _cnn.ConnectionString = _cnn.ConnectionString;
            _cnn.Open();
            _cmd.Connection = _cnn;
            _cmd.CommandTimeout = 60;
        }

Ca marche mieux comme ca

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
 
        public SeekUser()
        {
            _fcty = DbProviderFactories.GetFactory(ConfigurationManager.ConnectionStrings[1].ProviderName);
            _cnn = _fcty.CreateConnection();
            _cmd = _fcty.CreateCommand();
           _connectionString = ConfigurationManager.ConnectionStrings(1).ConnectionString;
            _cnn.ConnectionString = connectionString ;
            _cnn.Open();
            _cmd.Connection = _cnn;
            _cmd.CommandTimeout = 60;
        }[/

Et avoir ajouter ma fonction modifier dans cette meme classe

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
 
        Public Function UserNameExist(ByVal UserName As String) As Integer
            _cmd.CommandText = "SELECT COUNT(UserName) AS cpt FROM Users WHERE UserName=@UserName"
            _cmd.Parameters.Add(New SqlParameter("@UserName", SqlDbType.VarChar, 30))
            _cmd.Parameters("@UserName").Value = UserName
            Using reader As SqlDataReader = _cmd.ExecuteReader()
                Dim cpt As Integer = 0
                While reader.Read()
                    cpt = reader.GetValue(0)
                End While
                Return cpt
            End Using
        End Function

puis avoir ajouter ce bout de code dans la page d'appel de la fonction "UserNameExist"

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
 
 
    Private _NewDataAccessDisposable As DataAccessDisposable
 
    Private Property NewDataAccessDisposable As DataAccessDisposable
        Get
            Return _NewDataAccessDisposable
        End Get
        Set(ByVal value As DataAccessDisposable)
            _NewDataAccessDisposable = value
        End Set
    End Property
 
    Protected Sub UserNameValidator_ServerValidate(ByVal source As Object, ByVal args As System.Web.UI.WebControls.ServerValidateEventArgs) Handles UserNameValidator.ServerValidate
        NewDataAccessDisposable = New DataAccessDisposable
        Dim result As Integer = 0
        result = NewDataAccessDisposable.UserNameExist(source)
        If result = 0 Then
            args.IsValid = True
        Else
            args.IsValid = False
        End If
    End Sub

De nouvelle question viennent me taquiner les neurones

Est ce que ma fonction UserNameExist est correct
A quel momment on detruit l'objet creer?
Est ce que je vais devoir implementer une interface idisposable dans chacune de mes classes ou est ce que l'on peut faire autrement?

Merci pour votre aide les gars
ps: immobillis ne te vexe c'est une touche d'humour.

**lutecefalco** · 07/06/2010, 17h36

Sachant que tu ne lis qu'un résultat, je mettrais

Code :

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

1
2
3
4
if reader.Read() then
                    cpt = reader.GetValue(0)
                End if

**daviddu54** · 07/06/2010, 17h49

Merci,

toujours bon a prendre,sinon pas d'autre suggestions?
notemment pour la page d'appel de la fonction ,n'y a t'il pas de ce passe de cette propriété ou de la mettre ailleur?

**daviddu54** · 07/06/2010, 18h49

Bon j'ai fait encore quelque modif j'ai remplacer ce bout de 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
 
 
    Private _NewDataAccessDisposable As DataAccessDisposable
 
    Private Property NewDataAccessDisposable As DataAccessDisposable
        Get
            Return _NewDataAccessDisposable
        End Get
        Set(ByVal value As DataAccessDisposable)
            _NewDataAccessDisposable = value
        End Set
    End Property
 
NewDataAccessDisposable = New DataAccessDisposable

Par ca

Code :

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

1
2
 
Private DataAccessDisposable As DataAccessDisposable = New DataAccessDisposable

Ca fait tout de suite plus propre.

Bon il me reste encore une question sans réponse avant de pouvoir cloturer ce topic, est ce que je dois detruire l'objet que je viens de creer et si oui ,ou est ce qu'il faut le faire

**Immobilis** · 07/06/2010, 19h05

est ce que je dois detruire l'objet que je viens de creer et si oui ,ou est ce qu'il faut le faire

Le using le fait proprement pour toi. Par contre, un petit détail. Quand tu en auras assez de répéter ce bout de code

Code :

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

1
2
3
4
5
6
7
8
 
_fcty = DbProviderFactories.GetFactory(ConfigurationManager.ConnectionStrings[1].ProviderName);
_cnn = _fcty.CreateConnection();
_cmd = _fcty.CreateCommand();
_cnn.ConnectionString = _cnn.ConnectionString;
_cnn.Open();
_cmd.Connection = _cnn;
_cmd.CommandTimeout = 60;

dans tout tes gestionnaires de données, tu pourras créer un objet pour les encapsuler.

A+

**daviddu54** · 07/06/2010, 19h37

Le using le fait proprement pour toi

Le using du datareader?

dans tout tes gestionnaires de données, tu pourras créer un objet pour les encapsuler

Chouette , comment on fait?

**daviddu54** · 09/06/2010, 17h53

Salut tous le monde,

Bon voila ce que j'ai fais en réponse a ce que immobilis m'a dit au sujet de l'encapsulation par objet.
j'ai rajouter ce bout de code a mon gestionnaire de conexion

Code :

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

1
2
3
4
5
6
 
        Public ReadOnly Property GetCmd() As IDbCommand
            Get
                Return _cmd
            End Get
        End Property

et j'ai modifier la fonction UserNameExist comme ceci dans ma classe de gestion des utilisateurs

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
 
        Public Shared Function UserNameExist(ByVal UserName As String) As Integer
            Dim NewDataAccessDisposable As New DataAccessDisposable
            Dim _cmd As IDbCommand = NewDataAccessDisposable.GetCmd
            _cmd.CommandText = "SELECT COUNT(UserName) AS cpt FROM Users WHERE UserName=@UserName"
            _cmd.Parameters.Add(New SqlParameter("@UserName", SqlDbType.VarChar, 30))
            _cmd.Parameters("@UserName").Value = UserName
            Using reader As SqlDataReader = _cmd.ExecuteReader()
                Dim cpt As Integer = 0
                If reader.Read() Then
                    cpt = reader.GetValue(0)
                End If
                Return cpt
            End Using
        End Function

Par contre je suis obliger de laisser ma fonction en shared sinon je dois creer une nouvelle instance de ma classe de gestion des utilisateurs.

ESt ce que c'est correct maintenant? ou je peut encore optimiser mon code?

Bien formater une classe avec fonction sql

ASP.NET

Discussions similaires

Partager

Partager