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

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

  • 1
    点赞
  • 2
    收藏
    觉得还不错? 一键收藏
  • 0
    评论
评论
添加红包

请填写红包祝福语或标题

红包个数最小为10个

红包金额最低5元

当前余额3.43前往充值 >
需支付:10.00
成就一亿技术人!
领取后你会自动成为博主和红包主的粉丝 规则
hope_wisdom
发出的红包
实付
使用余额支付
点击重新获取
扫码支付
钱包余额 0

抵扣说明:

1.余额是钱包充值的虚拟货币,按照1:1的比例进行支付金额的抵扣。
2.余额无法直接购买下载,可以购买VIP、付费专栏及课程。

余额充值