Discussion :

[tamildark]

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 :

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

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

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

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

Merci de vos réponse

[LinkinSelim]

à premiere vue, t'a une erreur dans cette ligne tu compare c1 à lui même :

Code :

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

1
2
 
int result=c1.getCodeTypeBudget().trim().compareTo(c1.getCodeTypeBudget().trim());

[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

[LinkinSelim]

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 :

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

[tchize_]

pourrais tu nous sortir ces trois valeurs pour chaque élément de la liste après tri?

[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

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.