Code style



Не нужно проверять, содержит ли коллекция элементы, когда нужно по ней итерироваться. Просто итерируйтесь.



Плохо

if (!foos.isEmpty() {

for(Foo foo: foos) {

bar.delete(foo);

}

}




Хорошо

for(Foo foo: foos) {

bar.delete(foo);

}