Discussion :

[Ruddy]

Bonjour à tous.

Pour me familliariser avec l'accès au données, je me suis fixé comme objectif, de créer une classe produit dans une dll et une table produit dans sql server 2005. j'ai fais beaucoup de recherche sur la persistance avec dotnet. j'ai découvert une multitude d'outils mais je prèfere coder ma persistance manuellement. j'ai également découvert LINQ dont j'attends la sortie avec impatience.


Dans ma classe Produit, j'ai un constructeur qui crée l'objet en récupérant les infos dans la base selon l'id. je voulais savoir si c'etait la une façon correcte de procéder. Je suis persuader que ce n'est pas la bonne façon alors si quelqu'un pouvait me mettre sur la voie et m'apporter quelques précision. je serai ravi.

Je vous montre mon 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
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
public class Produit
    {
        int refProduit;
        string designationProduit;
        string descriptionProduit;
        double puProd;
        int qteStock;
        int refCat;
 
        public int RefProduit
        {
            get { return refProduit; }
            set { refProduit = value; }
        }
 
        public string DesignationProduit
        {
            get { return designationProduit; }
            set { designationProduit = value; }
        }
 
        public string DescriptionProduit
        {
            get { return descriptionProduit; }
            set { descriptionProduit = value; }
        }
 
        public double PuProd
        {
            get { return puProd; }
            set { puProd = value; }
        }
 
        public int QteStock
        {
            get { return qteStock; }
            set { qteStock = value; }
        }
 
        public int RefCat
        {
            get { return refCat; }
            set { refCat = value; }
        }
 
        public Produit()
        {
            RefCat = 0;
            DesignationProduit = "";
            DescriptionProduit = "";
            PuProd = 0;
            QteStock = 0;
            RefCat = 0;
 
        }
 
        public Produit(int refProduit, string designationProduit, string descriptionProduit, double puProd, int qteStock, int refCat)
        {
            RefCat = refCat;
            DesignationProduit = designationProduit;
            DescriptionProduit = descriptionProduit;
            PuProd = puProd;
            QteStock = qteStock;
            RefCat = refCat;           
        }
 
        public Produit(int refProduit)
        {
            SqlConnection cnx = new SqlConnection(@"Data Source=SURF2036\SQLEXPRESS;Initial Catalog=eCommerce;Integrated Security=SSPI;");
            cnx.Open();
            SqlCommand cmd = new SqlCommand();
            cmd.Connection = cnx;
            cmd.CommandText = "select ref_Produit, desig_prod from tbl_produit where ref_produit=@ref";
            cmd.Parameters.AddWithValue("@ref", refProduit);
            DbDataReader rdr = cmd.ExecuteReader();
            while (rdr.Read())
            {
                RefProduit = Int32.Parse(rdr.GetValue(0).ToString());
                DesignationProduit = rdr.GetValue(1).ToString();
 
            }
            cnx.Close();
 
        }

Merci a tous

[Mose]

Il n'y a pas "UNE" bonne façon de procéder, mais plusieurs, suivant les cas.

La seule chose réellement mauvaise dans ton exemple, c'est la boucle while qui n'a pas de sens. Vaudrait mieux tester qu'il n'y a qu'un seul enregistrement de remonté.

(sous entendu : le reste est donc OK)

[Ruddy]

Oui effectivement je n'ai pas réfléchi pour la boucle while.

Merci de ta contribution.

@ bientôt

[Mose]

Sinon, si tu préfères vraiment coder à la main, je te suggères de te renseigner sur le N-tiers. Ca consiste à séparer la couche d'accès aux données de la couche métier. Beaucoup plus pro.