iterator et comparaison

salut

je désire vérifier dans deux linkedlist les éléments qui ont correspondance et faire un traitement...

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

List listCheque101 = new LinkedList(); List listCheque100 = new LinkedList();   ....   public void matchCheque(){ Iterator it100=listCheque100.iterator(); Iterator it101=listCheque101.iterator();   while (it100.hasNext()){ // tant que j'ai un element non parcouru Cheque o100 = (Cheque) it100.next();   while (it101.hasNext()){ // tant que j'ai un element non parcouru Cheque o101 = (Cheque) it101.next();   //si les données sont identique, il y a addition des nombres de cheque if(o100.getFederation().equals(o101.getFederation()) && o100.getCaisse().equals(o101.getCaisse()) && o100.getGuichet().equals(o101.getGuichet()) && o100.getDateCollecte().equals(o101.getDateCollecte()) && o100.getHeureCollecte().equals(o101.getHeureCollecte()) ){ o100.setNbCheque(o100.getNbCheque() + o101.getNbCheque());   //ne passe jamais ici   } } } }

---

si je fais affiché le contenu de listCheque100, j'obtiens

815 92276 04 2007-05-13 20:06:10 339
815 20184 84 2007-05-13 21:03:53 152
815 20426 66 2007-05-13 20:33:57 164

si je fais affiché le contenu de listCheque101, j'obtiens
815 20184 84 2007-05-13 21:03:53 2
815 20426 66 2007-05-13 20:33:57 1


il y a donc une correspondance avec la deuxième ligne de listCheque100 et la première ligne de listCheque101...

pourtant il passejamais à l'intérieur du if....
une idée?

merci

J'ai pas vu où si situe l'erreur mais voici un conseil :
- plutot que de comparer dans tes boucles chaque champs, redéfini la méthode equals() dans ta classe Cheque.
- lorsque tu auras défini la méthode equals, tu n'auras plus besoin de faire deux boucles imbriquées, une seule suffira : la boucle itérera sur chaque objet contenu dans la première liste, ensuite il suffit d'utiliser la méthode contains() sur ta liste numéro deux ( if(maListeDeux.contains(monChèque)){... ). "Contains" utilise la méthode "equals" des objets pour tester l'inclusion. ;-)

A lire ton code, j'ai l'impression que ton but est de comparer CHAQUE chèque de ta première liste à CHAQUE chèque de la seconde, je me trompe?

Or ce que fait ton code, c'est comparer le PREMIER chèque de ta première liste à chaque chèque de la seconde! Comme ta correspondance est pour le deuxième élément de ta première liste, tu ne fais effectivement pas le traitement voulu.

Il faut régénérer l'itérateur it101 pour chaque traitement d'un chèque de ta première liste avant de faire ton 2ème while.

Sinon, je suis d'accord pour la surcharge de la méthode equals ;)

Tu peux aussi utiliser la méthode boolean retainAll(Collection) de List.
Ainsi tu fait une copie d'une de tes listes et retient ce qu'il y a dans l'autres; bien sûr tu dois redéfinir la méthode equals comme dit dans les précédents post

Code:

---

1 2 3 4

List tmp = new ArrayList(listCheque100); tmp.retainAll(listCheque101); System.out.println(tmp);

---

en plein ça CyberChouan

ok j'ai corrigé le problème concernant les deux boucle while

maintenant je veux tenter de mettre en oeuvre la solution à une boucle

voici ce que j'ai écrit (pas testé pas de compilo où je suis...)

Code:

---

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

//dans ma classe chèque public boolean equals(Cheque cheque){   if(this.getFederation().equals(cheque.getFederation()) && this.getCaisse().equals(cheque.getCaisse()) && this.getGuichet().equals(cheque.getGuichet()) && this.getDateCollecte().equals(cheque.getDateCollecte()) && this.getHeureCollecte().equals(cheque.getHeureCollecte()) ){   this.setNbCheque(this.getNbCheque()+ cheque.getNbCheque());   return true; } return false; }

---

ensuite je boucle

Code:

---

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

Iterator it100=listCheque100.iterator(); while (it100.hasNext()){ Cheque o100 = (Cheque) it100.next(); if(it100=listCheque100.contains(Cheque)){   /*   fait ce qu'on veux   */   } }

---

alors c'est gagnant?

Citation:

---

Envoyé par robert_trudel

alors c'est gagnant?

---

Heu... en fait NON! Apparemment tu as un peu de mal avec les différents objets...

Tout d'abord (c'est technique), il faudrait que tu surcharges la méthode equals(Object) pour être propre.

Ensuite, que vient faire une affectation dans une méthode "equals()"?... cette ligne est presque une hérésie de conception:

Code:

---

1 2	this.setNbCheque(this.getNbCheque()+ cheque.getNbCheque());

---

Et enfin, la ligne de test qui est incompréhensible:

Code:

---

1 2 3 4

if(it100=listCheque100.contains(Cheque)) { // ... }

---

un égal simple => affectation, et non comparaison d'égalité
que viens faire it100 (qui est un iétareur) dans ce test?
tu appliques contains à "Cheque" alors que tu veux certainement l'appliquer à cheque
la méthode contains() te renvoie déjà un booleen... pourquoi veux-tu le comparer à autre chose?...


Bref, je pense que tu devrais revoir calmement les bases de java avant de t'attaquer à ton problème de chèques

Citation:

---

Envoyé par CyberChouan

Heu... en fait NON! Apparemment tu as un peu de mal avec les différents objets...

Tout d'abord (c'est technique), il faudrait que tu surcharges la méthode equals(Object) pour être propre.

Ensuite, que vient faire une affectation dans une méthode "equals()"?... cette ligne est presque une hérésie de conception:

Code:

---

1 2	this.setNbCheque(this.getNbCheque()+ cheque.getNbCheque());

---

Et enfin, la ligne de test qui est incompréhensible:

Code:

---

1 2 3 4

if(it100=listCheque100.contains(Cheque)) { // ... }

---

un égal simple => affectation, et non comparaison d'égalité
que viens faire it100 (qui est un iétareur) dans ce test?
tu appliques contains à "Cheque" alors que tu veux certainement l'appliquer à cheque
la méthode contains() te renvoie déjà un booleen... pourquoi veux-tu le comparer à autre chose?...


Bref, je pense que tu devrais revoir calmement les bases de java avant de t'attaquer à ton problème de chèques

---

ça devrait être mieux

Code:

---

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

public boolean equals(Object o){   Cheque cheque = (Cheque) o;   if(this.getFederation().equals(cheque.getFederation()) && this.getCaisse().equals(cheque.getCaisse()) && this.getGuichet().equals(cheque.getGuichet()) && this.getDateCollecte().equals(cheque.getDateCollecte()) && this.getHeureCollecte().equals(cheque.getHeureCollecte()) ){ return true; } return false; }   }

---

j'avais mis l'affectation dans la comparaison, car je sais pas si c'est possible d'obtenir les infos de l'items qui est comparé dans le if ci-dessous

Code:

---

1 2 3 4 5 6 7

Iterator it100=listCheque100.iterator(); while (it100.hasNext()){ Cheque o100 = (Cheque) it100.next(); if(listCheque101.contains(o100)){ } }

---

C'est beaucoup mieux!

Il ne te reste plus qu'à faire quelque chose lorsque ton "if" est vérifié!

Code:

---

1 2 3 4 5 6 7 8 9

Iterator it100=listCheque100.iterator(); while (it100.hasNext()){ Cheque o100 = (Cheque) it100.next(); if(listCheque101.contains(o100)){ o100.setNbCheque(o100.getNbCheque() + listCheque101.get(listCheque101.indexOf(o100))); } }

---

Remarque: ce code fonctionne avec n'importe quelle implémentation corercte de l'interface "List"
Remarque 2: il n'est pas optimal car la deuxième liste est parcourue deux fois (une pour la méthode contains(), et une autre pour la méthode indexOf en cas de chèque trouvé)
Remarque 3: il peut être buggué (problème de conception), si la deuxième liste peut contenir deux chèques identiques (au sens de "equals") à un chèque de la première liste: seul le premier des 2 chèques de la 2ème liste sera traité.

puisqu'il peut avoir des doublons dans chaque liste, j'ai opté pour un HashSet

au lieu d'utiliser 2 while ou bien la dernière proposé, il y a t'il mieux avec un hashSet?