Java 线程内递归 Bug 一例

最新推荐文章于 2024-09-25 18:19:29 发布

转载最新推荐文章于 2024-09-25 18:19:29 发布 · 150 阅读

文章标签：

#java

讨论了一个线程的run方法中使用递归方法导致的问题，提出了两种解决方案：抛出异常和调试代码找到问题根源。详细分析了两种方法的优缺点，并解释了为什么作者不认可同事提出的通过抛出异常的解决方式。

摘要生成于 C知道，由 DeepSeek-R1 满血版支持，前往体验 >

一个线程的run方法里使用递归方法，出了Bug。

private boolean ispass(String creationId){
List<Map> maps =creationService.getCreationById(creationId);
if(maps != null && maps.size()>0){
Map m = maps.get(0);
String state= (String) m.get("STATE");
if(state != null && state.equals("0")){
return true;
}else{
try {
if(falg == 10){
return false;
}
Thread.sleep(1000 * 60);
falg++;
} catch (InterruptedException e) {
// TODO Auto-generated catch block
e.printStackTrace();
return false;
}
ispass(creationId);
}
}else{
return false;
}
return false;
}

同事的解决办法，修改代码通过抛出异常的方式，也算解决了。当然了我不认可。:)

private boolean ispass(String creationId) throws SecussException{
List<Map> maps =creationService.getCreationById(creationId);
if(maps != null && maps.size()>0){
Map m = maps.get(0);
String state= (String) m.get("STATE");
if(state != null && state.equals("0")){
throw new SecussException("ok");
}else{
try {
if(falg == 10){
return false;
}
Thread.sleep(1000 * 60);
falg++;
} catch (InterruptedException e) {
// TODO Auto-generated catch block
e.printStackTrace();
return false;
}
ispass(creationId);
}
}else{
return false;
}
return false;
}

我的解决办法：调试代码，发现问题根源，该返回的地方，没返回；不该返回的地方，返回了错误的值。

private boolean ispass(String creationId){
List<Map> maps =creationService.getCreationById(creationId);
if(maps != null && maps.size()>0){
Map m = maps.get(0);
String state= (String) m.get("STATE");
if(state != null && state.equals("0")){
return true;
}else{
try {
if(falg == 10){
return false;
}
Thread.sleep(1000 * 60);
falg++;
} catch (InterruptedException e) {
// TODO Auto-generated catch block
e.printStackTrace();
return false;
}
return ispass(creationId);
}
}else{
return false;
}
}