NullPointer, sort et Comparator

Bonjour à tous,

Je suis face à un problème assez incompréhensible, lors d'un appel à

Code:

---

Collection.sort(maList,new Comparator)

---

je reçois un NullPointer ...

Bon jusque là rien d'exceptionnel, mais en mode debug j'ai l'impression que le null pointer ne provient pas de mon code :( mais de de l'algo de tri fait pas la classe collection

Bon un bug à ce niveau là de l'api étant extrêmement hautement improbable, je me dis que ça doit venir de mes comparator ...

Je pense qu'il y a un comparator dans lequel je me retrouve avec une liste a,b,c
et avec les comparators qui donnent :
a > b
b > c
c > a
==> NullPointeur :(


Du coup je me demandais si il y a une "best practice" pour le code des comparator sur le traitement des null ????
Est ce que quelqu'un a déjà eu un cas similaire ??

Voila un exemple de comparator :

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 26

public class TotoComparator implements Comparator<Toto>{   public int compare(Toto toto1, Toto toto2) {   String nom1= toto1.getPseudo(); String nom2= toto2.getPseudo();   if (nom1 != null || nom2 != null){ if (nom1 == null){ return -1; } if (nom2 == null){ return 1; }   else{ int compareToNom = nom1.compareTo(nom2); if (compareToNom != 0){ return compareToNom; } } } }   }

---

Merci
:ccool:

Je ne sais pas s'il y a de "bonnes pratiques" pour l'écriture de compareTo, mais il me semble que ton comparator ne renvoie rien dans certains cas. Si toto1 non null et toto2 non null, mais que leurs 2 pseudos sont les mêmes, ton compareTo ne renvoie rien..... :roll:

effectivement mais c'est du à un mauvais copié collé de ma part ....

il y a un return 0 à la fin de la méthode dsl

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 26 27

public class TotoComparator implements Comparator<Toto>{   public int compare(Toto toto1, Toto toto2) {   String nom1= toto1.getPseudo(); String nom2= toto2.getPseudo();   if (nom1 != null || nom2 != null){ if (nom1 == null){ return -1; } if (nom2 == null){ return 1; }   else{ int compareToNom = nom1.compareTo(nom2); if (compareToNom != 0){ return compareToNom; } } } return 0; }   }

---

Tu n'as pas d'objet à null dans ta liste ?

Non pas d'objet Null :(

Tu n'arrives pas à trouver la ligne qui génère l'erreur ? Même en debug ?

Il y a bien sûr des bonnes pratiques. La première étant de s'assurer qu'aucun élément n'est null, ou de traiter ce cas. Voici un code amélioré ci-dessous. Au moins, il nous permettra de savoir d'où vient précisément l'erreur : si tu vois le texte "L'un des éléments de la liste est null", c'est que tu as malgré tout un élément null dans ta liste.

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20

public int compare(Toto toto1, Toto toto2) { if (toto1 == null || toto2 == null) { throw new NullPointerException("L'un des éléments de la liste est null"); }   String nom1 = toto1.getPseudo(); String nom2 = toto2.getPseudo();   if (nom1 == null) { if (nom2 == null) { return 0; } else { return -1; } } else if (nom2 == null) { return 1; } else { return nom1.compareTo(nom2); } }

---

Bon je pense avoir trouve le soucis

j'ai un comparator pas bien codé comme les autres .... .

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12

public int compare(Toto toto1, Toto toto2) {   String pseudo1 = toto1.getPseudo(); String pseudo2 = toto2.getPseudo();   if (pseudo1 == null){ return 1; }else{ return pseudo1.compareTo(pseudo2); } }

---

alors l'erreur provient quand le pseudo2 est null

Du coup reste les questions :
- best practice sur les comparators (je peux en avoir 4/5 imbriqués)
- pseudo1.compareTo(pseudo2) => null pointer
pseudo1.equals(pseudo2) => pas null pointer

Pour la deuxième je dirais que compareTo sur une primitive et equals sur Objet .... string VS String


Merci

Il faut juste lire la documentation de l'interface Comparator elle détaille très bien ce qu'il faut faire:ccool:
En gros il faut une relation d'ordre pour les objets de ta classe(réviser les cours de mathématique :mrgreen:).
A++

Sinon, les Google-collections permettent de gérer très facilement ce genre de cas, avec la classe Ordering.