Нашли или выдавили из себя код, который нельзя назвать нормальным,
на который без улыбки не взглянешь?
Не торопитесь его удалять или рефакторить, — запостите его на
говнокод.ру, посмеёмся вместе!
public Show extractDefault(Extract request) {
CriteriaBuilder criteriaBuilder = converter.convertToDO(request);
List<EntityDO> list = dao.extract(criteriaBuilder, 0, Byte.MAX_VALUE);
if (list.size() != 8) {
if (list.size() != 0) {
throw new AssertionError(list.size());
}
criteriaBuilder = createCriteriaBuilder();
list = dao.extract(criteriaBuilder, 0, 100);
}
if (list.size() != 8) {
throw new IllegalStateException("Entity not found");
}
return converter.convertToSystem(list);
}
Решил запостить код, который дергает сущность из базы как явный говнокод, но после повторного просмотра уже и не кажется таким уж ужасным. Разве что в extract передаются разные значения pagesize(3-ий аргумент) и list.size() сравнивается не понятно. А что скажет общественность?
>Просто один объект хранится 8 строками в таблице.
>Другое дело, что она не объявлена как константа.
CriteriaBuilder напейсали, dao напейсали, а такую простую штуку как RowMapper не осилили?
Зачем писать на жабе, пытаться делать сложные и модные нынче ооп-абсракции.
Но при этом не создать объект там где он реально нужен?
🙂
>Другое дело, что она не объявлена как константа.
CriteriaBuilder напейсали, dao напейсали, а такую простую штуку как RowMapper не осилили?
Зачем писать на жабе, пытаться делать сложные и модные нынче ооп-абсракции.
Но при этом не создать объект там где он реально нужен?