Problème boucle imbriquée SqlDataReader

**xouzi** · 12/01/2011, 20h06

Bonjour,

J'ai besoin d'effectuer plusieurs boucles imbriquées mais celle-ci ne fonctionnent pas. Je crois avoir cernée le problème mais je ne sais pas le résoudre. SqlDataReader on ne peut être utiliser que si le précèdent est fermé. Mais je ne sais pas comment lire les requête autrement.

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
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
 
private void (string requete1, string requete1_bis, string requete_Insertion_, string code_athlete, string mdp, string id_athlete, string bdd_idathlete, string server_idathlete, string utilisateur_idathlete, string pwd_idathlete, string id, string mp, string requete_verif)
        {
            //recupération du ID_athlete de la table athlete :
            // Chaine de connexion
            try
            {
 
                string connectString = "database='" + bdd_idathlete + "'; server='" + server_idathlete + "';User ID='" + utilisateur_idathlete + "'; pwd='" +pwd_idathlete + "'";
                // Objet connection
                SqlConnection connection = new SqlConnection(connectString);
 
                connection.Open();// Ouverture
                //TRI IMPRIMANTE
 
 
                requete1 = "SELECT * FROM imprimante WHERE code_athlete is not null AND mdp_athlete is not null";
        // Création d'un objet SqlDataAdapter
                SqlCommand Command = new SqlCommand(requete1, connection);
                SqlDataReader dex = Command.ExecuteReader();
 
                while (dex.Read())
                {
                    code_athlete = dex["code_athlete"].ToString();
                    mdp = dex["mdp_athlete"].ToString();
 
                    requete_verif = "SELECT * FROM  ";
                    SqlCommand verif_ = new SqlCommand(requete_verif, connection);
                    SqlDataReader verif_dex = verif_.ExecuteReader();
 
                    while (verif_dex.Read())
                    {
                        id = verif_dex["LOGIN_"].ToString();
                        mp = verif_dex["MDP_"].ToString();
                        Console.WriteLine(id + "--" + mp);
                        if ((id != code_athlete) && (mp != mdp))
                        {
 
                            requete1_bis = "SELECT id_athlete FROM athlete WHERE code_athlete ='" + code_athlete + "'";
                            SqlCommand insert_ = new SqlCommand(requete1_bis, connection);
                            //// Objet DataReader
                            Console.WriteLine(requete1_bis);
                            SqlDataReader dex_bis = insert_.ExecuteReader();
                            while (dex_bis.Read())
                            {
                                Console.WriteLine(requete1_bis);
 
                                id_athlete = dex_bis["id_athlete"].ToString();
 
                                requete_Insertion_ = "INSERT INTO  VALUES('" + id_athlete.Replace("'", "''") + "','" + code_athlete.Replace("'", "''") + "','" + mdp.Replace("'", "''") + "') ";
                                Console.WriteLine(requete_Insertion_);
                                SqlCommand insert = new SqlCommand(requete_Insertion_, connection);
                                insert.ExecuteNonQuery();
                                insert.Dispose();
                            }
                            dex_bis.Close();
                        }
                    } verif_dex.Close();
                 }
                dex.Close();
 
                connection.Close();
            }

PS: J'ai testé les requêtes elles fonctionnent

**girouxv** · 12/01/2011, 20h56

Il te manque quelque chose là

Code :

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

requete_verif = "SELECT * FROM  ";

**girouxv** · 12/01/2011, 20h58

Il te manque ton nom de table dans ton insert into

Code :

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

requete_Insertion_ = "INSERT INTO  VALUES('" + id_athlete.Replace("'", "''") + "','" + code_athlete.Replace("'", "''") + "','" + mdp.Replace("'", "''") +

**xouzi** · 12/01/2011, 21h27

Sa ne marche toujours pas. Le problème vient que le sqldatareader doit etre fermé avant d'être réutiliser.

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
private void (string requete1, string requete1_bis, string requete_Insertion_, string code_athlete, string mdp, string id_athlete, string bdd_idathlete, string server_idathlete, string utilisateur_idathlete, string pwd_idathlete, string id, string mp, string requete_verif)
        {
            //recupération du ID_athlete de la table athlete :
            // Chaine de connexion
            try
            {
 
                string connectString = "database='" + bdd_idathlete + "'; server='" + server_idathlete + "';User ID='" + utilisateur_idathlete + "'; pwd='" +pwd_idathlete + "'";
                // Objet connection
                SqlConnection connection = new SqlConnection(connectString);
 
                connection.Open();// Ouverture
                //TRI IMPRIMANTE
 
 
                requete1 = "SELECT * FROM imprimante WHERE code_athlete is not null AND mdp_athlete is not null";
        // Création d'un objet SqlDataAdapter
                SqlCommand Command = new SqlCommand(requete1, connection);
                SqlDataReader dex = Command.ExecuteReader();
 
                while (dex.Read())
                {
                    code_athlete = dex["code_athlete"].ToString();
                    mdp = dex["mdp_athlete"].ToString();
 
                    requete_verif = "SELECT * FROM athlete ";
                    SqlCommand verif_ = new SqlCommand(requete_verif, connection);
                    SqlDataReader verif_dex = verif_.ExecuteReader();
 
                    while (verif_dex.Read())
                    {
                        id = verif_dex["LOGIN_"].ToString();
                        mp = verif_dex["MDP_"].ToString();
                        Console.WriteLine(id + "--" + mp);
                        if ((id != code_athlete) && (mp != mdp))
                        {
 
                            requete1_bis = "SELECT id_athlete FROM athlete WHERE code_athlete ='" + code_athlete + "'";
                            SqlCommand insert_ = new SqlCommand(requete1_bis, connection);
                            //// Objet DataReader
                            Console.WriteLine(requete1_bis);
                            SqlDataReader dex_bis = insert_.ExecuteReader();
                            while (dex_bis.Read())
                            {
                                Console.WriteLine(requete1_bis);
 
                            id_athlete = dex_bis["id_athlete"].ToString();
 
                                requete_Insertion_ = "INSERT INTO athlete VALUES('" + id_athlete.Replace("'", "''") + "','" + code_athlete.Replace("'", "''") + "','" + mdp.Replace("'", "''") + "') ";
                                Console.WriteLine(requete_Insertion_);
                                SqlCommand insert = new SqlCommand(requete_Insertion_, connection);
                                insert.ExecuteNonQuery();
                                insert.Dispose();
                            }
                            dex_bis.Close();
                        }
                    } verif_dex.Close();
                 }
                dex.Close();
 
                connection.Close();
            }

**girouxv** · 12/01/2011, 21h44

Voilà ma suggestion

au lieu de

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
while (dex_bis.Read())
                            {
                                Console.WriteLine(requete1_bis);
 
                            id_athlete = dex_bis["id_athlete"].ToString();
 
                                requete_Insertion_ = "INSERT INTO athlete VALUES('" + id_athlete.Replace("'", "''") + "','" + code_athlete.Replace("'", "''") + "','" + mdp.Replace("'", "''") + "') ";
                                Console.WriteLine(requete_Insertion_);
                                SqlCommand insert = new SqlCommand(requete_Insertion_, connection);
                                insert.ExecuteNonQuery();
                                insert.Dispose();
                            }
                            dex_bis.Close();

Fait tes insert en une seule fois en utilisant un stringbuilder

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
 
System.Text.StringBuilder sb = new StringBuilder();
...
while (dex_bis.Read())
 {
     sb.AppendLine("INSERT INTO athlete VALUES('" + id_athlete.Replace("'", "''") + "','" + code_athlete.Replace("'", "''") + "','" + mdp.Replace("'", "''") + "'); ");
}
dex_bis.Close();
...
...
SqlCommand insert = new SqlCommand(sb.ToString(), connection);
                                insert.ExecuteNonQuery();
connection.Close();

**StormimOn** · 12/01/2011, 21h46

Envoyé par xouzi

Le problème vient que le sqldatareader doit etre fermé avant d'être réutiliser.

Tu as la réponse à ton problème, tu ne peux pas procéder ainsi. Tu dois donc modifier ton code de manière à ne plus avoir d'imbrication et donc plus de problèmes

**xouzi** · 12/01/2011, 21h58

Mais je vois pas comment je peux faire vue que mes requêtes s'exécutent par rapport au résultat des requêtes précédentes.
Sinon il faudrait que je récupérai le résultat des requêtes dans un tableau mais je pense qu'il y a mieu.

**girouxv** · 12/01/2011, 22h04

Petite note au passage tu peux remplacer tes values par des paramètres. Ça t'assure une syntaxe sql correcte et un protection contre des hackers si c'est pour un site web.

Code :

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

1
2
3
4
 
 sb.AppendLine("INSERT INTO athlete VALUES('" + id_athlete.Replace("'", "''") + "','" + code_athlete.Replace("'", "''") + "','" + mdp.Replace("'", "''") + "'); ");
SqlCommand insert = new SqlCommand(sb.ToString(), connection);
                                insert.ExecuteNonQuery();

Code :

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

1
2
3
4
5
6
7
 
sb.AppendLine("INSERT INTO athlete VALUES(@id_athlete,@code_athlete,@mdp); ");
SqlCommand insert = new SqlCommand(sb.ToString(), connection);
                                insert.ExecuteNonQuery();
SqlCommand.Parameters.AddWithValues("@id_athlete",id_athlete);
SqlCommand.Parameters.AddWithValues("@code_athlete",code_athlete);
SqlCommand.Parameters.AddWithValues("@mdp",mdp);

Ton code sera plus lisible.

**xouzi** · 12/01/2011, 22h09

Ok merci. Mais je sais toujours pas comment empecher les boucle imbriquer. il y a rien pour remplacer sqldatareader.

**StormimOn** · 12/01/2011, 22h13

Envoyé par xouzi

Sinon il faudrait que je récupérai le résultat des requêtes dans un tableau mais je pense qu'il y a mieu.

En quoi ce ne serait pas bien ?

Par contre je comprends mal ce que tu veux faire, car faire un SELECT d'un enregistrement d'une table pour récupérer son ID afin d'insérer un autre enregistrement dans la même table et avec le même ID, ça me semble étrange

Tu pourrais expliquer simplement ce que tu cherches à faire ? ^^

**girouxv** · 12/01/2011, 22h21

Tu dois prendre 2 requêtes sql et en faire qu'une seule avec un inner join sur ton code_athlete et une condition where sur imprimante.mdp <> athelte.mdp

Tu auras qu'un datareader.

Problème boucle imbriquée SqlDataReader

C#

Vue hybride

Discussions similaires

Partager

Partager