Исходное сообщение
"Языку программирования LISP исполнилось 60 лет" Отправлено шухер, 23-Окт-18 11:03
Посмотрел код, в принципе неплохо. Ставит везде final, это одобрямс. Но... В некоторых местах неоправдано создаются лишние объекты. Например: public Response act(final Request req) throws IOException {     final String name = new RqFormBase(req).param("name")            .iterator().next().trim(); На парсинг параметра он породил как минимум 2 объекта - RqFormBase и Iterator. Я как главный разраб за это бы наказал, читай - недопустил бы подобный код в проекте за который я отвечаю. Жава и без этого любит покушать памяти и для меня странно видеть, как люди делают вид, что этого не знают. Надо уважать железо и считаться с ним хоть в какой-то степени. Чем хуже это? final String name = req.firstParam("name").trim(); trim() тоже лучше спрятать под капот, если он его неизбежно делает каждый раз в подобных местах. Итого: final String name = req.firstParam("name"); vs final String name = new RqFormBase(req).param("name")            .iterator().next().trim(); Портянка ТкApp тоже на любителя. Оно вроде наглядно, но присмотревшись понимаешь, что эта лапша заставляет тебя нехило всматриваться. Этакий непонятый гений получается.