Discussion :

[Chacha35]

Bonjour,

Je voudrais savoir s'il est possible d'emboiter des DataReaders.
Voici mon code. Il y a un problème car il me dit

Un DataReader associé à cette commande est déjà ouvert. Il doit d'abord être fermé.

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

public void RemplListCltAvecRef(string RefEnCours)
        {
            SqlDataReader DataReader1 = null;
            SqlDataReader DataReader2 = null;

            Lst_Clt.Items.Clear();

            string sql1 = "SELECT AKKNDI,AKARTK, AKARTN  FROM ARKF_ACTIF WHERE AKARTN = '" + RefEnCours + "'ORDER BY AKARTN";
            SqlCommand cmd1 = new SqlCommand(sql1, ConsultConnection);
            DataReader1 = cmd1.ExecuteReader();

            int NbreDeClientsPourPieceSelect; NbreDeClientsPourPieceSelect = 0;
            int PositionCltEnCours; PositionCltEnCours = 0;

            if (DataReader1.HasRows)
            {
                while (DataReader1.Read())
                {
                    string sql2 = "SELECT KNKNDI, KNKNDK, KNKNMN FROM KNDF WHERE KNKNDI = '" + DataReader1["AKKNDI"].ToString()  + "' ORDER BY KNKNDK";
                    SqlCommand cmd2 = new SqlCommand(sql2, ConsultConnection);
                    DataReader2 = cmd2.ExecuteReader();

Merci d'avance

[laedit]

Là, je pense que le problème vient tout simplement de la requête : tu ne devrais pas être obligé d'utiliser deux DataReader pour obtenir ce que tu veux.
Tu peux faire une sous-requête du genre :

Code :

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

SELECT ... FROM myTable WHERE column IN (SELECT ....) ORDER BY column

Je t'avoue que j'en ai pas fait depuis un bail, je ne te garantis donc pas la viabilité de ce code, mais l'idée est là

Et sinon, n'oublie pas que tu peux te faciliter la vie de cette façon :

Code :

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

1
2
3
 
int NbreDeClientsPourPieceSelect = 0;
int PositionCltEnCours = 0;

[Pol63]

c'est en effet contreperformant de faire ce que tu fais

demande à google ce qu'est une jointure (INNER JOIN)

[Chacha35]

Ok je regarde ça tout de suite.

Merci!

[Chacha35]

J'ai encore le même problème alors que je ferme mes DataReader.

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

        public void RemplListCltAvecRef(string RefEnCours)
        {
            SqlDataReader DataReader1 = null;

            Lst_Clt.Items.Clear();
             
            string sql1 = "SELECT AKKNDI,AKARTK, AKARTN, KNKNDI, KNKNDK, KNKNMN  FROM ARKF_ACTIF INNER JOIN KNDF ON ARKF_ACTIF.AKKNDI = KNDF.KNKNDI WHERE AKARTN = '" + RefEnCours + "'ORDER BY KNKNDK";
            SqlCommand cmd1 = new SqlCommand(sql1, ConsultConnection);
            DataReader1 = cmd1.ExecuteReader();

            int NbreDeClientsPourPieceSelect; NbreDeClientsPourPieceSelect = 0;
            int PositionCltEnCours; PositionCltEnCours = 0;

            if (DataReader1.HasRows)
            {
                while (DataReader1.Read())
                {
                     (...)
                }
            DataReader1.Close();
            cmd1.Dispose();
        }

        public void RemplRefCltAvecCodeClt(string CodeCltEnCours, string RefEnCours)
        {
            SqlDataReader DataReader2 = null;

            string sql2 = "SELECT KNKNDI, KNKNDK, KNKNMN? AKKNDI,AKARTK, AKARTN FROM KNDF INNER JOIN ARKF_ACTIF ON KNDF.KNKNDI=ARKF_ACTIF.AKKNDI WHERE KNKNDK = '" + CodeCltEnCours + "' + AKARTN = '" + RefEnCours + "'ORDER BY KNKNDK";
            SqlCommand cmd2 = new SqlCommand(sql2, ConsultConnection);
            DataReader2 = cmd2.ExecuteReader();

[laedit]

Tu as le même message d'erreur ?

Au passage, il manque une '}' dans la première fonction après le while pour fermer le if.
Et dans la requête de la 2° fonction, une virgule a été transformée en point d'interrogation.

[Chacha35]

Oui c'est la même erreur.

Merci pour d'avoir remarqué les petites boulettes lol

[Pol63]

il faut un objet connection par reader, tu utilises tout le temps le meme
il faut réellement en instancier un à chaque fois
et fermer la connexion le plus tot possible


sinon ton if (DataReader1.HasRows) doit servir à rien aussi ...

[laedit]

Il est certain qu'il vaut mieux fermer la connexion le plus rapidement possible.

Mais ayant déjà dû travailler avec des DataReaders et la connexion ouverte, je n'ai cependant pas eu ce genre d'erreurs...

[Chacha35]

Daccord je tente ça.

Merci

------------

C'est bon ça marche super bien