SimpleDateFormat线程安全问题排查

夕阳也是醉了

发布于 2023-10-16 09:23:31

2280

发布于 2023-10-16 09:23:31

文章被收录于专栏：夕阳醉了

一. 问题现象

运营部门反馈使用小程序配置的拉新现金红包活动二维码，在扫码后跳转至404页面。

二. 原因排查

首先，检查扫码后的跳转链接地址不是对应二维码的实际URL，根据代码逻辑推测，可能是accessToken在微信端已失效导致，检查数据发现，数据库存储的accessToken过期时间为2022-11-29（排查问题当日为2022-10-08），发现过期时间太长，导致accessToken未刷新导致。
接下来，继续排查造成这一问题的真正原因。排查日志发现更新sql语句对应的的过期时间与数据库记录的一致，推测赋值代码存在问题，如下。

tokenInfo.setExpireTime(simpleDateFormat.parse(token.getString("expireTime")));

其中，simpleDateFormat在代码中定义是该类的成员变量。

跟踪代码后发现源码中有明确说明SimpleDateFormat不应该应用于多线程场景下。

Synchronization
//SimpleDateFormat中的日期格式化不是同步的。
Date formats are not synchronized. 
//建议为每个线程创建独立的格式实例。
It is recommended to create separate format instances for each thread. 
//如果多个线程同时访问一个格式，则它必须保持外部同步。
If multiple threads access a format concurrently, it must be synchronized externally.

至此，基本可以判断是simpleDateFormat.parse在多线程情况下造成错误的过期时间入库，导致accesstoken无法正常更新。

三. 原因分析

接下来写个测试类来模拟：

@RunWith(SpringRunner.class)
@SpringBootTest
public class SimpleDateFormatTest {

    SimpleDateFormat simpleDateFormat = new SimpleDateFormat("yyyy-MM-dd HH:mm:ss");

    /**
     * 定义线程池
     **/
    private static final ExecutorService threadPool = new ThreadPoolExecutor(16,
            20,
            0L,
            TimeUnit.MILLISECONDS,
            new LinkedBlockingDeque<>(1024),
            new ThreadFactoryBuilder().setNamePrefix("[线程]").build(),
            new ThreadPoolExecutor.AbortPolicy()
    );

    @SneakyThrows
    @Test
    public void testParse() {
        Set<String> results = Collections.synchronizedSet(new HashSet<>());
        // 每个线程都对相同字符串执行“parse日期字符串”的操作，当THREAD_NUMBERS个线程执行完毕后，应该有且仅有一个相同的结果才是正确的
        String initialDateStr = "2022-10-08 18:30:01";
        for (int i = 0; i < 20; i++) {
            threadPool.execute(() -> {
                Date parse = null;
                try {
                    parse = simpleDateFormat.parse(initialDateStr);
                } catch (ParseException e) {
                    e.printStackTrace();
                }
                System.out.println(Thread.currentThread().getName() + "---" + parse);
            });
        }
        threadPool.shutdown();
        threadPool.awaitTermination(1, TimeUnit.HOURS);
    }
}

运行结果如下：

[线程]5---Sat Jan 08 18:30:01 CST 2000
[线程]0---Wed Oct 08 18:30:01 CST 2200
[线程]4---Sat Oct 08 18:30:01 CST 2022
Exception in thread "[线程]3" java.lang.NumberFormatException: multiple points
	at sun.misc.FloatingDecimal.readJavaFormatString(FloatingDecimal.java:1890)
	at sun.misc.FloatingDecimal.parseDouble(FloatingDecimal.java:110)
	at java.lang.Double.parseDouble(Double.java:538)
	at java.text.DigitList.getDouble(DigitList.java:169)
	at java.text.DecimalFormat.parse(DecimalFormat.java:2089)
	at java.text.SimpleDateFormat.subParse(SimpleDateFormat.java:1869)
	at java.text.SimpleDateFormat.parse(SimpleDateFormat.java:1514)
	at java.text.DateFormat.parse(DateFormat.java:364)
	at com.SimpleDateFormatTest.lambda$testParse$0(SimpleDateFormatTest.java:49)
	at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1149)
	at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:624)
	at java.lang.Thread.run(Thread.java:748)
[线程]6---Sat Oct 08 18:30:01 CST 2022
[线程]11---Wed Mar 15 18:30:01 CST 2045
Exception in thread "[线程]2" java.lang.ArrayIndexOutOfBoundsException: 275
	at sun.util.calendar.BaseCalendar.getCalendarDateFromFixedDate(BaseCalendar.java:453)
	at java.util.GregorianCalendar.computeFields(GregorianCalendar.java:2397)
	at java.util.GregorianCalendar.computeTime(GregorianCalendar.java:2818)
	at java.util.Calendar.updateTime(Calendar.java:3393)
	at java.util.Calendar.getTimeInMillis(Calendar.java:1782)
	at java.util.Calendar.getTime(Calendar.java:1755)
	at java.text.SimpleDateFormat.parse(SimpleDateFormat.java:1532)
	at java.text.DateFormat.parse(DateFormat.java:364)
	at com.SimpleDateFormatTest.lambda$testParse$0(SimpleDateFormatTest.java:49)
	at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1149)
	at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:624)
	at java.lang.Thread.run(Thread.java:748)
