多么痛的领悟-代码优化导致的BUG

版权声明:本文为博主原创文章,未经博主允许不得转载。 https://blog.csdn.net/goldenfish1919/article/details/8755378
一大早服务器报警,top一下,系统CPU负载已经飚到了15,导致无法访问!
jstack pid打印堆栈,大量解析json的线程在RUNNABLE。
忽然想到之前为了json解析的效率,放弃了现成的类库,手动做json解析,这样做的结果是效率提高了50倍以上,当时还沾沾自喜了好一阵子!
万万没想到今天忽然一条格式不合法的数据,竟然导致了死循环!

暂且不去说脏数据是如何进入到系统的,看一下解析Json的代码:
public static List<Long> getPartneridsFromJson(String data){
    //{\"data\":[{\"partnerid\":982,\"count\":\"10000\",\"cityid\":\"11\"},{\"partnerid\":983,\"count\":\"10000\",\"cityid\":\"11\"},{\"partnerid\":984,\"count\":\"10000\",\"cityid\":\"11\"}]}
    //上面是正常的数据
    List<Long> list = new ArrayList<Long>(2);
    if(data == null || data.length() <= 0){
        return list;
    }    
    int datapos = data.indexOf("data");
    if(datapos < 0){
        return list;
    }
    int leftBracket = data.indexOf("[",datapos);
    int rightBracket= data.indexOf("]",datapos);
    if(leftBracket < 0 || rightBracket < 0){
        return list;
    }
    String partners = data.substring(leftBracket+1,rightBracket);
    if(partners == null || partners.length() <= 0){
        return list;
    }
    while(partners!=null && partners.length() > 0){
        int idpos = partners.indexOf("partnerid");
        if(idpos < 0){
            break;
        }
        int colonpos = partners.indexOf(":",idpos);
        int commapos = partners.indexOf(",",idpos);
        if(colonpos < 0 || commapos < 0){
            //partners = partners.substring(idpos+"partnerid".length());//1
            continue;
        }
        String pid = partners.substring(colonpos+1,commapos);
        if(pid == null || pid.length() <= 0){
            //partners = partners.substring(idpos+"partnerid".length());//2
            continue;
        }
        try{
            list.add(Long.parseLong(pid));
        }catch(Exception e){
            //do nothing
        }
        partners = partners.substring(commapos);
    }
    return list;
}

注意1处和2处,如果格式不合法就直接进行下一次循环。问题就出在这里,如果partners.substring(colonpos+1,commapos)取出来是空,进入下一次循环的时候没有更改原字符串,下次还会重复执行,导致了死循环!
很低级的一个失误!Fuck!
教训:
(1)不要轻易的优化代码。

(2)一定要做好单元测试,考虑正常分支和异常分支,正常数据和异常数据。

(3)CPU负载过高,很可能是出现了死循环!

ps:看到一个打印堆栈的方法:

写一个打印堆栈的方法:
public static java.util.List<Thread> list_threads(){
    int tc = Thread.activeCount();
    Thread[] ts = new Thread[tc];
    Thread.enumerate(ts);
    return java.util.Arrays.asList(ts);
}
@Path(value = "/api/stackinfo")
@GET
public ActionResult stackinfo() {
	String str = "<strong>Memory:</strong>";
	str +="<ol>";
	str +="<li>freeMemory="+Runtime.getRuntime().freeMemory()/(1024*1024)+"M</li>";
	str +="<li>totalMemory="+Runtime.getRuntime().totalMemory()/(1024*1024)+"M</li>";
	str +="<li>maxMemory="+Runtime.getRuntime().maxMemory()/(1024*1024)+"M</li>";
	str +="</ol>";
	str +="<br/>";
	str +="<strong>Thread:</strong>";
	str +="<ol>";
	for(Thread t : list_threads()){
		str +="<li>"+t.getName()+","+t.getState()+":"+ t.getClass().getName()+"</li>";
		StackTraceElement[] elems = t.getStackTrace();
		str += "<ol>";
		for(StackTraceElement elem : elems){
			str +="<li>    "+elem.toString()+"</li>";
		}
		str += "</ol>";
	}
	str +="</ol>";	
	beat.getResponse().setContentType("text/html;charset=UTF-8");
	return new ContentResult(str);
}


阅读更多
换一批

没有更多推荐了,返回首页