Добавьте 1 Неделю к Дате, какой путь предпочтен?

Question

Добавьте 1 Неделю к Дате, какой путь предпочтен?

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

Первым был служебный метод:

public static Date addDaysToDate(final Date date, int noOfDays) {
    Date newDate = new Date(date.getTime());

    GregorianCalendar calendar = new GregorianCalendar();
    calendar.setTime(newDate);
    calendar.add(Calendar.DATE, noOfDays);
    newDate.setTime(calendar.getTime().getTime());

    return newDate;
}

И вторая используемая простая арифметика миллисекунды:

long theFuture = System.currentTimeMillis() + (86400 * 7 * 1000);
Date nextWeek = new Date(theFuture);

Второй метод, очевидно, использует 'магические числа' для определения недели, но это могло быть перемещено в константу MILLISECONDS_IN_ONE_WEEK = 86400 * 7 * 1000 Таким образом кроме этого, там какие-либо причины, почему один из этих методов должен быть предпочтен по другому?

В основном я хочу изменить код, чтобы быть последовательным повсюду, но я не совсем уверен который удалить. Таким образом, любые аргументы так или иначе были бы полезны.

25

java datetime date calendar

задан DaveJohnston 21 July 2010 в 14:29

8 ответов

Будьте очень осторожны при использовании второго метода, который поймал меня на днях. Рассмотрим

private static long ONE_YEAR_AS_MILLISECONDS = 365*24*60*60*1000;

Это выглядит достаточно невинно, но на самом деле не даст ожидаемого результата, поскольку при умножении используются целые числа, которые при умножении друг на друга целых чисел вызывают числовое переполнение и приводят к неожиданному результату. Это связано с тем, что максимальное значение int в Java составляет 2 147 483 647, а в году 31 536 000 000 мс. На моей машине приведенный выше код дает 1 471 228 928, что явно неверно.

Вместо этого вам нужно сделать следующее:

private static long ONE_YEAR_AS_MILLISECONDS = 365L*24L*60L*60L*1000L;

3