[线程]6---Fri Oct 01 18:30:01 CST 8202
[线程]12---Sat Oct 08 18:30:01 CST 2022
Exception in thread "[线程]1" java.lang.NumberFormatException: multiple points
	at sun.misc.FloatingDecimal.readJavaFormatString(FloatingDecimal.java:1890)
	at sun.misc.FloatingDecimal.parseDouble(FloatingDecimal.java:110)
	at java.lang.Double.parseDouble(Double.java:538)
	at java.text.DigitList.getDouble(DigitList.java:169)
	at java.text.DecimalFormat.parse(DecimalFormat.java:2089)
	at java.text.SimpleDateFormat.subParse(SimpleDateFormat.java:1869)
	at java.text.SimpleDateFormat.parse(SimpleDateFormat.java:1514)
	at java.text.DateFormat.parse(DateFormat.java:364)
	at com.SimpleDateFormatTest.lambda$testParse$0(SimpleDateFormatTest.java:49)
	at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1149)
	at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:624)
	at java.lang.Thread.run(Thread.java:748)
[线程]0---Sat Oct 08 18:30:01 CST 2022
[线程]12---Sat Oct 08 18:30:01 CST 2022
[线程]13---Sat Oct 08 18:30:01 CST 2022
[线程]18---Sat Oct 08 18:30:01 CST 2022
[线程]6---Sat Oct 01 18:30:01 CST 2022
[线程]7---Sat Oct 08 18:30:01 CST 2022
[线程]10---Sat Oct 08 18:30:01 CST 2022
[线程]15---Sat Oct 08 18:00:01 CST 2022
[线程]17---Sat Oct 08 18:30:01 CST 2022
[线程]14---Sat Oct 08 18:30:01 CST 2022
预期结果个数 1---实际结果个数7

不仅有的线程结果不正确，甚至还有一些线程还出现了异常！

为什么SimpleDateFormat类不是线程安全的？

SimpleDateFormat继承了DateFormat，DateFormat内部有一个Calendar对象的引用，主要用来存储和SimpleDateFormat相关的日期信息。

SimpleDateFormat对parse()方法的实现。关键代码如下：

 @Override
public Date parse(String text, ParsePosition pos) {
    ...省略中间代码
    Date parsedDate;
    try {
        ...
        parsedDate = calb.establish(calendar).getTime();
    } catch (IllegalArgumentException e) {
       ...
    }

    return parsedDate;
}

establish()的实现如下：

Calendar establish(Calendar cal) {
    ...省略中间代码
    cal.clear();
    for (int stamp = MINIMUM_USER_STAMP; stamp < nextStamp; stamp++) {
        for (int index = 0; index <= maxFieldIndex; index++) {
            if (field[index] == stamp) {
                cal.set(index, field[MAX_FIELD + index]);
                break;
            }
        }
    }
    ...
    return cal;
}

在多个线程共享SimpleDateFormat时，同时也共享了Calendar引用，在如上代码中，calendar首先会进行clear()操作，然后进行set操作，在多线程情况下，set操作会覆盖之前的值，而且在后续对日期进行操作时，也可能会因为clear操作被清除导致异常。

四. 解决方案

将SimpleDateFormat定义成局部变量，每次使用时都new一个新对象，频繁创建对象消耗大，性能影响一些（JDK文档推荐此做法）

    public static Date parse(String strDate) throws ParseException {
         SimpleDateFormat sdf = new SimpleDateFormat("yyyy-MM-dd HH:mm:ss");
         return sdf.parse(strDate);
    }

维护一个SimpleDateFormat实体，转换方法上使用 Synchronized 保证线程安全：多线程堵塞（并发大系统不推荐）

    private static SimpleDateFormat sdf = new SimpleDateFormat("yyyy-MM-dd HH:mm:ss");
      
    public static String formatDate(Date date)throws ParseException{
        synchronized(sdf){
            return sdf.format(date);
        }  
    }
    
    public static Date parse(String strDate) throws ParseException{
        synchronized(sdf){
            return sdf.parse(strDate);
        }
    }

使用ThreadLocal : 线程独享不堵塞，并且减少创建对象的开销（如果对性能要求比较高的情况，推荐这种方式）。

    public static ThreadLocal<DateFormat> threadLocal = ThreadLocal.withInitial(
            () -> new SimpleDateFormat("yyyy-MM-dd HH:mm:ss")
    );
    public static Date parse(String strDate) throws ParseException {
        return threadLocal.get().parse(strDate);
    }

DateTimeFormatter是Java8提供的新的日期时间API中的类，DateTimeFormatter类是线程安全的，可以在高并发场景下直接使用。

    String dateTimeStr= "2016-10-25 12:00:00";
    DateTimeFormatter formatter02 = DateTimeFormatter.ofPattern("yyyy-MM-dd HH:mm:ss");
    LocalDateTime localDateTime = LocalDateTime.parse(dateTimeStr,formatter02);
    System.out.println(localDateTime);
    String format = localDateTime.format(formatter02);
    System.out.println(format);

    2016-10-25T12:00
    2016-10-25 12:00:00

最终，我们根据实际情况公共包DateUtil类提供的strConvertDate方法，原理是按照方案1来解决该问题。

本文参与腾讯云自媒体同步曝光计划，分享自作者个人站点/博客。

原始发表：2022-11-08，如有侵权请联系 cloudcommunity@tencent.com 删除

java