Imbrication de for

Version imprimable

Imbrication de for

Bonjour,
Mon problème et le suivant :

Je veux remplir une 1ère table sql server en utilisant des données présentes dans une 2e table sql server et une 3e as400.
Pour cela j'utilise 3 DataTables, un par table, le 3e (DT) me sert à récupérer des deux premiers les lignes que je veux insérer dans ma 1ère table.

J'ai créer l'algo qui permet de comparer les deux 1ers DataTables en fonction des champs qui m'intéressent.
Comme je boucle sur ces deux derniers, le temps que prend mon programme à s'exécuter est fonction du nombre de lignes concernées. Afin de réduire cela au maximum je supprime au fur et à mesure les lignes que je ne veux pas copier afin de ne pas reboucler dessus au prochain passage. Ce qui fait que je boucle sur un total de ligne d'un DataTable qui évolue. Et ce qui me pète de jolis IndexOutOfRange Exception.

Avec le code ci-dessous, l'erreur est la suivante : Aucune ligne à la position 482.
J'ai donc supprimé 18lignes avant que l'exception pointe le bout de son nez, j'ai donc j=482 et i=0.
J'ai tenté un paquet de coup de Trafalgar mais rien de fonctionne.

J'espère que mon code ne vous paraitra pas trop obscur (sinon ne vous concentrez que sur les conditions des for) :

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
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
 
DT = dtas.Clone();
            for (int i = 0; i < dtas.Rows.Count; i++)
            {                                                                                                                                       //on parcourt les lignes de l'AS400
                check = false;
                for (int j = 0; j < dtsql.Rows.Count || dtsql.Rows.Count > 0 || !check; j++)                                                        //on parcourt les lignes de SQL Server
                {
                    if ((string)dtas.Rows[i][dtas.Columns["ORDNO"].Ordinal] == (string)dtsql.Rows[j][dtsql.Columns["SNOF"].Ordinal] &&
                        (string)dtas.Rows[i][dtas.Columns["OPSEQ"].Ordinal] == (string)dtsql.Rows[j][dtsql.Columns["SOPE"].Ordinal])                //si le couple clé primaire est identique
                    {                                                                                                                               
                        if ((string)dtas.Rows[i][dtas.Columns["OPSTC"].Ordinal] != (string)dtsql.Rows[j][dtsql.Columns["SITUATION"].Ordinal] &&
                            Convert.ToInt32((string)dtas.Rows[i][dtas.Columns["OPSTC"].Ordinal]) == 0)                                              //si la situation est différente et OPSTC = "00"
                        {                                                                                                                           
                            DT.Rows.Add(dtas.Rows[i].ItemArray);                                                                                    //on ajoute la ligne au datatable temporaire
                            dtsql.Rows.Remove(dtsql.Rows[j]);                                                                                       //et on supprime la ligne
                            check = true;
                        }
                        if ((string)dtas.Rows[i][dtas.Columns["WKCTR"].Ordinal] != (string)dtsql.Rows[j][dtsql.Columns["SPDC"].Ordinal])            //si les postes sont identiques
                        {
                            DT.Rows.Add(dtas.Rows[i].ItemArray);                                                                                    //on ajoute la ligne au datatable temporaire
                            DT.Rows[DT.Rows.Count - 1][DT.Columns["OPSTC"].Ordinal] = (string)dtsql.Rows[j][dtsql.Columns["SITUATION"].Ordinal];    //on rajoute la situation de SQL Server
                            dtsql.Rows.Remove(dtsql.Rows[j]);                                                                                       //et on supprime la ligne
                            check = true;
                        }
                        if (!check)
                        {
                            dtsql.Rows.Remove(dtsql.Rows[j]);
                            check = true;
                        }
                    }
                    else                                                                                                                            //si le couple clé primaire est différent
                    {
                        if (Convert.ToInt32(((string)dtas.Rows[i][dtas.Columns["ORDNO"].Ordinal]).TrimStart('M')) > Convert.ToInt32(((string)(dtsql.Rows[j][dtsql.Columns["SNOF"].Ordinal])).TrimStart('M')) ||
                            Convert.ToInt32(((string)dtas.Rows[i][dtas.Columns["ORDNO"].Ordinal]).TrimStart('M')) == Convert.ToInt32(((string)(dtsql.Rows[j][dtsql.Columns["SNOF"].Ordinal])).TrimStart('M')) &&
                            Convert.ToInt32(((string)dtas.Rows[i][dtas.Columns["OPSEQ"].Ordinal]).TrimStart('M')) > Convert.ToInt32(((string)(dtsql.Rows[j][dtsql.Columns["SOPE"].Ordinal])).TrimStart('M')))
                        {                                                                                                                           //si (ORDNO > SNOF) ou (ORDNO = SNOF et OPSEQ > SOPE)
                            dtsql.Rows.Remove(dtsql.Rows[j]);                                                                                       //on supprime la ligne et on passe à la suivante
                            check = true;
                        }
                    }
                }
 
                if (!check)                                                                                                                         //si non trouvée, on ajoute la ligne au datatable temporaire
                {
                    DT.Rows.Add(dtas.Rows[i].ItemArray);
                }
            }

