TreeSet est un menteur !

Bonjour à tous, si vous cherchez dans l'api vous trouvez ceci piur la méthode contains de java.util.TreeSet :

Citation:

---

contains

public boolean contains(Object o)

Returns true if this set contains the specified element.

Specified by:
contains in interface Set
Overrides:
contains in class AbstractCollection

Parameters:
o - the object to be checked for containment in this set.
Returns:
true if this set contains the specified element.
Throws:
ClassCastException - if the specified object cannot be compared with the elements currently in the set.

---

En suivant le lien vers Set on trouve ceci :

Citation:

---

contains

public boolean contains(Object o)

Returns true if this set contains the specified element. More formally, returns true if and only if this set contains an element e such that (o==null ? e==null : o.equals(e)).

Specified by:
contains in interface Collection

Parameters:
o - element whose presence in this set is to be tested.
Returns:
true if this set contains the specified element.
Throws:
ClassCastException - if the type of the specified element is incompatible with this set (optional).
NullPointerException - if the specified element is null and this set does not support null elements (optional).

---

Et pourtant en fouillant les sources on tombe sur ceci :
TreeSet#contains(Object o) délègue à la TreeMap#contaisnKey(Object k) qui utilise le résultat de TreeMap#getEntry(Object k) qui utilise TreeMap#compare(Object k1, Object k2) qui fais appel à la méthode Comparable#compareTo(Object o)

donc l'API nous ment. TreeSet utilise compareTo et pas equals.

des réactions ? ou me suis-je gourré ?

Salut,


Dans l'API de TreeSet on peut lire ceci :

Citation:

---

Note that the ordering maintained by a set (whether or not an explicit comparator is provided) must be consistent with equals if it is to correctly implement the Set interface. (See Comparable or Comparator for a precise definition of consistent with equals.) This is so because the Set interface is defined in terms of the equals operation, but a TreeSet instance performs all element comparisons using its compareTo (or compare) method, so two elements that are deemed equal by this method are, from the standpoint of the set, equal. The behavior of a set is well-defined even if its ordering is inconsistent with equals; it just fails to obey the general contract of the Set interface.

---

:arrow: Bref TreeSet utilise compareTo() alors que les Set devraient utiliser equals(), et ainsi pour respecter les spécificité du Set il faut que compareTo() doit être cohérent avec equals() (bref si a.equals(b) alors a.compareTo(b)==0 et inversement).


C'est juste une spécificité d'une implémentation... et l'API de Collections est pleine de spécificité :mrgreen:

a++

Ok, n'empêche ils auraient pu être plus clair dans leur commentaires de méthode. Enfin merci pour l'info, maintenant je ferais plus attention lorsque je coderais un objet Comparable