Discussion :

[sebagot]

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 :

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

[foetus]

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

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

J'aurais plutôt fait un code comme cela ... mais non testé et améliorable selon tes données :

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
    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 ) .... sans test donc
Édit sauvage 2: C'est vrai que l'algo de thelvin est plus propre 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

[thelvin]

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 :

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

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

[sebagot]

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

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

[thelvin]

Ma foi ça marche, mais je ne trouve pas ça très clair, comme construction.

[sebagot]

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

Merci