Discussion :

[Jimalexp]

Salut,

Que se passes t-il si lp est null lorsque la méthode compareTo est utilisée ? Je suppose que cela provoque une erreur. Mais comment reformuler le traitement alors ? En sortant un -1 si lp == null ?

Code :

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

1
2
3
4
5
	public int compareTo( LivrePret lp )
	{
		// On peut ajouter d'autres critères d'identicité comme l'isbn
		return getTitre().compareTo( lp.getTitre() ) ;
	}

[Heimdal]

Code :

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

lp.getTitre()

Si lp est null, t'obtiens un joli nullPointerException.

[Jimalexp]

Exacte mais quelle valeur devrais-je renvoyer si lp est justement null ?

J'ai trouvé cette page assez exhaustive sur le sujet et on y mentionne ceci :
http://www.javapractices.com/topic/TopicAction.do?Id=10

Object : use compareTo. (Note that possibly-null fields present a problem : while x.equals(null) returns false, x.compareTo(null) will always throw a NullPointerException)

Cela sous-entendrais t-il qu'il faudrais systématiquement tester si une référence vers un objet est null avant de faire qu'une compareTo est enclenché (dans un Collections.sort() par exemple) ?

[Heimdal]

A priori, l'attribut d'un objet qui te sert de comparaison ne doit jamais être null.
C'est dans le constructeur de l'objet qu'il est bon de mettre en place ce genre de protection. En reprenant ton exemple du LivrePret et en supposant que deux livres différents ne peuvent avoir le même titre (Rien n'est moins sur cela dit).

Code :

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

1
2
3
4
5
6
7
8
String titre;
 
public LivrePret(String titre) {
if(titre == null) {
throw new IllegalArgumentException("titre ne peut etre null");
}
this.titre = titre;
}

[Coulon Arnaud]

Je pense qu'effectivement, tu ne dois pas avoir à traiter le cas du paramètre à null.

Quand tu appliques les méthodes de tris, celle-ci vont s'appuyer sur la méthode compareTo pour savoir quel objet est plus grand que tel autre.

Ces méthodes prennent un objet de la collection à trier et applique la méthode compareTo à l'autre objet en paramètre.
Dans ton code, tu peux effectivement traiter si le paramètre est à null mais par contre si c'est l'objet d'instance qui sert à appliquer la méthode qui est à null, tu ne peux rien faire.

Donc pour un fonctionnement identique entre le paramètre et l'objet sur lequel est appliqué la méthode, il ne faut pas traiter le cas du null.

Si par contre, c'est toi seulement qui appel la méthode compareTo, tu peux faire toi même tes tests de nullité sur l'objet sur lequel est appliqué la méthode.

Dans le cas où tu veux vraiment faire du tri et que tes objets peuvent être à null tout en considérant que null==null et que null est plus petit que le reste, tu peux peut être travailler avec la classe comparator et la méthode compare.
Cette méthode contient en paramètre les 2 données à tester. Donc le null peut être tester pour chacun d'eux.

Bon courage

Arnaud