最近碰到一个特无语的bug, 俺只在原来代码基础上添加了一个功能,结果就出现BUG. 具体的功能是去数据库里面取数据展示在饼图里面,问题饼图仅仅更新了一次并没有在一定时间内去更新数据,也就是loadCache出现了问题。具体代码在下面:
private class loadCacheThread extends Thread {
private boolean run = true;
public void run(){
while (true) {
runOnce();
}
}
public void runOnce() {
if(run){
run = false;
cacheServerStatus();
getAllAgentVersionFromCMSRP();
run = true;
}
try {
Thread.sleep(5 * 60 * 1000);
} catch (InterruptedException e) {
log.info("sleep error");
}
}
}
初看这段代码我没发现任何bug,而且在test的时候也是很顺利的运行。但是一到服务器上就出现了bug.,真是坑姐呀! 后来看看可能原因就是run这个变量被设置成false,导致cache都没有被更新。因为cacheServerStatus连接数据库可能会抛出异常。导致后面run = true 永远不会执行,所以我用try catch finally , 在finally执行 run = true.
if(run){
run = false;
try{
cacheServerStatus();
getAllAgentVersionFromCMSRP();
} catch (Exception e){
log.error("runOnce error ",e);
} finally {
run = true;
}
}
try {
Thread.sleep(5 * 60 * 1000);
} catch (InterruptedException e) {
log.info("sleep error");
}