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

一大早服务器报警，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:看到一个打印堆栈的方法：

 

写一个打印堆栈的方法：
@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);
}