dimOn писал(а):
Феанор писал(а):
Код:
public ArrayList<Datable> selectDeleted(ArrayList<Datable> array){
for(Datable item:array){
if(!item.getDate2().after(new Date())
array.remove(item);
}
return array;
}
for(Datable item:array){
if(!item.getDate2().after(new Date())
array.remove(item);
}
return array;
}
так нельзя делать ни в коем случае, этот код не будет работать или будет работать неправильно. при удалении итератор тупо сломается. надо удалять из самого итератора, для этого надо не пользоваться enhanced foreach loop а юзать несахарный итератор.
да да
либо создать новую коллекцию и элементы которые под условие попадают в него добавить а потом их после завершения лупа удалять
dimOn писал(а):
Хм... ну в целом ясно. Но всё равно бессмысленные интерфейсы - типичный антипаттерн.
Угу, вот я сейчас свой код сел приводить в порядок. Накожено (почти как нагажено, хотя по сути так оно и есть =)) много, причем с типичным так у нас никогда не будет.... ну блин, ладно, сейчас мы добавим вот сюда и вот сюда и все будет как надо... а теперь эта часть нужна немного по другому поэтому здесь мы поправим а туда добавим... Сейчас вроде устаканилось все более или менее, но чтоб на будущее такого не ловить решил поспрашивать у людей более опытных =)
