# 586 | Festive Freedom | Code review ### 1. Переглянути, чи ці функції використовуються (FestiveFreedom_types.hpp): ![](https://hackmd.io/_uploads/SkTstkPwn.png) ### 2. Перенести перевизначені методи в protected секцію (FestiveFreedom_BasicGame.cpp): ![](https://hackmd.io/_uploads/rkHT9yPPh.png) #### В тому ж файлі форматування: ![](https://hackmd.io/_uploads/HkLliJDv2.png) ### 3. Дописати ідентифікатор override(FestiveFreedom_BonusGame.hpp): ![](https://hackmd.io/_uploads/r1saoyDwh.png) ### 4. Форматування при наслідуванні: ![](https://hackmd.io/_uploads/rk9I2JvD2.png) ![](https://hackmd.io/_uploads/B1NOnywD3.png) ![](https://hackmd.io/_uploads/SkNKhkDP2.png) ![](https://hackmd.io/_uploads/rJTt2yDvn.png) ![](https://hackmd.io/_uploads/SJ89hkPP2.png) ###### Варто привести до одного стилю ### 5. Метод без імплементації(Утіліти): ![](https://hackmd.io/_uploads/ByAR2kDw3.png) ### 6. Дописати до методів ідентифікатор const(FestiveFreedom_JmEffectBuilder.cpp): ![](https://hackmd.io/_uploads/B1zcpkwD3.png) ![](https://hackmd.io/_uploads/HJIeR1wP3.png) ##### Той самий файл, видалити коментарі, якщо вони не потрібні: ![](https://hackmd.io/_uploads/r1fpayww2.png) ### 7. Брекет з нового рядка (JackpotManiaAnimations::AddEffect): ![](https://hackmd.io/_uploads/ryF501vD3.png) #### JackpotManiaAnimations::IsOpenLineEffect ![](https://hackmd.io/_uploads/Hy23R1wDn.png) ### 8. Додати до методів ідентифікатор const (FestiveFreedom_JackpotMania.cpp): ![](https://hackmd.io/_uploads/HJaEygwP3.png) ### 9. Почистити інклуди (FestiveFreedom_EnterPickemBonus.cpp): ![](https://hackmd.io/_uploads/Byw91ewDn.png) ### 10. Додати до методу ідентифікатор const (FestiveFreedom_PickemCellsManager.cpp): ![](https://hackmd.io/_uploads/BkRRygwvh.png) ### 11. Почистити коментарі (FestiveFreedom_EnterWheelBonus.cpp): ![](https://hackmd.io/_uploads/rkFWxxvDh.png) ### 12. Можливо, є варіант винесення віртуальних методів за межі конструктора (FestiveFreedom_HorizontalWheel.cpp): ![](https://hackmd.io/_uploads/Hkn_geDDh.png) ##### Той самий файл, форматування: if {       //////////////// } ![](https://hackmd.io/_uploads/r1piggPv2.png) ![](https://hackmd.io/_uploads/rJQ1WgwDh.png) ![](https://hackmd.io/_uploads/B1eybbevDn.png) ##### +const на змінну shift_symbols ##### Додати константність до змінних ![](https://hackmd.io/_uploads/S1NEbeDPn.png) ### 13. Додати ідентифікатор const до методу(FestiveFreedom_Wheel.cpp): ![](https://hackmd.io/_uploads/BJKhbxwPn.png) ### 14. Додати ідентифікатор const до методу(FestiveFreedom_WheelBonus.cpp): ![](https://hackmd.io/_uploads/SyxobgDPn.png) ### 15. Додати константність до змінної(FestiveFreedom_WheelView.cpp): ![](https://hackmd.io/_uploads/SJ_eGePD3.png) ### 16. Замінити pass-by-value на посилання та, можливо, мало б сенс винести ці методи в анонімний namespace(FestiveFreedom_BasicGame.cpp): ![](https://hackmd.io/_uploads/HyVj7gDD2.png) ### 17. Почистити зайві інклуди(FestiveFreedom_BasicGameAnimations.cpp): ![](https://hackmd.io/_uploads/S15y4lvv2.png) ##### Додати ідентифікатор const: ![](https://hackmd.io/_uploads/BJBmVgwwh.png) ![](https://hackmd.io/_uploads/HJ1UNlvwh.png) ### 18. Форматування, зайвий ентер (FestiveFreedom_BigWin.cpp): ![](https://hackmd.io/_uploads/rkhONePPn.png) ### 19. Використати using::slot_features::BaseBigWin: ![](https://hackmd.io/_uploads/Skz1Sgwwh.png) ### 20. - || - ![](https://hackmd.io/_uploads/Hyorrevv2.png) ### 21. Видалити або =default (FestiveFreedom_DecorManager.cpp): ![](https://hackmd.io/_uploads/Skx5BxDw3.png) ### 22. Використати using: ![](https://hackmd.io/_uploads/BJfgLeDw2.png) ### 23. Почистити коментарі(FestiveFreedom_SeriesProcessor.cpp): ![](https://hackmd.io/_uploads/rJh78gPw2.png)