loadCache之实习那些事

最近碰到一个特无语的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");
}