---
# System prepended metadata

title: GreekFortunes_review

---

# GreekFortunes_review

1. Неиспользуемые переменные
![](https://i.imgur.com/JNCbn23.png)
![](https://i.imgur.com/c1yJD7N.png)

2. Вызов виртуального метода в конструкторе
![](https://i.imgur.com/vNux3vS.png)

3. Лишний инклуд
![](https://i.imgur.com/sHZhEMj.png)

4. Зачем останавливать экшн в Prepare методе? тут явно что-то не то
![](https://i.imgur.com/t5qAtJ5.png)

5. Неявное приведение типов. Сейчас отключен данный ворнинг, но для x64 сборки пришлось исправить множество подобных ошибок. Лучше всегда следить за таким, так как может быть потеря точности/переполнение типа, что приводит к очень неприятным, трудноуловимым багам. Это нужно поправить.
![](https://i.imgur.com/MQmpXVc.png)

![](https://i.imgur.com/unSqAVr.png)

![](https://i.imgur.com/ki2kWM0.png)

![](https://i.imgur.com/hr1Q7wu.png)

![](https://i.imgur.com/TwYjYNw.png)

![](https://i.imgur.com/sV4eytP.png)

![](https://i.imgur.com/XdqArhv.png)

![](https://i.imgur.com/o9Zrt3a.png)

6. Опечатка 
![](https://i.imgur.com/1sSuT7h.png)

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

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


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



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

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