Encore problème d'imbrication de boucles

Bonjour,
j'ai encore un souci avec l'imbrication des boucles for,je croyais que c'était résolu depuis ma dernière discussion..
j'ai le code suivant:

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14

for (iter = product.values().iterator(); iter.hasNext();) { productID =((Integer)iter.next()); for (Iterator itr = clients.keySet().iterator(); itr.hasNext();) { aClientId = (Integer)itr.next(); for (Iterator itr2 = clients.keySet().iterator(); itr2.hasNext();) { bClientId = (Integer)itr2.next(); if (aClientId != bClientId) { rapp = getRapp(aClientId,bClientId); form = rapp*getNbr(); System.out.println( aClientId+"|"+ productID+"|"+form); } } }

---

on dirait que c'est une boucle infinie, ça m'affiche:

Citation:

---

aClient1|product1|form
aClient1|product1|form
aClient1|product1|form
aClient2|product2|form
aClient2|product2|form
aClient3|product3|form
aClient3|product3|form
aClient3|product3|form

---

...............ça génère des millions de lignes :(

Normalement ça devrait afficher une seule valeur "form" par "aClient" et par "product", ça ne devrait pas se répéter.
et il doit y avoir finalement 10000 valeurs correspondant au nombre de aClient et de produits que j'ai dans mon fichier d'entrée.
Merci pour votre aide.

Avec la balise

Code:

---

...

---

ça serait plus lisible.

Tu fais une double boucle sur ta map "client", donc pour chaque client, tu vas afficher (nombre_clients -1) fois ce client par produit... Peut-être devrais-tu cumuler pour chaque client avant d'afficher ?

En fait le productID et aClient, je les récupère d'une même HashMap où les keys sont les aClients et les values sont les products. et j'ai besoin de récupérer à la fin 10000 résultats, correspondant aux itérations sur cette même HashMap.
concernant le bClient, je le récupère d'une autre HashMap indépendante.
j'ai essayé le code suivant:

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14

Iterator iter = clients.values().iterator(); iter.hasNext(); for (Iterator itr = clients.keySet().iterator(); itr.hasNext() { aClientId = (Integer)itr.next(); productID =((Integer)iter.next()); for (Iterator itr2 = clients.keySet().iterator(); itr2.hasNext() { bClientId = (Integer)itr2.next(); if (aClientId != bClientId) { rapp = getRapp(aClientId,bClientId); form = rapp*getNbr(); System.out.println( aClientId+"|"+ productID+"|"+form); } } }

---

ça me donne une seule valeur par aClient et par product:
aClient1|product1|form
aClient2|product3|form
aClient3|product4|form
il fait pas toutes les itérations pour chauqe aClient. :roll:

N'hésite pas à utiliser les foreach plutot que les iterators, ça fait un code plus léger

Code:

---

1 2 3 4 5 6

for (Map.Entry<t1, t2> e : clients.keyset()) { e.getKey(); e.getValue(); }

---

Du coup, si j'ai bien compris ton code ça donnerait :

Code:

---

1 2 3 4 5

for (Map.Entry<Integer, Product> entA : clientsA.keySet()) for (Map.Entry<Integer, Product> entB : clientsB.keySet()) if (! entA.getKey().equals(entB.getKey())) // blabla

---

Maintenant, ton explication de la boucle est mi-claire du coup je suis peut etre à coté de la plaque.