Discussion :

[rmed92]

Bonjour,

j'ai une ArrayList de produits que je veux trier selon le prix puis la désignation et enfin l'identifiant, si le prix est égal je trie par désignation et si la désignation est égale je trie par id, mais mon code n'affiche pas le résultat attendu, ma classe Produit implémente Comparable,

voici mon code :

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
  // Comparator pour le tri des produits par prix 
 
     public static Comparator<Produit> ComparatorPrix = new Comparator<Produit>() {
 
        @Override
 
        public int compare(Produit p1, Produit p2) {
 
               if(p1.getPrix()!=(p2.getPrix())){return (int)(p1.getPrix()-(p2.getPrix()));
 
               }else if((p1.getDesignation().compareTo(p2.getDesignation())!=0)) {
 
           return (int)(p1.getDesignation().compareTo(p2.getDesignation()));
 
            }else           return (int)(p1.getIdentifiant() - p2.getIdentifiant());
 
        }};

MERCI d'avance pour votre aide

[thelvin]

Et c'est quoi le résultat attendu et c'est quoi le résultat obtenu et comment tu as fait pour croire que tu avais pas besoin de le dire ?

[andry.aime]

Bonjour,

À partir de java 8:

Code :

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

1
2
3
Comparator<Produit> comparatorPrix = Comparator.comparing(Produit::getPrix) // 1- Par le prix
				.thenComparing(Produit::getDesignation) // 2- Par la désignation 
				.thenComparing(Produit::getIdentifiant); // 3- Par l'identifiant

A+.

[boijea]

Hello,

Ta méthode compare() est beaucoup trop dense et presque illisible.
Commences par y mettre des espaces entre les if et (, des sauts de ligne, et des alignements verticaux sur les {}.
Si je voulais être pas trop gentil: mets donc tout cela sur une seule ligne!

Quand tu es prêt, va dans ton outils de développement, par exemple Eclipse, et tu fais du pas à pas (débogage) sur chaque if et else, avec différents objets p1 et p2 correspondant à toutes les combinaisons possibles.
Tu trouveras alors la ou les fautes, et tu seras assuré que ta méthode compare() fonctionne.
Je l'ai déjà expliqué ici ou dans le groupe C++, il faut écrire des classes de tests, et ici aussi pour ce Comparator!

[andry.aime]

Pour l’implémentation de la méthode compareTo de Comparable:

Code :

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

1
2
3
4
5
6
7
@Override
public int compareTo(Comparator o){
return Comparator.comparing(Produit::getPrix) // 1- Par le prix
		.thenComparing(Produit::getDesignation) // 2- Par la désignation 
		.thenComparing(Produit::getIdentifiant) // 3- Par l'identifiant
		.compare(this, o); 
}

[thelvin]

Tout ça c'est bien joli, mais 'faudrait déjà qu'on sache précisément pourquoi son propre essai lui a pas plu.

[andry.aime]

Tout ça c'est bien joli, mais 'faudrait déjà qu'on sache précisément pourquoi son propre essai lui a pas plu.

Je soupçonne que getPrix retourne un type objet (Double / Float...), donc p1.getPrix()!=(p2.getPrix()) sera toujours évalué à true. La classe Double/Float implémente déjà Comparable. Inutile aussi de caster en int la valeur retourner de compareTo de désignation puisque c'est déjà de type int. Si identifiant est de type Integer ou Long, il implémente aussi Comparable.

Code :

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

1
2
3
4
5
6
7
if(p1.getPrix().compareTo(p2.getPrix())!=0){
    	 return p1.getPrix().compareTo(p2.getPrix());
     }else if((p1.getDesignation().compareTo(p2.getDesignation())!=0)){
    	 return (p1.getDesignation().compareTo(p2.getDesignation()));
     }else{
    	 return (p1.getIdentifiant().compareTo(p2.getIdentifiant()));
     }

Il faut aussi vérifier les null pour éviter un NullPointerException, pour mes premiers exemples:

Code :

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

Comparator.comparing(Produit::getPrix, Comparator.nullsFirst(Comparator.naturalOrder())) // voir aussi les autres méthode de Comparator (comparingDouble, ...)

A+.