最近自己用UnitTest测出了自己犯的比较隐蔽的错误, 就是在两层for 循环中, 在最里层的循环里面动态的改了外层循环里面的数据,导致java的循环计数器出错, 抛出ConcurrentModificationException. 这个错误在系统中不会出现, 只有在UnitTest的时候才出现.
片段如下
这里personTO里面expense的计数器在内循环被破坏了, 计数器和实际数据数量不符, 这个问题不大容易测出来,有很大的隐患. 可以做如下修改, 用倒序for循环:
片段如下
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);
}
}
}