StackOverflowError с использованием joda-time new Period (long) - PullRequest
2 голосов
/ 11 сентября 2010

Во-первых, извините, это так долго.Мне, вероятно, не нужен весь код, но я хотел быть уверен.

Во-вторых, мой реальный вопрос: я что-то не так делаю или это ошибка в библиотеке joda-time?

Я пытаюсь использовать joda-time (1.6.1) для вычисления, а затем форматировать длительности.

В настоящее время я использую Period, что может быть неправильным выбором.Пожалуйста, дайте мне знать, если это так.Однако, даже если это неправильный выбор, я уверен, что этого не произойдет.

Я инициализирую Period, используя миллисекунды (умножая длительность в секундах на 1000).Я использую Period, поэтому я могу отформатировать его и распечатать:

long durationLong = durationSec * 1000;
Period duration = new Period(durationLong);

PeriodFormatter daysHoursMinutes = new PeriodFormatterBuilder()
    .appendHours()
    .appendSeparator(":")
    .appendMinutes()
    .appendSeparator(":")
    .appendSeconds()
    .toFormatter();

String formattedString = daysHoursMinutes.print(callDuration.normalizedStandard());

Я получаю исключение ниже и просмотрел источник, чтобы подтвердить цикл.

Caused by: java.lang.StackOverflowError
    at java.util.Hashtable.get(Hashtable.java:274)
    at java.util.Properties.getProperty(Properties.java:177)
    at java.lang.System.getProperty(System.java:440)
    at java.lang.System.getProperty(System.java:412)
    at org.joda.time.DateTimeZone.getDefault(DateTimeZone.java:132)
    at org.joda.time.DateTimeZone.forID(DateTimeZone.java:190)
    at org.joda.time.DateTimeZone.getDefault(DateTimeZone.java:132)
    at org.joda.time.DateTimeZone.forID(DateTimeZone.java:190)

...snip (all the same)...

    at org.joda.time.DateTimeZone.getDefault(DateTimeZone.java:132)
    at org.joda.time.DateTimeZone.forID(DateTimeZone.java:190)
    at org.joda.time.DateTimeZone.getDefault(DateTimeZone.java:132)
    at org.joda.time.DateTimeZone.forID(Dat

Период (длинный):

public Period(long duration) {
    super(duration, null, null);
}

супер (длинный, PeriodType, хронология):

protected BasePeriod(long duration, PeriodType type, Chronology chrono) {
    super();
    type = checkPeriodType(type);
    chrono = DateTimeUtils.getChronology(chrono);
    iType = type;
    iValues = chrono.get(this, duration);
}

DateTimeUtils.getChronology (хроно):

public static final Chronology getChronology(Chronology chrono) {
    if (chrono == null) {
        return ISOChronology.getInstance();
    }
    return chrono;
}

ISOChronology.getInstance ():

public static ISOChronology getInstance() {
    return getInstance(DateTimeZone.getDefault());
}

DateTimeZone.getDefault ():

public static DateTimeZone getDefault() {
    DateTimeZone zone = cDefault;
    if (zone == null) {
        synchronized(DateTimeZone.class) {
            zone = cDefault;
            if (zone == null) {
                DateTimeZone temp = null;
                try {
                    try {
                        temp = forID(System.getProperty("user.timezone"));
                    } catch (RuntimeException ex) {
                        // ignored
                    }
                    if (temp == null) {
                        temp = forTimeZone(TimeZone.getDefault());
                    }
                } catch (IllegalArgumentException ex) {
                    // ignored
                }
                if (temp == null) {
                    temp = UTC;
                }
                cDefault = zone = temp;
            }
        }
    }
    return zone;
}

forID (String) вызывает getDefault (), который создает цикл:

 public static DateTimeZone forID(String id) {
    if (id == null) {
        return getDefault();
    }
    if (id.equals("UTC")) {
        return DateTimeZone.UTC;
    }
    DateTimeZone zone = cProvider.getZone(id);
    if (zone != null) {
        return zone;
    }
    if (id.startsWith("+") || id.startsWith("-")) {
        int offset = parseOffset(id);
        if (offset == 0L) {
            return DateTimeZone.UTC;
        } else {
            id = printOffset(offset);
            return fixedOffsetZone(id, offset);
        }
    }
    throw new IllegalArgumentException("The datetime zone id is not recognised: " + id);
}

Ответы [ 2 ]

5 голосов
/ 11 сентября 2010

Поскольку часть зацикливания есть только в коде joda, я бы сказал, что это ошибка.


Она была исправлена ​​в trunk и будет доступна в V2.0.


Ресурсы:

3 голосов
/ 11 сентября 2010

Похоже, что это ошибка в том смысле, что предполагается, что свойство user.timezone установлено.

Это способ обойти это в этом случае - просто убедитесь, что user.timezone установлен правильно. Обидно, что ты должен все же.

Joda Time использует "null означает default" во многих местах - к сожалению, на мой взгляд. Я предпочитаю "null is invalid" обычно. В Noda Time (порт Joda Time для .NET) мы пытаемся избавиться от многих подобных вещей, а также предотвращаем распространение часового пояса по умолчанию в первом место.

...