for 循环使用不当导致死循环，CPU激增

版权声明：本文为博主原创文章，遵循 CC 4.0 BY-SA 版权协议，转载请附上原文出处链接和本声明。

本文链接： https://blog.csdn.net/red_sheeps/article/details/78964718

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就一直成立，最终导致死循环。