Trier un arrayList

Version imprimable

Trier un arrayList

Bonjour,
j’essaye de trier une liste d'objet.
Pour cela j'ai créer une classe qui implémente de comparator mais le tri n'a pas lieu.

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
 
public class CutOffCommandeCompareCode  implements Comparator {
 
	public CutOffCommandeCompareCode()
	{
 
	}
 
	public int compare(Object arg0, Object arg1) {
 
		CutOffCommande c1=(CutOffCommande)arg0;
		CutOffCommande c2=(CutOffCommande)arg1;
		int result=c1.getCodeTypeBudget().trim().compareTo(c1.getCodeTypeBudget().trim());
		if(result==0)
		{
			result=c1.getCodeFournisseur().trim().compareTo(c2.getCodeFournisseur().trim());
			if(result==0)
			{
				result=c1.getNumeroCommande().trim().compareTo(c2.getNumeroCommande().trim());
			}
		}
		return result;
	}
 
}

et pour faire le tri j'utilise la syntaxe suivante:

Code:

1
2
 
Collections.sort(coList,new CutOffCommandeCompareCode());

Je ne vois pas d’où viens le problème.

Merci de vos réponse

05/04/2011, 16h37
LinkinSelim
à premiere vue, t'a une erreur dans cette ligne tu compare c1 à lui même :
Code:

1 2 int result=c1.getCodeTypeBudget().trim().compareTo(c1.getCodeTypeBudget().trim());
05/04/2011, 16h45
tamildark

Yep, je suis un boulet. J'aurai voulu que ça ne résout pas mon problème.
Malheureusement le tri ne fonctionne toujours pas :s

T'as un problème avec quelques else et la valeur de ton result sera peut etre modifié avant le dernier return.

Essai plutot ça :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
 
	public int compare(Object arg0, Object arg1) {
 
		CutOffCommande c1=(CutOffCommande)arg0;
		CutOffCommande c2=(CutOffCommande)arg1;
		int result=c1.getCodeTypeBudget().trim().compareTo(c2.getCodeTypeBudget().trim());
		if (result<>0) return result; //Si les codeTypeBudget ne sont pas egaux retourner le resultat leur comparaison
		//à ce point du programme c1.getCodeTypeBudget().trim et c2.getCodeTypeBudget().trim() sont egaux
		//Alors on compare les codeFournisseurs.
		result=c1.getCodeFournisseur().trim().compareTo(c2.getCodeFournisseur().trim());
		if (result<>0) return result; //Si les codeFournisseur ne sont pas egaux retourner le resultat leur comparaison
		//à ce point du programme c1.getCodeFournisseur().trim() et c2.getCodeFournisseur().trim() sont egaux
		//Alors on compare les NumeroCommande
		result=c1.getNumeroCommande().trim().compareTo(c2.getNumeroCommande().trim());
		if (result<>0) return result; //Si les numeroCommande ne sont pas egaux retourner le resultat de leur comparaison.
		//à ce point du programme c1.getNumeroCommande().trim() et c2.getNumeroCommande().trim() sont egaux
		//Alors les deux objets sont egaux donc retourner 0
		return 0;
	}

05/04/2011, 20h19
tchize_

pourrais tu nous sortir ces trois valeurs pour chaque élément de la liste après tri?
05/04/2011, 23h34
tamildark

Désolé les amis,
Je devais quitter le boulot à 18h,
le code que j'ai mis au dessus fonctionne, je me suis juste tromper sur l'affichage de l'arrayList :aie:

Merci pour ta reflexion LinkinSelim, l'algo que tu propose et celui que j'ai publié font la même chose, la méthodologie de retour du compare qui change.

En tout cas merci pour vos réponse je le marque comme résolu.