Tri de collections selon 3 critéres

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:

---

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

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 ?

Bonjour,

À partir de java 8:

Code:

---

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

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!

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

Code:

---

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

---

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

Citation:

---

Envoyé par 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.

---

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:

---

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:

---

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

---

A+.