Alternative date reader dans un autre datareader. DataSource

Version imprimable

Bonjour à tous, alors je suis assez débutant en ADO.NET et j'utilise principalement des datareader...

Mais apparemment, il est impossible d'imbriquer 2 DataReader ? :roll:

Donc j'ai cherché une alternative pour stocker les données. Et je suis tombé sur le dataSet ?

J'ai fait de la recherche mais j'ai tjs une erreur. A mon avis vous allez vite trouver , enfin j'espère :mrgreen:

Alors voici mon code
Code:

1 2 3 4 5 6 SqlCommand cmd = new SqlCommand("SELECT Title, PlanID FROM dbo.Evaluation Where Step = '" + step + "'", conn); SqlDataAdapter da = new SqlDataAdapter(); da.SelectCommand = cmd; DataSet ds = new DataSet("Plans"); da.Fill(ds);
En fait je pense que cette partie là est bonne, Je bloque plus pour accéder aux données ?

Code:

Response.Write(ds.Tables["Plans"].Rows[0]["Title"].ToString());

j'ai à peu près tout testé pour accéder, j'vois pas...

Merci de bien vouloir m'éclairer là dessus !

16/03/2011, 16h07
calagan99

Alors, une petite remarque pour commencer.
Regarde ce tutoriel est met le en application. Il concerne les requêtes paramétrées et te permettra de ne plus faire de requêtes à base de concaténations de string dangereuses et sources d'erreur.
Ca ne coute pas plus cher et t'évitera bien des ennuis :)

Pourquoi passes-tu par un Dataset ici ? Un DataReader conviendrait très bien (au vu du code que tu donnes).

Enfin, tu ne pourras pas accéder à la table "plans" de ton dataSet puisque c'est le dataSet lui-même auquel tu as donné le Name "plan".

Détaille un peu ton code pour que nous puissions t'en dire plus stp.

J'utilise les paramètres quand ce sont de longues requêtes. Ici elles sont pas trop longues , enfin je trouve.

Je veux juste accéder aux valeurs Title et PlanID.

C'est pas possible ?

Je sais qu'avec juste la partie de code que j'ai mise je peux utiliser un datareader mais ici je veux juste savoir une utiliser un dataset.

je pense pas que ça soit nécéssaire de mettre mon bout de code imbriquant mes 2 datareader mais bon je vous le met en option 8-)

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
 
 
 SqlConnection conn = new SqlConnection(ConfigurationManager.AppSettings["connectionString"]);
            conn.Open();
            try
            {
                SqlDataReader dr2;
                SqlCommand cmd = new SqlCommand("SELECT Title,PlanID  FROM EvaluationListPlansFiche  Where Step = '" + step + "'", conn);
 
                SqlDataReader dr = cmd.ExecuteReader();
                 while (dr.Read())
                 {
                     plan = new Label();
                     plan.Text = dr["Title"].ToString();
                     PlaceHolder1.Controls.Add(plan);
                     PlaceHolder1.Controls.Add(new LiteralControl("<br />"));
 
                     SqlCommand cmd2 = new SqlCommand("SELECT Title,DescriptionSec  FROM dbo.EvaluationSection Where PlanID = '" + Int32.Parse(dr["PlanID"].ToString()) + "'", conn);
                     dr2 = cmd2.ExecuteReader();
                     while (dr2.Read())
                     {   
 
                     }
                     dr2.Close();
                     dr2 = null;
                 }
                 dr.Close();

Utiliser des requêtes paramétrées n'est pas seulement utile pour la lisibilité des requêtes. Elles le sont surtout pour sécuriser ton application car avec tes concaténations, tu ne te protèges pas contre les injections SQL.

Pour ce qui est de ton DataSet, tu n'as pas nécessairement besoin d'imbriquer deux dataReaders :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 DataSet ds = new DataSet(); DataTable dt = new DataTable("Plans"); SqlCommand cmd = new SqlCommand("SELECT Title,PlanID FROM EvaluationListPlansFiche Where Step = '" + step + "'", conn); SqlDataAdapter da = new SqlDataAdapter(cmd); da.Fill(dt); ds.Tables.Add(dt); SqlCommand cmd2 = null; foreach( DataRow row in dt.Rows ){ cmd2 = new SqlCommand("SELECT Title,DescriptionSec FROM dbo.EvaluationSection Where PlanID = '" + row["PlanID"].ToString() + "'", conn); dr2 = cmd2.ExecuteReader(); while (dr2.Read()) { } dr2.Close(); dr2 = null; }
A indenter et adapter car j'ai écrit le code ici et ne l'ai pas testé.

Citation:
Envoyé par calagan99
Utiliser des requêtes paramétrées n'est pas seulement utile pour la lisibilité des requêtes. Elles le sont surtout pour sécuriser ton application car avec tes concaténations, tu ne te protèges pas contre les injections SQL.

Pour ce qui est de ton DataSet, tu n'as pas nécessairement besoin d'imbriquer deux dataReaders :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 DataSet ds = new DataSet(); DataTable dt = new DataTable("Plans"); SqlCommand cmd = new SqlCommand("SELECT Title,PlanID FROM EvaluationListPlansFiche Where Step = '" + step + "'", conn); SqlDataAdapter da = new SqlDataAdapter(cmd); da.Fill(dt); ds.Tables.Add(dt); SqlCommand cmd2 = null; foreach( DataRow row in dt.Rows ){ cmd2 = new SqlCommand("SELECT Title,DescriptionSec FROM dbo.EvaluationSection Where PlanID = '" + row["PlanID"].ToString() + "'", conn); dr2 = cmd2.ExecuteReader(); while (dr2.Read()) { } dr2.Close(); dr2 = null; }
A indenter et adapter car j'ai écrit le code ici et ne l'ai pas testé.
Ok merci :) , je vais tester ça !

J'étais pas au courant pour les injections sql *-) je vais changer ça :)

16/03/2011, 16h48
calagan99

Citation:

Envoyé par eleve_prog

J'étais pas au courant pour les injections sql *-) je vais changer ça :)

Là, tu me fais plaisir :)
16/03/2011, 16h52
eleve_prog

Citation:

Envoyé par calagan99

Là, tu me fais plaisir :)

Tout marche impec' :ccool::mouarf:

Tchu's :)
17/03/2011, 08h23
iberserk

Citation:

Utiliser des requêtes paramétrées n'est pas seulement utile pour la lisibilité des requêtes. Elles le sont surtout pour sécuriser ton application car avec tes concaténations, tu ne te protèges pas contre les injections SQL.

Cela permet également de meilleures performances côté SGBD du fait de la ré-utilisabilité des plans d’exécution de la requête.