Безпечне лиття довгих до int в Java

Що найбільш ідіоматичних спосіб в Java , щоб переконатися , що виливок з longдо intне втрачає яку - небудь інформацію?

Це моя поточна реалізація:

public static int safeLongToInt(long l) {
    int i = (int)l;
    if ((long)i != l) {
        throw new IllegalArgumentException(l + " cannot be cast to int without changing its value.");
    }
    return i;
}

Додано новий метод з Java 8, щоб зробити саме це.

import static java.lang.Math.toIntExact;

long foo = 10L;
int bar = toIntExact(foo);

Викине ArithmeticExceptionв разі переливу.

Подивитися: Math.toIntExact(long)

До Java 8. додано кілька інших безпечних методів переповнення. Вони закінчуються точно .

Приклади:

• Math.incrementExact(long)
• Math.subtractExact(long, long)
• Math.decrementExact(long)
• Math.negateExact(long),
• Math.subtractExact(int, int)

Я думаю, що зробив би це так само просто:

public static int safeLongToInt(long l) {
    if (l < Integer.MIN_VALUE || l > Integer.MAX_VALUE) {
        throw new IllegalArgumentException
            (l + " cannot be cast to int without changing its value.");
    }
    return (int) l;
}

Я думаю, що це виражає наміри чіткіше, ніж повторне кастинг ... але це дещо суб'єктивно.

Зверніть увагу на потенційний інтерес - у C # це було б просто:

return checked ((int) l);

З класом Ints Google Guava ваш метод можна змінити на:

public static int safeLongToInt(long l) {
    return Ints.checkedCast(l);
}

Із пов’язаних документів:

зареєстрованоКаст

public static int checkedCast(long value)

Повертає значення int, яке дорівнює value , якщо це можливо.

Параметри: value - будь-яке значення в діапазоніint типу

Повертає:int значення, відповіднеvalue

Кидки: IllegalArgumentException - якщо valueбільше Integer.MAX_VALUEабо меншеInteger.MIN_VALUE

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

З BigDecimal:

long aLong = ...;
int anInt = new BigDecimal(aLong).intValueExact(); // throws ArithmeticException
                                                   // if outside bounds

ось рішення, якщо ви не піклуєтесь про значення, якщо воно більше, ніж потрібно;)

public static int safeLongToInt(long l) {
    return (int) Math.max(Math.min(Integer.MAX_VALUE, l), Integer.MIN_VALUE);
}

ДОНТ: Це не рішення!

Мій перший підхід:

public int longToInt(long theLongOne) {
  return Long.valueOf(theLongOne).intValue();
}

Але це просто відкидає довгий час до int, потенційно створюючи нові Longекземпляри або витягаючи їх з пулу Long.

Недоліки

1. Long.valueOfстворює новий Longекземпляр, якщо число не знаходиться в межах Longдіапазону пулів [-128, 127].

2. intValueРеалізація нічого не робить більше , ніж:

   return (int)value;

Тож це можна вважати навіть гірше, ніж просто кастинг longдо int.

Я стверджую, що очевидним способом побачити, чи змінив значення зміна значення, було б передача та перевірка результату. Я б, однак, усунув би непотрібний склад при порівнянні. Я також не дуже захоплююсь назвами змінних однієї літери (виняток xі y, але не тоді, коли вони означають рядок і стовпець (іноді відповідно)).

public static int intValue(long value) {
    int valueInt = (int)value;
    if (valueInt != value) {
        throw new IllegalArgumentException(
            "The long value "+value+" is not within range of the int type"
        );
    }
    return valueInt;
}

Однак дійсно я хотів би уникнути цього перетворення, якщо це можливо. Очевидно, що іноді це неможливо, але в тих випадках IllegalArgumentExceptionмайже напевно неправильним винятком є ​​кидання, що стосується клієнтського коду.

Цілі типи Java представлені як підписані. При введенні між 2 ³¹ та 2 ³² (або -2 ³¹ та -2 ³² ) амплуа буде успішною, але ваш тест не зможе.

На що потрібно перевірити, чи всі високі біти з longних однакові:

public static final long LONG_HIGH_BITS = 0xFFFFFFFF80000000L;
public static int safeLongToInt(long l) {
    if ((l & LONG_HIGH_BITS) == 0 || (l & LONG_HIGH_BITS) == LONG_HIGH_BITS) {
        return (int) l;
    } else {
        throw new IllegalArgumentException("...");
    }
}

(int) (longType + 0)

але Лонг не може перевищувати максимум :)

Ще одним рішенням може бути:

public int longToInt(Long longVariable)
{
    try { 
            return Integer.valueOf(longVariable.toString()); 
        } catch(IllegalArgumentException e) { 
               Log.e(e.printstackstrace()); 
        }
}

Я спробував це для тих випадків, коли клієнт виконує POST, а серверна БД розуміє лише цілі особи, тоді як у клієнта є Long.