Java книги

Привет всем!

Продолжаю работу над своей игрушкой в телеграме. На этот раз решил переработать весь код, потому как со старой версией было не очень удобно добавлять новые функции. А из новых функций это добавление умений и их применение в бою, в основном. В процессе реализации умений столкнулся с проблемой действия умения на протяжении нескольких раундов. Перед тем, как рассказать о реализации, несколько слов о том как реализована боевая система в целом. Есть сущность пользователь и противник (моб). Обе эти сущности сохраняются в бд и имеют такие характеристики как: hp, защита, id приемов, id противника с которым идет сражение и другие, которые относятся к сражению. Сначала сделал отдельные таблицы в базе для боев с мобом и живым игроком. Идея хорошая, потому как можно вести статистику и смотреть результаты боев, но у меня пока проблемы с взаимосвязями таблиц, так как надо сохранять в них те приемы, которые были использованы, а это значит, что в одном из столбцов таблиц надо хранить список с объектами "прием", а с этим пока есть сложность, поэтому на сегодня обошел ее созданием HashMap боев прямо в коде. В этом тоже есть плюсы, так как запросов к базе поменьше, а для статистики боев сделана отдельная таблица и есть логи, где можно посмотреть результаты, пока идет тестирование.

Вообще, для меня каждая задача по реализации каких то функций это вызов, над решением которого интересно думать и проверять что получилось. Казалось бы, всего одна строка кода не дописана, не учтена и это тянет ворох проблем. В общем, крайне интересное занятие.

Забавную ситуацию вспомнил: как-то на ревью коллега стал меня убеждать в необходимости своих правок. Обосновывал тем, что "из теста вернули с комментарием — очень долго работает запрос, не можем дождаться ответа и обрубаем". Поэтому он решил изменить способ итерации, чтобы всё ускорилось 🚀

Краткая суть: был метод, возвращавший некие данные за период (12 месяцев с отмоткой назад, endDate = 12 месяцев назад, нижняя граница). Внутри себя - он итерировался по этим месяцам, дергал другой метод для получения помесячных наборов, и агрегировал их. Конструкция была такая:

LocalDate startDate = LocalDate.of(2022, 11, 1);

LocalDate endDate = startDate.minusMonths(12);

for (LocalDate date = endDate; date.isBefore(startDate); date = date.minusMonths(1))

Подобный метод-агрегатор был не один, и на ревью первого из них я коллеге посоветовал генерировать набор дат вызовом библиотечного АПИ datesUntil с шагом в месяц, и дальше сгенерированный набор использовать. Это дело благополучно забылось, даты продолжали создаваться внутри for и кочевали с копипастой, дожив до обсуждаемого момента 🤷🏻‍♂️

Я сильно засомневался, что в совершенно ненагруженном вызове, при итерации по 12 элементам — иной способ итерации даст какую-то заметную разницу. Сделал иллюстративные наброски в JShell (полноценные бенчмарки лень было, да и ни к чему) для сравнения разных способов с исходным. Типа таких: