# GreekFortunes_review

1. Неиспользуемые переменные

2. Вызов виртуального метода в конструкторе

3. Лишний инклуд

4. Зачем останавливать экшн в Prepare методе? тут явно что-то не то

5. Неявное приведение типов. Сейчас отключен данный ворнинг, но для x64 сборки пришлось исправить множество подобных ошибок. Лучше всегда следить за таким, так как может быть потеря точности/переполнение типа, что приводит к очень неприятным, трудноуловимым багам. Это нужно поправить.








6. Опечатка

7. Много использования таймеров по типу как AddTask. В многих местах есть их запуск, но не предусмотрена остановка при релизе моделей. В случае экстренного выхода из игры, могут быть краши. Стоит пересмотреть эти моменты.

8. Подобное также и с SetEndHandler, там где экшины используются из разметки нужно пересмотреть чтобы они обязательно релизились при релизе модели. Так как в EndHandlers часто обращаешься к другим сущностям, не просто бросаешь нотифаи, и может быть обращение к nullptr. Или добавить проверки на nullptr.

9. Может, стоит поменять провокационное название CheatBonusGame :)

Графика
-------

Сделала профилировку количества дровколлов. В некоторых сценах доходит до 260, что может дать просадку фпс на слабых устройствах.
Проседает в бигвине и попапах. Стоит выполнить следующие шаги для улучшения, пересобрать апк и протестировать на устройстве типа zte:
- добавить список статических изображений игры в конфигурационный файл texture_packer_info для дальнейшей их правильной упаковки - посмотри как в EasternJakpots;
- уменьшить в партиклах coins_right/coins_left maxParticles до 110. Визуально отличие небольшое, а это около - 70 дровколлов. Так как там анимированная монета, получается, что практически каждая из них - это отдельный дровколл;
- попросить у Лены отключить неиспользуемые аттачменты в спайн проектах, используемых в попапах;
- для спайн вью, которые не отображаются в сцене - обязательно устанавливать visible=false