Java: Ціле число дорівнює = ==

Що стосується Java 1.5, ви можете в значній мірі взаємодіяти Integerз intбагатьма ситуаціями.

Однак я знайшов потенційний дефект у своєму коді, який мене трохи здивував.

Наступний код:

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

виявилося невірно встановлення невідповідності, коли значення були рівні, хоча я не можу визначити, за яких обставин. Я встановив точку розриву в Eclipse і побачив, що Integerзначення є обома 137, і я перевірив булевий вираз, і він сказав, що це помилково, але коли я переступив це, він встановлював невідповідність істинному.

Зміна умовного на:

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

виправили проблему.

Чи може хтось пролити світло на те, чому це сталося? Поки що я бачив лише поведінку свого локального господаря на своєму ПК. У цьому конкретному випадку код успішно пройшов приблизно 20 порівнянь, але не вдався до 2. Проблема була постійно відтворюваною.

Якщо це поширена проблема, вона повинна викликати помилки в інших наших середовищах (розробник і тест), але поки що ніхто не повідомив про проблему після сотень тестів, що виконують цей фрагмент коду.

Чи все ж не є законним використовувати ==для порівняння двох Integerзначень?

На додаток до всіх тонких відповідей нижче, наступне посилання stackoverflow містить ще трохи додаткової інформації. Насправді він би відповів на моє первісне запитання, але оскільки я не згадав про автобоксинг у своєму питанні, він не відображався у вибраних пропозиціях:

Чому компілятор / JVM не може зробити автобоксинг «просто діючим»?

JVM кешує цілісні значення. == працює лише для чисел від -128 до 127 http://www.owasp.org/index.php/Java_gotchas#Immutable_Objects_.2F_Wrapper_Class_Caching

Ви не можете порівнювати два Integerз простими ==вони об'єктами, так що більшість часу посилання не будуть однаковими.

Існує хитрість, коли Integerвід -128 до 127 посилання будуть такими ж, як і для автоматичного використання, Integer.valueOf()який кешує невеликі цілі числа.

Якщо значення p, яке міститься у вікні, є істинним, хибним, байтом, знаком в діапазоні \ u0000 до \ u007f або int або коротким числом від -128 до 127, то нехай r1 і r2 є результатами будь-яких двох перетворень боксу з р. Це завжди так, що r1 == r2.

Ресурси:

• JLS - Бокс

На цю ж тему:

• автобоксинг проти Java-ручного боксу

Проблема полягає в тому, що ваші два об'єкти Integer - це саме це, об'єкти. Вони не відповідають тому, що ви порівнюєте дві ваші посилання на об'єкт, а не значення в межах. Очевидно, .equalsце перевизначається для забезпечення порівняння значень на відміну від порівняння посилань на об'єкт.

Integerпосилається на посилання, тобто при порівнянні посилань, які ви порівнюєте, якщо вони вказують на один і той же об'єкт, а не значення. Отже, проблема, яку ви бачите. Причина, з якою вона так добре працює з простими intтипами, полягає в тому, що вона розблоковує значення, що міститься в Integer.

Чи можу додати, що якщо ви робите те, що ви робите, з чого ifпочинати заяву?

mismatch = ( cdiCt != null && cdsCt != null && !cdiCt.equals( cdsCt ) );

"==" завжди порівнюють значення пам'яті або посилання на об'єкти значень. метод рівних завжди порівнюють значення. Але рівне також опосередковано використовує оператор "==" для порівняння значень.

Integer використовує кеш Integer для зберігання значень від -128 до +127. Якщо оператор == використовується для перевірки значень від -128 до 127, то він повертає значення true. для інших, крім цих значень, воно повертається помилковим.

Перейдіть за посиланням для отримання додаткової інформації

Для правильності використання ==ви можете просто розв’язати одне зі порівняних Integerзначень перед ==порівнянням, наприклад:

if ( firstInteger.intValue() == secondInteger ) {..

Другий буде автоматично нерозподіленим (звичайно, ви повинні перевірити це на nullперший).

Окрім цих наведених чудових відповідей, я дізнався, що:

НІКОЛИ не порівнюйте об’єкти з ==, якщо ви не збираєтесь порівнювати їх за посиланнями.