# Piggy Bank review

1. Лишняя ;

2. Неявное преобразование типов

3. Лишние инклуды, файл path, файл без изменений

4. Перенос из секции private в секцию protected родительского класса необходимо обговорить с автором, так как в этом закладывалась некоторая логика и было сделано не просто так. Это нарушение логики архитектуры. И я так понимаю, может повлечь в дальнейшем к багам, когда кто-то будет переопределять эти методы. Лучше так не делать.

----------------------
5. Для чего

6. Это, наверное, лишнее, так как нет Attach

7. RepeatedSpineAnimation - класс, который в перспективе можно переиспользовать. Единственное, вызов статических методов хелпера игры может препятствовать этому. Возможно, стоит подумать и типичные чистые методы, которые повторяются с игры в игру перенести куда-то в глобальное место gui::Utils. Название класса не соответствует сути, так как это не просто повторяющаяся спайн анимация, а повторяющаяся спайн анимация на барабанах.

8. Есть AddTask - нет RemoveAllTasks при релизе на случай принудительного выхода

9. Лучше тут использовать count, так как при проверке через квадратные скобки создается элемент при отсутствии элемента по ключу. Если по логике не задумано подобное. Ранее мы сталкивались с неприятными последствиями подобного в логере когда-то. По коду нужно просмотреть подобные моменты, много где встречается.

10. Может, тут стоит удалять экшн со списка по его окончанию

11. Нужно проверить кейсы имитации свертывания приложения на телефонах на данном моменте.В виндовом клиенте достаточно зажать шапку окна и подержать некоторое время его так. Такая логика выглядит очень сомнительно для данного кейса.

12. Тут бы добавить схему связей моделей для более простого понимания как мы обычно делаем и небольшие комментарии что за что отвечает

13. Странный тип записи с форвард декларейшн, в нашем проекте такого не встречала

14. Для PigletMergeAnimation я бы использовала композицию, так как этот класс слушает по-сути только LockItFeature. Это бы инкапсулировало логику и упростило общее понимание.
А LockItFeatureFinalAnimation - часть PigletMergeAnimation. Это на первый взгляд, может, что-то не видно.
Не ясно для чего связь