最近自己用UnitTest测出了自己犯的比较隐蔽的错误, 就是在两层for 循环中, 在最里层的循环里面动态的改了外层循环里面的数据,导致java的循环计数器出错, 抛出ConcurrentModificationException. 这个错误在系统中不会出现, 只有在UnitTest的时候才出现.
片段如下
for (PersonTO personTO : accountTO.getPersons()) {
  for (ExpenseTO expTO : personTO.getExpenses()) {
    if (expTO.getPersonalFinanceId()==null){
      personTO.getExpenses().remove(expTO);
    }
  }
}

这里personTO里面expense的计数器在内循环被破坏了, 计数器和实际数据数量不符, 这个问题不大容易测出来,有很大的隐患. 可以做如下修改, 用倒序for循环:
for (PersonTO personTO : accountTO.getPersons()) {    
  for(int i=personTO.getExpenses().size()-1;i>=0;i--){
    	ExpenseTO eto=personTO.getExpenses().get(i);
    	  if(eto.getPersonalFinanceId()==null){
    		personTO.getExpenses().remove(i);
          }
  }
}
评论
发表评论

您还没有登录,请登录后发表评论

darkjune
搜索本博客
我的相册
45a46d56-bef4-4041-ba1f-3dbf35966fe7-thumb
siebel
共 1 张
最近加入圈子
存档
最新评论