---
# System prepended metadata

title: 586 | Festive Freedom | Code review

---

# 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)