Les mauvaises pratiques de Classe définit compareTo(...) et utilise l'Objet.equals()

Se demander ce qui doit être fait pour répertoriés méthode

 public final int compareTo(final FieldDTO o) {
        return o.available.compareTo(this.available);

son lancement exception à la ligne 2, en indiquant
Les mauvaises pratiques de Classe définit compareTo(...) et utilise l'Objet.equals() 16 jours
champ définit compareTo(FieldDTO) et utilise l'Objet.equals()

Pas sûr de savoir comment dois-je gérer cela.
Merci à l'avance.

Il est vraiment lancer une Exception? Ou est-ce juste un avertissement (peut-être par findBugs)?
FieldDTO semble être une classe interne de votre projet, poser cette question qui l'a créé.

OriginalL'auteur Maulzey | 2013-05-22

  1. 9

    Si vous définissez compareTo vous devriez au moins définir equals

    boolean equals(it) { 
  return compareTo(it) == 0; 
}

    sinon vous aurez des problèmes étranges lorsque vous mettez votre objet en Maps et Sets. Il est généralement une bonne pratique également définir hashCode.

    OriginalL'auteur OldCurmudgeon

  2. 5

    C'est la documentation de FindBugs:

    Eq: Classe définit compareTo(...) et utilise l'Objet.equals()
    (EQ_COMPARETO_USE_OBJECT_EQUALS)

    Cette classe définit un compareTo(...) la méthode, mais hérite de sa equals()
    méthode de java.lang.Objet. En général, la valeur de compareTo devrait
    de retour à zéro si et seulement si est égal renvoie la valeur true. Si ce n'est plus respectée,
    bizarre et imprévisible d'échecs dans des classes telles que
    PriorityQueue. Dans Java 5 le PriorityQueue.supprimer la méthode utilise l'
    méthode compareTo, tandis que dans la version 6 de Java, il utilise la méthode equals.

    De la JavaDoc de la méthode compareTo dans l'interface Comparable:

    Il est fortement recommandé, mais n'est pas strictement nécessaire que
    (x.compareTo(y)==0) == (x.equals(y)). De manière générale, toute la classe
    qui implémente l'interface Comparable et qu'il contrevient à cette condition
    doit indiquer clairement ce fait. Le texte est "Remarque:
    cette classe a un ordre naturel qui est incompatible avec égal à égal."

    Il semble donc que vous avez besoin pour mettre en œuvre les equals méthode de ce qui annule le défaut de mise en œuvre de Object.

    OriginalL'auteur Ziyao Wei

  3. 4

    vous avez besoin pour remplacer la classe d'Objet equals() et hashCode() méthodes.
    L'utilisation de l'IDE code généré pour ces derniers, il va tirer tous les attributs d'un Objet et crée méthode pour vous.

    Sur eclipse IDE:

    1. Clic droit sur la classe
    2. Sélectionner La Source
    3. Générer hashCode() et equals()...

    OriginalL'auteur Raj Pandiri

  4. 0

    Votre classe, comme il semble, met en œuvre la Comparable interface.

    Javadoc sont strictes: si vous utilisez votre propre compareTo() méthode, elle doit retourner une 0 uniquement si le equals() méthode renvoie vrai.

    Donc:

    (x.compareTo==0) == (x.equals).
    Mais la JavaDoc, il "n'est pas strictement nécessaire".
    Oui, c'est pourquoi je upvoted le premier commentaire, je pense que ça devrait être un avertissement à la place.

    OriginalL'auteur aran