Discussion :

[flora806]

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 :

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

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:

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.

[doons]

Avec la balise

Code :

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

 ...

ça serait plus lisible.

[DevTeam_]

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 ?

[flora806]

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 :

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

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.

[Faiche]

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

Code :

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

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 :

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

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.