Merci aux courageux qui tenteront l'expérience...

pas lu le code, mais pour comparer des données autant laisser ça aux bases de données

il est possible de faire des requetes sur différentes bases d'un même serveur
et il est possible de faire des liens entres 2 serveurs (serveur lié), y compris vers un AS400

après pour savoir s'il y a des différentes il suffit d'écrire des choses dans le genre

Code:

select id from table1 where id not in (select id from autrebase.table1)

ou encore
Code:

1 2 3 if (select * from table 1 union select * from autrebase.table1) <> (select count(*) from table1) print 'il y a au moins une différence sur une "cellule" pour table1' -- UNION supprime les doublons contrairement à UNION ALL<
et selon ce que tu veux faire il y a moyen de faire des requêtes dans le genre
c'est simple et performant

Oui mais non...

Citation:
Envoyé par Pol63
pas lu le code, mais pour comparer des données autant laisser ça aux bases de données

il est possible de faire des requetes sur différentes bases d'un même serveur
et il est possible de faire des liens entres 2 serveurs (serveur lié), y compris vers un AS400

après pour savoir s'il y a des différentes il suffit d'écrire des choses dans le genre

Code:

select id from table1 where id not in (select id from autrebase.table1)

ou encore
Code:

1 2 3 if (select * from table 1 union select * from autrebase.table1) <> (select count(*) from table1) print 'il y a au moins une différence sur une "cellule" pour table1' -- UNION supprime les doublons contrairement à UNION ALL<
et selon ce que tu veux faire il y a moyen de faire des requêtes dans le genre
c'est simple et performant
Oui je sais tout ça mais ce n'est pas ce qui m'intéresse. Actuellement mon serveur AS400 est déclaré en tant que serveur lié dans SQL Server (je passe donc des commandes sql alimentant directement mes tables), mais j'ai des pb de perf et de gestion des erreurs. J'ai donc créé un programme censé copier le comportement de ces requêtes qui sont somme toutes fonctionnelles, mais ajoutant une couche de gestion relativement importante. Voilà pk j'utilise des DataTables, et voilà pk j'ai besoin de ce que je demande :)
Le but est justement de se passer du serveur lié.

15/05/2014, 13h36
Pol63

