一、問題
private static final SimpleDateFormat sdf = new SimpleDateFormat("yyyy-MM-dd HH:mm:ss");
public static Date formatDate(String date) throws ParseException {
return sdf.parse(date);
}
單線程調(diào)用formatDate(String date)，并不會出現(xiàn)任何問題。然而多線程并發(fā)執(zhí)行調(diào)用該方法時，如下：
public class test {
public static final ExecutorService executor = Executors.newFixedThreadPool(5);
public static void main(String[] args) {
for (int i = 0; i < 10000; i++) {
executor.execute(new Runnable() {
@Override
public void run() {
try {
System.out.println(DateUtil.formatDate("2008-08-08 08:08:08"));
} catch (ParseException e) {
e.printStackTrace();
}
}
});
}
}
}
為什么會出現(xiàn)這種情況呢？SimpleDateFormat 類內(nèi)部有一個 Calendar 對象引用，它用來儲存和這個 SimpleDateFormat 相關(guān)的日期信息，例如 sdf.parse(dateStr)/sdf.format(date) 諸如此類的方法參數(shù)傳入的日期相關(guān) String/Date 等等，都是交給 Calendar 引用來儲存的。這樣就會導(dǎo)致一個問題，如果此時 SimpleDateFormat 是個 static 的，在多線程的環(huán)境中，在 calendar 進(jìn)行操作的時候，后面的線程有可能會覆蓋上一個線程設(shè)置好的值，此時便返回一個錯誤的值。因此它并非線程安全。那該如何避免這個問題？
二、解決方案
- 使用 ThreadLocal，每個線程中返回各自的實(shí)例，避免了多線程環(huán)境中共用同一個實(shí)例而導(dǎo)致的問題。
public class test {
private static ThreadLocal<SimpleDateFormat> sdft = new ThreadLocal<>();
public static Date formatDate(String date) throws ParseException {
SimpleDateFormat sdf = getSdfMethod();
return sdf.parse(date);
}
private static SimpleDateFormat getSdfMethod() {
SimpleDateFormat sdf = sdft.get();
if (sdf == null) {
sdf = new SimpleDateFormat("yyyy-mm-dd HH:mm:ss");
sdft.set(sdf);
}
return sdf;
}
}
推薦升級到 JDK8+，使用 LocalDateTime，LocalDate，LocalTime 來代替。
使用第三方包，推薦 Joda-Time，對于日期的增減操作也是相當(dāng)便捷。
最簡單的解決方案?？梢园?static 去掉，這樣每個新的線程都會有一個自己的 sdf 實(shí)例，從而避免線程安全的問題。弊端：在高并發(fā)的情況下會因大量的 new sdf 以及銷毀 sdf，耗費(fèi)大量資源。
