NullPointer, sort et Comparator

**LittleBean** · 01/10/2009, 18h45

Bonjour à tous,

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

Code :

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

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 :

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

Invité · 02/10/2009, 06h27

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

**LittleBean** · 02/10/2009, 09h55

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 :

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