在 Java 代码中从 SimpleDateFormat 返回 NumberFormatException

Returning NumberFormatException form SimpleDateFormat in Java code

我已将 SimpleDateFormat 对象声明为常量文件中的静态字段,如下所示,

Constants.java

public static final SimpleDateFormat GENERAL_TZ_FORMATTER = new SimpleDateFormat("yyyy-MM-dd HH:mm:ss z");

在我的 class 文件中我的实现是这样的。

String fDate = getTextValue(empNo, "firstDate");
if (null != fDate && !fDate.isEmpty()) {
    try {
        Date date = (Date)(Constants.GENERAL_TZ_FORMATTER).parse(fDate);
        issue.setDate(date.getTime());
    } catch (ParseException e) {
        logUtil.error(LOG, e+ "date : " + date);
    }
}

错误:

Exception while importing data. package name.SecureException: For input string: ""
at java.lang.NumberFormatException.forInputString(NumberFormatException.java:65)

我的问题是,在某些情况下,这会引发 NumberFormatException(一种非常罕见的情况),所以我一直想知道这个问题并做了诊断,其中大部分都在解释这种情况可能是因为 SimpleDateFormat 不是线程安全的。如果是这种情况,我不清楚这段代码 运行 在不使用多线程的多线程环境中如何使 DateFormat.parse() 的输入成为空字符串?

Java's SimpleDateFormat is not thread-safe article

我已经尝试解决这个问题,但是重现这个问题真的很难,我想知道你对此的想法,这将帮助我找到更好的解决方案。非常感谢您的建议。 谢谢。

好吧,正如您 post 下面的评论已经提到的,无论如何您都不应该使用 SimpleDateFormat

您可能偶然发现 SimpleDateFormat 这种情况很麻烦。然而,这并不是唯一的原因。 This Whosebug post explains why 太麻烦了。 post 提到 SimpleDateFormat 不是 thread-safe 的原因之一。 Thread-safety是当多个进程作用于formatter时,即利用formatter格式化日期,不会因为干扰而出现不正确、不准确或未定义的结果。

Your link to the article on Callicoder很好的解释了为什么SimpleDateFormat惹麻烦。 post 提到了与您得到的相同的异常:

java.lang.NumberFormatException: For input string: ""

简而言之,线程在使用格式化程序时会发生干扰,因为格式化程序未同步。这意味着 SimpleDateFormat class 不会强制一个线程必须等到其他线程完成修改其内部状态。使用的三个 classes 是 SimpleDateFormat, DateFormat and FieldPosition.

Here's erroneous code in action.

使用java.time

您需要移动到更新的 Java 8 日期和时间 API,在 java.time 包中可用。它们绝对是 thread-safe,因为它们具有不变的性质。在你的情况下,使用 java.time.format.DateTimeFormatter:

public static final DateTimeFormatter GENERAL_TZ_FORMATTER = DateTimeFormatter.ofPattern("yyyy-MM-dd HH:mm:ss z");
ZonedDateTime zdt = ZonedDateTime.parse(fDate, GENERAL_TZ_FORMATTER);
Instant instant = zdt.toInstant();

// Your setDate should really accept an Instant:
issue.setDate(instant);
// If that's REALLY not possible, then you can convert it to an integer
// value equal to the number of milliseconds since 1 January 1970, midnight
//issue.setDate(instant.toEpochMilli());