DateFormat不是线程安全的,这意味着它们维护状态的内部表示。 如果多个线程同时访问同一实例,则在静态上下文中使用它们会产生一些非常奇怪的错误。
我的建议是使变量在使用变量的位置本地化,而不是使它们成为类的静态属性。 初始化类时似乎正在执行此操作,因此可以在构造函数中执行此操作:
public class MyClass {
private String fileName;
public MyClass() {
final Date today = Calendar.getInstance().getTime();
final DateFormat yymmdd = new SimpleDateFormat("yyMMdd");
this.fileName = "file_" + yymmdd.format(TODAY);
}
...
}
并且,如果您需要在多个地方使用格式化程序,则可以只制作模式static final,并在需要时创建一个新的本地DateFormat:
public class MyClass {
private static final String FILENAME_DATE_PATTERN = "yyMMdd";
public void myMethod() {
final DateFormat format = new SimpleDateFormat(FILENAME_DATE_PATTERN);
// do some formatting
}
}
该问题的FindBugs文档说:
正如JavaDoc所述,DateFormats是 本质上对多线程不安全 用。 检测器找到了一个呼叫 DateFormat的实例具有 是通过静态场获得的。 这个 看起来可疑。
有关更多信息,请参见Sun。 Bug#6231579和Sun Bug#6178997。
而且DateFormat的javadoc建议:
日期格式不同步。 它 建议创建单独的 格式化每个线程的实例。 如果 多线程访问格式 同时,必须同步 在外部。
Jack Leow的答案也很好地说明了您静态使用“ TODAY”的语义。
顺便说一句,我实际上已经在高流量的生产环境中看到了这种情况,一开始调试起来是一件非常令人困惑的事情。 因此,根据我的经验,FindBugs警告实际上是一个有用的建议(与其他一些静态分析规则不同,有时似乎很挑剔)。