Рефакторинг синхронизации и наследования

Murlakotamus / Boggarton

0 stars 0 forks source link

Closed Murlakotamus closed 6 years ago

Murlakotamus commented 6 years ago

Избавиться от Lock/Condition там, где достаточно Synchronized
Класс RealGame нарушает принцип Лисков, нужно его и SimpleGame унаследовать от общего абстрактного класса

Murlakotamus commented 6 years ago

В иерархии классов игр сильно дублируется код метода processStage(), необходимо общие его части вынести в базовый класс

Murlakotamus commented 6 years ago

Переименовать класс GameOutcomeParams в просто GameParams
Класс SimpleGlass нарушает принцип Лисков, необходимо сделать абстрактный класс AbstractSimpleGlass и все три реальных класса стакана (SimpleGlass, MultiplayerGlass и ReplayGlass) унаследовать от него