一、背景:正式环境偶尔数据查询不出来,而且有时候即使查询出来数据也是错误,很是头疼!多次测试又还原不了这种场景,只有通过后台日志一点点分析,终于抓到一次报错,信息如下:
java.lang.NumberFormatException: For input string: ""
at java.lang.NumberFormatException.forInputString(NumberFormatException.java:65) ~[?:1.8.0_60]
at java.lang.Long.parseLong(Long.java:601) ~[?:1.8.0_60]
at java.lang.Long.parseLong(Long.java:631) ~[?:1.8.0_60]
at java.text.DigitList.getLong(DigitList.java:195) ~[?:1.8.0_60]
at java.text.DecimalFormat.parse(DecimalFormat.java:2051) ~[?:1.8.0_60]
at java.text.SimpleDateFormat.subParse(SimpleDateFormat.java:1867) ~[?:1.8.0_60]
at java.text.SimpleDateFormat.parse(SimpleDateFormat.java:1514) ~[?:1.8.0_60]
at java.text.DateFormat.parse(DateFormat.java:364) ~[?:1.8.0_60]
at com.XXXX.atom.XXXX.XXX.dao.XXXXlDao.getAlarmByIds(XXXXXDao.java:232)
二、分析:看日志字面理解是空字符串强转报错,看了代码发现是:
sdfOne.parse(timeInfo);其中public static SimpleDateFormat sdfOne = new SimpleDateFormat("yyyyMMdd");是全局声明,三个异步方法同时调用。莫非是资源竞争引起,带着这个疑问,测试了下SimpleDateFormat 并发下的转换结果确实有问题 。
三、原因:SimpleDateFormat是非线程安全。
四、解决方法:
1.SimpleDateFormat改为非全局,这样可以解决问题,但是开销比较大。
2.使用的时候加同步锁。
3.使用线程安全的commons-lang包的DateUtils和DateFormatUtils类
五、复盘:
对于偶发的现象,代码逻辑看不出明显的错误的场景,需要从资源竞争方向切入,一点点的剖析。