Java: Integer est égale à vs ==

De Java 1.5, vous pouvez très bien l'échange Integer avec int dans de nombreuses situations.

Cependant, j'ai trouvé un défaut potentiel dans mon code qui m'a un peu surpris.

Le code suivant:

Integer cdiCt = ...;
Integer cdsCt = ...;
...
if (cdiCt != null && cdsCt != null && cdiCt != cdsCt)
    mismatch = true;

semble être la configuration incorrecte de l'inadéquation lorsque les valeurs sont égales, bien que je ne peux pas déterminer dans quelles circonstances. J'ai mis un point d'arrêt dans Eclipse et vu que le Integer valeurs étaient à la fois 137, et j'ai inspecté l'expression booléenne, et il a dit que c'était faux, mais quand je suis monté sur elle, c'était incohérence de vrai.

Changer le conditionnel pour:

if (cdiCt != null && cdsCt != null && !cdiCt.equals(cdsCt))

résolu le problème.

Quelqu'un peut jeter un peu de lumière sur les raisons de ce qui s'est passé? Jusqu'à présent, j'ai seulement vu le comportement sur mon localhost sur mon propre PC. Dans ce cas particulier, le code bien passé environ 20 comparaisons, mais a échoué sur 2. Le problème a toujours été reproductible.

Si c'est un problème répandu, il devrait être à l'origine des erreurs sur nos autres environnements (dev et test), mais jusqu'à présent, personne n'a signalé le problème après des centaines de tests l'exécution de ce fragment de code.

Encore s'agit-il pas légitime d'utiliser == pour comparer deux Integer valeurs?

En plus de toutes les réponses ci-dessous, la suite de stackoverflow lien a un peu d'informations supplémentaires. Il fait aurait répondu à ma question d'origine, mais parce que je n'ai pas mentionné l'autoboxing dans ma question, il n'apparaît pas dans les suggestions:

Pourquoi ne peut pas le compilateur/JVM juste faire l'autoboxing “fonctionne”?

InformationsquelleAutor Jeremy Goodell | 2010-09-03
  1. 216

    La JVM est la mise en cache des valeurs entières. == ne fonctionne que pour les numéros compris entre -128 et 127
    http://www.owasp.org/index.php/Java_gotchas#Immutable_Objects_.2F_Wrapper_Class_Caching

    • Merci, cela explique certainement pourquoi 137 échoue! Et elle répond à ma question de savoir pourquoi il n'est pas un problème répandu, dans 95% des cas je vais à la rencontre, de la valeur serait en vertu de 127. Bon pour attraper ce que maintenant pour les 5% dans le cas où il ne l'est pas.
    • Note intéressante: jusqu'à il y a quelques semaines, cdiCt et cdsCt étaient tous les deux ints donc c'était bien, mais j'ai dû faire des Entiers afin de vérifier le nul de la situation qui est traitée différemment ...
    • Ouais, c'est assez obscur problème, mais en règle générale, vous utilisez .equals() pour les Objets et == pour les primitives. Vous ne pouvez pas compter sur autounboxing pour l'égalité des tests.
    • Lol, case à cocher pour vous, alors! Ressemble Colin a plus qu'assez de points déjà de toute façon.
    • sur ma machine System.out.println(128L == 128); imprime true. Pourriez-vous donner un exemple qui ne fonctionne pas?
    • Essayez System.out.println(new Integer(128) == new Integer(128));
    • Notez que new Integer(1) != new Integer(1) ainsi. de nouvelles retourne TOUJOURS une nouvelle adresse. L'autoboxing utilise une version mise en cache. D'autres façons de retour Entiers (sans newing eux) probablement renvoyer la valeur mise en cache ainsi.

    InformationsquelleAutor Adam
  2. 70

    Vous ne pouvez pas comparer les deux Integer avec un simple == ils sont les objets de sorte que la plupart du temps, les références ne seront pas les mêmes.

    Il y a un truc, avec Integer entre -128 et 127, les références seront les mêmes que l'autoboxing utilise Integer.valueOf() qui met en mémoire cache les petits entiers.

    Si la valeur de p étant le coffret est vrai, de faux, d'un octet, un char dans la gamme \u0000 à \u007f, ou un int ou un numéro court entre -128 et 127, puis laissez r1 et r2 les résultats de deux de boxe conversions de p. C'est toujours le cas que r1 == r2.

    Ressources :

    Sur le même sujet :

    • C'est la garantie de la JLS ou tout simplement pour l'Oracle de la JVM?
    • La cité de la partie est de la JLS, donc c'est une garantie de l'JLS
    • Re: garantie. Je ne vous fiez pas trop. new Integer(1) == new Integer(1) est encore faux.
    • nouveau ... est toujours false.
    • exactement. Donc, juste prendre l'habitude de ne jamais utiliser ==. Si vous obtenez deux entiers en paramètres à partir de quelque part, vous n'avez aucun moyen de savoir si ils ont été créés à partir du cache ou pas.
    • Vrai, toujours utiliser equals() lorsque vous traitez avec des objets. Ce devrait être l'une des premières choses que l'on doit savoir lors de l'apprentissage de Java. En passant, j'aurais deviné que le constructeur de Integer a été privé, c'est à dire que les instances ont toujours été créés par le biais de la valueOf() méthode. Mais je vois que le constructeur est public.
    • Le constructeur ne pas être privé doit y avoir une surveillance de retour dans la journée. valueOf et la mise en cache a été ajouté plus tard.

    InformationsquelleAutor Colin Hebert
  3. 5

    Le problème est que vos deux Entiers objets ne sont que des objets. Ce n'est pas parce que vous comparez vos deux références d'objet, pas de les valeurs à l'intérieur. Évidemment .equals est remplacé afin de fournir une comparaison de la valeur, par opposition à un objet de référence de comparaison.

    • Bonne réponse, mais ça n'explique pas pourquoi c'est seulement à défaut de 137.
    InformationsquelleAutor MattC
  4. 4

    Integer renvoie à la référence, qui est, lorsque l'on compare les références que vous êtes la comparaison s'ils pointent vers le même objet, pas de valeur. Par conséquent, la question que vous vous voyez. La raison pour laquelle il fonctionne tellement bien avec la plaine int types, c'est qu'il unboxes la valeur contenue par la Integer.

    Peut-je ajouter que si vous êtes en train de faire ce que vous faites, pourquoi les if déclaration pour commencer?

    mismatch = ( cdiCt != null && cdsCt != null && !cdiCt.equals( cdsCt ) );
    InformationsquelleAutor wheaties
  5. 3

    "==" toujours comparer l'emplacement de la mémoire ou des références à des objets de valeurs. méthode equals de toujours comparer les valeurs. Mais équivaut aussi, indirectement, utilise le "==" opérateur de comparer les valeurs.

    Entier Entier utilise le cache pour stocker les valeurs de -128 à +127. Si l'opérateur == est utilisé pour vérifier les valeurs entre -128 à 127 puis il retourne la valeur true. pour d'autres de ces valeurs, elle retourne false .

    Reportez-vous à la lien pour quelques informations supplémentaires

    InformationsquelleAutor vijay