查询网站备案进度精准客源app

最近自己用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);}}
}