Double boucle pour un resultset

Version imprimable

Bonjour,

Je fais un select sur une table commandeClient qui est composé de ligneCommande. Mon but est de créer une ArrayList de Commande client.
Je cale un peu pcq je boucle pour avoir les infos sur ma commandes et dans cette boucle je boucle pour prendre toutes les lignes. Au début j'avais fais deux while(donnees.next) mais alors quand je passe à la nouvelle commande il lit deux fois. du coup j'ai essayé avec un donnees.islAst mais il n'en veut pas.
Et quand je test ce code il me trouve qu'une commande alors qu'il y en a deux.

PS : les méthodes constructionCommande et constructionLigneCommande ne font que garnir les objets

Merci :)

Voilà le code

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
ArrayList<CommandeClient> liste=new ArrayList<CommandeClient>();
        CommandeClient cc;
 
        String requeteSQL="select  * "
                          +"from Client c join CommandeClient cc "
                          +"on c.Numéro=cc.NumClient "
                          +"join ligneCommande l " 
                          +"on cc.\"NUMÉRO\"=l.COMMANDECLIENTNUM " 
                          +"join marchandise m " 
                          +"on l.MARCHANDISE=m.\"RÉFÉRENCE\" " 
                          +"join adresse a "
                          +"on(a.rue=cc.LivraisonRue and a.numéro=cc.LivraisonNuméro and a.codepostal=cc.LivraisonCodepostal and a.localité=cc.LivraisonLocalité) "
                          +"where cc.NUMCLIENT=?";
        try{
            PreparedStatement prepStat = connexion.prepareStatement(requeteSQL);
            prepStat.setInt(1, numCli);
 
            ResultSet donnees = prepStat.executeQuery();
            ResultSetMetaData meta = donnees.getMetaData();            
            donnees.next();
 
            while(donnees.isLast()==false){
                int numCom=donnees.getInt("Numéro");
                cc=constructionCommande(donnees,meta);
                cc.addLigneCommande(constructionLigneCommande(donnees));
 
                while(donnees.next()&&numCom==donnees.getInt("Numéro")){
                    /*
                    On ajoute les lignes de commandes
                    */
 
                    cc.addLigneCommande(constructionLigneCommande(donnees));                    
                }               
                liste.add(cc);
            }
        }

C'est subtil mais tu ne gère pas le dernier élément dans 2 cas :mrgreen: :

Si ton résultat ne contient qu'un seul élément
Ton dernier élément à un numéro différent de l'avant dernier

Au vu de ces 2 cas, c'est très difficile de te répondre :?

Es-tu obligé de faire des insertions à la fin? :koi:

J'aurais plutôt fait un code comme cela ... mais non testé et améliorable selon tes données :whistle: :
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 ResultSet donnees = prepStat.executeQuery(); ResultSetMetaData meta = donnees.getMetaData(); if ( donnees.next() ) { int numCom = donnees.getInt("Numéro"); int current_numCom = ((numCom == 0)? 1: 0); bool is_first = true; do { numCom = donnees.getInt("Numéro"); if (current_numCom != numCom) { // Ou alors tester si cc n'est pas vide au lieu de créer un booléen if (!is_first) { liste.add(cc); } current_numCom = numCom; cc = constructionCommande(donnees,meta); is_first = false; } cc.addLigneCommande( constructionLigneCommande(donnees) ); } while( donnees.next() ); liste.add(cc); }
Édit sauvage 1: Avec des insertions à la fin, il faut toujours ajouter en dernier (si je ne dis pas de bêtises :mouarf:) .... sans test donc
Édit sauvage 2: C'est vrai que l'algo de thelvin est plus propre :mrgreen: avec ces insertions non pas à la fin, le premier next() qui permet de positionner sur le premier élément ... si on n'est pas obligé de le faire et son test OU pour gérer le premier numéro

Bien que d'habitude je n'aime pas initialiser de variables à une valeur bidon, une structure en if/while me semble un peu rude. Je ferais ceci :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
int currentNum = 0;
CommandeClient cc = null;
 
while(donnees.next()) {
  int num = donnees.getInt("Numéro");
  LigneCommande ligneCommande = constructionLigneCommande(donnees);
 
  if(cc == null || currentNum != num) {
    cc = constructionCommande(donnees,meta);
    list.add(cc);
    currentNum = num;
  }
  cc.addLigneCommande(ligneCommande);
}

Merci pour vos réponse,
finalement j'ai réussi en ajoutant une variable boolean qui prenait la sortie

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
Boolean suivant;
suivant=donnees.next());
while(suivant){
                int numCom=donnees.getInt("Numéro");
                cc=constructionCommande(donnees,meta);
                cc.addLigneCommande(constructionLigneCommande(donnees));
 
                while(suivant&&numCom==donnees.getInt("Numéro")){
                    /*
                    On ajoute les lignes de commandes
                    */
 
                    cc.addLigneCommande(constructionLigneCommande(donnees));          
                    suivant=donnees.next();
                }               
                liste.add(cc);
            }

12/05/2014, 18h15
thelvin

Ma foi ça marche, mais je ne trouve pas ça très clair, comme construction.
12/05/2014, 18h50
sebagot

Oui, c'est vrai que ta solution est peut-être plus clair, je vais faire comme ça alors :)

Merci :)