diff --git "a/review-\320\223\320\260\320\261\320\276\320\262.md" "b/review-\320\223\320\260\320\261\320\276\320\262.md" new file mode 100644 index 0000000..14f3ad6 --- /dev/null +++ "b/review-\320\223\320\260\320\261\320\276\320\262.md" @@ -0,0 +1,17 @@ +Преимущества игры: +1) Старая добрая игра на запоминание картинок на карточках, люблю в такое играть :) +2) Продуманы и прорисованы рубашки карточек и картинки на обороте +3) Ведётся счёт игры + +Замечания и вопросы: +1) В классе CardGame есть метод getRandomNumbers, который, по моему мнению, мог бы +находиться в классе Random, назначение вроде у него похожее +2) В Counter много повторяющегося кода из-за отрисовки каждой цифры. Могу предположить, +что можно было сделать фабричные методы для каждой из 6 частей, где зафиксирован address, +и, подавая им на вход массивы нужных значений длины 16, отрисовывать текущую цифру. То же +самое и с отрисовкой лицевой стороны в зависимости от id в классе Picture. Код, +конечно, сократится в разы, но с другой стороны понимаю, что очень многое нужно проделать, +чтобы получить это видоизменение, поэтому это, скорее всего, лишь пожелание. Тем более в +рамках языка Jack, думаю, что это не так критично + +В целом игра понравилась, судя по коду, всё продумано до мелочей :)