Discussion :

[facilus68]

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 :

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
 
 
@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

[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.

[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 :

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

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

[facilus68]

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

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

[thelvin]

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