Discussion :

[eleve_prog]

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 ?

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

Alors voici mon code

Code :

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

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 :

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

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 !

[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.

[eleve_prog]

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

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
 
 
 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();

[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 :

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
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é.

[eleve_prog]

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 :

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

[calagan99]

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

Là, tu me fais plaisir

[eleve_prog]

Là, tu me fais plaisir

Tout marche impec'

Tchu's

[iberserk]

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.