77

我正在使用 Joda 解析包含日期/时间的第三方日志文件。日期/时间是两种不同格式之一,具体取决于我正在解析的日志文件的年龄。

目前我有这样的代码:

try {
    return DateTimeFormat.forPattern("yyyy/MM/dd HH:mm:ss").parseDateTime(datePart);
} catch (IllegalArgumentException e) {
    return DateTimeFormat.forPattern("E, MMM dd, yyyy HH:mm").parseDateTime(datePart);
}

这可行,但违反了来自 Effective Java 2nd Edition 的 Joshua Bloch 的建议(第 57 条:仅在异常情况下使用异常)。这也使得很难确定 IllegalArgumentException 是否由于日志文件中的错误日期/时间而发生。

您能否建议一种不会滥用异常的更好方法?

4

3 回答 3

147

您可以使用DateTimeFormatterBuilder.append方法创建多个解析器并将它们添加到构建器:

DateTimeParser[] parsers = { 
        DateTimeFormat.forPattern( "yyyy-MM-dd HH" ).getParser(),
        DateTimeFormat.forPattern( "yyyy-MM-dd" ).getParser() };
DateTimeFormatter formatter = new DateTimeFormatterBuilder().append( null, parsers ).toFormatter();

DateTime date1 = formatter.parseDateTime( "2010-01-01" );
DateTime date2 = formatter.parseDateTime( "2010-01-01 01" );
于 2010-12-02T00:15:10.803 回答
17

Joda-Time 通过允许指定多个解析器来支持这一点 - DateTimeFormatterBuilder#append

只需使用构建器创建两个格式化程序并调用toParser()每个格式化程序。然后使用 builder 将它们组合在一起append

于 2010-07-23T08:52:10.400 回答
7

不幸的是,我不相信 Joda Time 有任何这样的能力。有一个“tryParseDateTime”方法会很好,但它不存在。

我建议你将这种行为隔离到你自己的类中(一个需要一系列模式,并依次尝试每个模式),这样丑陋就只在一个地方。如果这会导致性能问题,您可能需要尝试使用一些启发式方法来猜测首先尝试哪种格式。例如,在您的情况下,如果字符串以数字开头,那么它可能是第一个模式。

请注意,DateTimeFormatterJoda Time 中的 s 通常是不可变的 - 您不应该在每次要解析一行时都创建一个新的。创建一次并重复使用它们。

于 2010-07-22T09:22:37.177 回答