штучного значения распакованного затем reboxed

Question

FindBugs дает мне предупреждение о следующей строке, где invoiceNumber является Integer объектом:

text.append(String.format("%010d-", (invoiceNumber == null) ? 0 : invoiceNumber)); 

Предупреждения: «штучное значение распакованное, а затем сразу же reboxed»

Теперь я думаю, что понимаю (не) бокс, но я не вижу, как вы будете делать то же самое, не получая предупреждение?

Я обнаружил, что я могу избавиться от предупреждения, используя следующий код вместо этого, но это, кажется, более многословно:

int invNo = (invoiceNumber == null) ? 0 : invoiceNumber; 
text.append(String.format("%010d-", invNo)); 

Может кто-то показать мне, что такое «правильный» способ сделать выше?

BTW, я рассмотрел связанные вопросы, и я понимаю, что с ними происходит, но это, похоже, не соответствует ни одному из них.

Answer 1

Тип выражения (invoiceNumber == null) ? 0 : invoiceNumber): int. Он требует распаковки invoiceNumber в случае invoiceNumber не null.

С другой стороны, String.format ожидает один аргумент String следуют Object ссылки, а это значит, ваш int получить немедленно снова боксировал в Integer.

Вы можете попытаться избежать первоначального распаковки с помощью (invoiceNumber == null) ? Integer.valueOf(0) : invoiceNumber), что сделало бы это выражение возвратом Integer.

Answer 2

Попробуйте изменить (invoiceNumber == null) ? 0 : invoiceNumber на (invoiceNumber == null) ? 0 : invoiceNumber.intValue(). Я думаю, что предупреждение приходит от вас, используя Integer снова в «ложном» случае, а не int.