le datatable n'est pas ce qu'il y a de plus performant (c'est peut etre quand même mieux que le serveur lié) et puis c'est pas très joli à lire
les serveurs liés sont plus performants avec des openquery que des serveur.base..table aussi mais ca peut rester plus poussif que de l'accès direct via ado.net

plus de détails sur le fonctionnement souhaité ? (peu de colonnes à vérifier, un clé primaire intervient ? ...)
15/05/2014, 14h11
q.migliore

Redéfinition du problème

Citation:

Envoyé par Pol63

le datatable n'est pas ce qu'il y a de plus performant (c'est peut etre quand même mieux que le serveur lié) et puis c'est pas très joli à lire
les serveurs liés sont plus performants avec des openquery que des serveur.base..table aussi mais ca peut rester plus poussif que de l'accès direct via ado.net

plus de détails sur le fonctionnement souhaité ? (peu de colonnes à vérifier, un clé primaire intervient ? ...)

Je n'ai peut-être pas assez clair mais mon soucis n'est pas dans la technologie mais dans la technique pure. J'ai présenté mon contexte mais en réalité c'était plus pour la forme et la compréhension globale.
Mon seul soucis à l'heure actuelle vient simplement d'une exception (IndexOutOfRange) d'un For. Si vous jeter un œil au code peut-être que vous trouverez immédiatement d'où vient le problème. Il vient certainement des conditions de sortie de mes for imbriqués mais j'ai essayé pas mal de choses et je n'y arrive pas.
Le reste du code est également là pour faire joli ^^
Donc si vous le trouvez trop difficile à lire ne vous concentrez que sur les for...
15/05/2014, 15h31
katkiller

Vu l'erreur que tu as, on va avoir du mal à t'aider.
Il faut debuger...
Tu as toutes les informations : Tu sais que tu essai de lire une ligne ou une colonne d'un tableau ou (DataTable) qui n'existe pas.

Et personnellement tes "if" avec pleins de conditions qui utilisent des "i" des "j" et des "Ordinal", je trouve ça horrible à lire...
Et :

Code:

j < dtsql.Rows.Count || dtsql.Rows.Count > 0 || !check

Hum ça veut dire quoi ?

Citation:

Envoyé par katkiller

Vu l'erreur que tu as, on va avoir du mal à t'aider.
Il faut debuger...
Tu as toutes les informations : Tu sais que tu essai de lire une ligne ou une colonne d'un tableau ou (DataTable) qui n'existe pas.

Et personnellement tes "if" avec pleins de conditions qui utilisent des "i" des "j" et des "Ordinal", je trouve ça horrible à lire...
Et :

Code:

j < dtsql.Rows.Count || dtsql.Rows.Count > 0 || !check

Hum ça veut dire quoi ?

En vérité je me suis un peu embrouillé dans les conditions de sortie en essayant de debugguer et c'est justement pour ça que je post mon soucis.
Mais actuellement si je supprime mes dtsql.Rows.Remove(dtsql.Rows[j]) (et donc que j'arrête de supprimer des lignes d'un datatable sur lequel je suis en train de boucler xD) et que je mets ceci comme conditions de sortie, cela fonctionne :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 ... for (int i = 0; i < dtas.Rows.Count; i++) //on parcourt les lignes de l'AS400 { ... for (int j = 0; j < dtsql.Rows.Count && dtsql.Rows.Count > 1 || !check; j++) //on parcourt les lignes de SQL Server { ... } if (!check) //si non trouvée, on ajoute la ligne au datatable temporaire { DT.Rows.Add(dtas.Rows[i].ItemArray); } } ...
dtsql.Rows.Count correspond au nombre de ligne du DataTable dtsql (celui qui récupère les données de ma table venant de SQL Server), et !check est la condition qui me permet de quitter ma boucle si la ligne comparée est traitée (soit ajoutée au datatable temporaire soit supprimée de dtsql).
Suis-je clair ?

En fait concernant la raison d'être de check, tout est soucis de performance.
Si je me passais de tous ces "stratagèmes" mon traitement prendrait une demi-journée alors que là j'ai réussi à le réduire à moins d'une demi-heure... (sans la suppression de ligne, qui elle réduirait encore ce temps d'au moins 50%
-> raison de ma présence ici ^^).

15/05/2014, 16h07
egautier18
Bonjour,

Je pense que vous pouvez lire votre deuxième boucle en partant de la fin.
Code:

1 2 3 4 5 for (int i = dtsql.Rows.Count - 1; i >= 0; i--) { dtsql.Rows[i].Delete(); }
A+

Pour la variable "check" je te conseille d'aller faire un tour du coté de "break"-"continue" qui peuvent être très intéressante dans une boucle.

Voici un petit exemple en mode console pour ne pas traiter les boucles quand i=1 et j=3 :

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
28
29
            var ramdom = new Random();
            int x = 5;
            var tableau = new Double[x + 1, x + 1];
            for (int i = 0; i <= x; i++)
            {
                if (i == 1)
                {
                    Console.WriteLine("----- Stop i -----");
                    continue;
                }
                else
                {
                    Console.WriteLine("i-->{0}", i);
                }
                for (int j = 0; j <= x; j++)
                {
                    if (j == 3)
                    {
                        Console.WriteLine("--- Stop j ---");
                        continue;
                    }
                    else
                    {
                        Console.WriteLine("{0}<--j", j);
                    }
                    var num = ramdom.Next(2000) * 0.55;
                    tableau[i, j] = num;
                }
            }

15/05/2014, 16h28
q.migliore
Citation:
Envoyé par egautier18

Bonjour,

Je pense que vous pouvez lire votre deuxième boucle en partant de la fin.

Code:

1 2 3 4 5 for (int i = dtsql.Rows.Count - 1; i >= 0; i--) { dtsql.Rows[i].Delete(); }

A+
Heu... Très bien mais dans quel but ?
Je vois déjà deux soucis à cela.
Le 1er, de moindre importance, est que [I]dtsql.Rows.Delete() ne fais que vider la ligne sans la supprimer (étant donné que je fais des tests sur les champs ça me pètera à la figure à ce moment là), il me paraît donc plus judicieux de supprimer carrément la ligne (gain au prochains tours de boucle) avec dtsql.Rows.Remove(dtsql.Rows[j]);.
Et le 2ème est que, toujours dans un soucis d'optimisation des performances xD, ça m'arrange bcp de parcourir mes tables dans l'ordre car les résultats sont triés par ordre croissant, ce qui me permet de trouver rapidement les lignes identiques.
Je m'explique, imaginez deux listes (ou DataTables), contenant des lettres triées par ordre alphabétique, si je suis en train de chercher si la lettre C présente dans l1 existe dans l2, je vais chercher à partir de la 1ère ligne de l2, jusqu'à ce que je trouve, sois C, soit la lettre suivante disponible (D, ou E, ou F, etc.).
Dans mon cas il s'agit en fait de trier sur des couples de clés primaires, donc une fois un couple identique trouvé, il est impossible qu'il soit présent deux fois, donc je peux le supprimer afin de ne pas avoir à le traiter à nouveau dans mes prochains tours de boucle.

Citation:

Envoyé par katkiller

Pour la variable "check" je te conseille d'aller faire un tour du coté de "break"-"return"-"continue". Dans les boucle.

Voici un petit exemple en mode console pour ne pas traiter les boucles quand i=1 et j=3 :

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
28
29
            var ramdom = new Random();
            int x = 5;
            var tableau = new Double[x + 1, x + 1];
            for (int i = 0; i <= x; i++)
            {
                if (i == 1)
                {
                    Console.WriteLine("----- Stop i -----");
                    continue;
                }
                else
                {
                    Console.WriteLine("i-->{0}", i);
                }
                for (int j = 0; j <= x; j++)
                {
                    if (j == 3)
                    {
                        Console.WriteLine("--- Stop j ---");
                        continue;
                    }
                    else
                    {
                        Console.WriteLine("{0}<--j", j);
                    }
                    var num = ramdom.Next(2000) * 0.55;
                    tableau[i, j] = num;
                }
            }

Il est clair que "break return et continue" m'aiderai bien mais on m'a tjr appris que c'était le mal de les utiliser donc j'essai de faire sans xD.
Je suis presque sûr que la grouille vient de mes conditions mais j'ai mal au crane à force de me prendre la tête là-dessus...

Si je peux apporter ma contribution

Citation:

Envoyé par egautier18

Bonjour,

Je pense que vous pouvez lire votre deuxième boucle en partant de la fin.

Citation:

Envoyé par q.migliore

Heu... Très bien mais dans quel but ?
Je vois déjà deux soucis à cela.
Le 1er, de moindre importance, est que [I]dtsql.Rows.Delete() ne fais que vider la ligne sans la supprimer (étant donné que je fais des tests sur les champs ça me pètera à la figure à ce moment là), il me paraît donc plus judicieux de supprimer carrément la ligne (gain au prochains tours de boucle) avec dtsql.Rows.Remove(dtsql.Rows[j]);.

C'est justement parce que tu supprimes des lignes qu'il faut itérer en arrière étant donnée que la condition de ton itération est basée sur le Count !

-----------

Autre chose :
Code:

1 2 3 4 for (int j = 0; j < dtsql.Rows.Count && dtsql.Rows.Count > 1 || !check; j++) //on parcourt les lignes de SQL Server { ... }
Je dis peux être une connerie mais ne serait-il pas préférable d'encadrer les conditions qui doivent être testées ensembles lorsque qu'il est des ET et OU dans la condition.

Citation:
Envoyé par popo
Si je peux apporter ma contribution

C'est justement parce que tu supprimes des lignes qu'il faut itérer en arrière étant donnée que la condition de ton itération est basée sur le Count !

-----------

Autre chose :
Code:

1 2 3 4 for (int j = 0; j < dtsql.Rows.Count && dtsql.Rows.Count > 1 || !check; j++) //on parcourt les lignes de SQL Server { ... }
Je dis peux être une connerie mais ne serait-il pas préférable d'encadrer les conditions qui doivent être testées ensembles lorsque qu'il est des ET et OU dans la condition.
Merci pour ta contribution ^^
Ok j'avais pas vu l'intérêt de la boucle à l'envers, je vais tester ça demain.
Pour les conditions... Et bien bonne question. Je ne crois pas que cela soit indispensable mais à tout hasard je vais le faire qd même ! xD