Skip to content
Open
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
17 changes: 17 additions & 0 deletions review-Габов.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,17 @@
Преимущества игры:
1) Старая добрая игра на запоминание картинок на карточках, люблю в такое играть :)
2) Продуманы и прорисованы рубашки карточек и картинки на обороте
3) Ведётся счёт игры

Замечания и вопросы:
1) В классе CardGame есть метод getRandomNumbers, который, по моему мнению, мог бы
находиться в классе Random, назначение вроде у него похожее
2) В Counter много повторяющегося кода из-за отрисовки каждой цифры. Могу предположить,
что можно было сделать фабричные методы для каждой из 6 частей, где зафиксирован address,
и, подавая им на вход массивы нужных значений длины 16, отрисовывать текущую цифру. То же
самое и с отрисовкой лицевой стороны в зависимости от id в классе Picture. Код,
конечно, сократится в разы, но с другой стороны понимаю, что очень многое нужно проделать,
чтобы получить это видоизменение, поэтому это, скорее всего, лишь пожелание. Тем более в
рамках языка Jack, думаю, что это не так критично

В целом игра понравилась, судя по коду, всё продумано до мелочей :)