CPU激增
最近发现生产上的某些台服务区CPU激增,跟着内存也增长的比较厉害,就跟踪了下(CPU过高查询方法此处不再赘述,自行百度)发现线程卡在某一方法处,如下:
代码
看了这个方法的代码,原来是一个for循环中的代码,就猜想可能是死循环了,如下:
public static String getGMTNumOfYear(Date yyyyDateTime) throws ParseException {
String gmtDate = getGMT(yyyyDateTime);
SimpleDateFormat simpleDateFormat = new SimpleDateFormat(
"yyyyMMddHHmmss");
Date date =simpleDateFormat.parse(gmtDate);
Calendar cd = Calendar.getInstance();
cd.setTime(date);
String yearOfNumber = cd.get(Calendar.DAY_OF_YEAR) + "";
if (yearOfNumber.length() < 3) {
for (int i = 3 - yearOfNumber.length(); i > 0; i++) {
yearOfNumber = "0" + yearOfNumber;
}
}
return yearOfNumber;
}
这个方法的目的是为了获取某一天在该年中是第几天,如果不足三位(一年365天),则前补0然后返回出去.
原因
可是死循环不应该测不出来的啊,一个功能的上线都是经过测试部测试的。
后来分析了下,果然找到了原因,查看代码提交日期,是下半年开发的代码,也就是说当时测试的时候不管怎么测试,返回的yearOfNumber都是三位,经过for循环的初始化int i = 3 - yearOfNumber.length(),i为0就不会执行这个for循环,所以当时就没测出来这个问题。
然而刚过元旦,今天是1月3号,华丽丽的执行了这个for循环,当时代码开发者也是想当然了,觉得for循环的初始化会一直执行,熟不知,只会执行一次,最终的结果就导致了i一直增加,for循环的条件i > 0就一直成立,最终导致死循环。