Tri d'une List en fonction de 3 criteres (nom, DateDebut, DateFin)

Version imprimable

J'ai implementé un comparator pour faire le tri en fonction de 3 criteres (nom, dateDebut et dateFin).

J'ai appelé la méthode Collections.sort(List, new ObjetVoComparator())

Apres, J'ai affiché la liste trié et les elements ne le sont pas.

Voici le code du comparator :
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 @Override public int compare(AffretementVO pO1, AffretementVO pO2) { String sousTraitant1 = pO1.getSousTraitant(); String sousTraitant2 = pO2.getSousTraitant(); Date dateDebut1 = pO1.getDebutPeriode(); Date dateDebut2 = pO2.getDebutPeriode(); Date dateFin1 = pO1.getFinPeriode(); Date dateFin2 = pO2.getFinPeriode(); if(!sousTraitant1.equals(sousTraitant2)) { return 0; } else { if(!dateDebut1.equals(dateDebut2)) { return 0; } else { if(!dateFin1.equals(dateFin2)) { return 0; } else { return 1; } } } }
LOG Affichée :
TRP88170-0002 - 2011-12-05 - 2011-12-11
TRP88170-0002 - 2011-11-28 - 2011-12-04
TRP88170-0002 - 2011-10-10 - 2011-10-20
TRP88170-0002 - 2011-12-05 - 2011-12-11
TRP88170-0002 - 2011-11-28 - 2011-12-04
TRP88170-0002 - 2011-11-28 - 2011-12-04
TRP88170-0002 - 2011-11-28 - 2011-12-04

Le tri sur le premier critere nom se fait correctement mais les dates non, le résultat attendu est le suivant :

TRP88170-0002 - 2011-10-10 - 2011-10-20
TRP88170-0002 - 2011-11-28 - 2011-12-04
TRP88170-0002 - 2011-11-28 - 2011-12-04
TRP88170-0002 - 2011-11-28 - 2011-12-04
TRP88170-0002 - 2011-11-28 - 2011-12-04
TRP88170-0002 - 2011-12-05 - 2011-12-11
TRP88170-0002 - 2011-12-05 - 2011-12-11

12/12/2011, 10h18
thelvin

Je ne sais pas ce que tu croyais, mais la méthode compare() doit renvoyer :
- 0 si les deux objets sont égaux
- un nombre strictement négatif si le premier objet est strictement inférieur au second
- un nombre strictement positif si le premier objet est strictement supérieur au second

Or, ta méthode :
- passe son temps à renvoyer 0 quand certaines propriétés ne sont pas égales.
- ne renvoie jamais de nombre négatif.

Deux raisons pour lesquelles elle est forcément incorrecte.
12/12/2011, 10h21
hwoarang
Salut,

D'apres la javadoc de Comparator, public int compare(Object o1, Object o2) doit renvoyer une valeur :
< 0 si o1 < o2
= 0 si o1 = o2
> 0 si o1 > o2.

Dans ton cas, tu fais :
Code:

1 2 3 if(!sousTraitant1.equals(sousTraitant2)) { return 0; }
Donc tu renvoies 0 si ils sont differents. A mon avis, le resultat que tu obtiens depend de l'ordre dans lequel tu inseres tes données.
Bref, il faut reprendre ton comparateur ;)

EDIT : grilled

Il faillais que j'utilise le compareTo au lieu du equals :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
 
if(sousTraitant1.compareTo(sousTraitant2) != 0) {
 
            return sousTraitant1.compareTo(sousTraitant2) >0 ? 1:-1;
 
        } else {
 
            if (dateDebut1.compareTo(dateDebut2) !=0 ){
 
                return dateDebut1.compareTo(dateDebut2)>0 ? 1: -1;
 
            }else {
 
                return dateFin1.compareTo(dateFin2)>0 ? 1:-1;
 
           }
 
        }

12/12/2011, 10h54
thelvin

Ouaip, mais personnellement, je stockerais le résultat de compareTo() dans un int, plutôt que l'appeler deux fois de suite.