From 3d377a632029e8d7ade3cf6d2594f937c9d71fa9 Mon Sep 17 00:00:00 2001 From: antoshkaxxr <63500718+antoshkaxxr@users.noreply.github.com> Date: Sun, 26 Mar 2023 20:53:12 +0500 Subject: [PATCH] =?UTF-8?q?Create=20review-=D0=93=D0=B0=D0=B1=D0=BE=D0=B2.?= =?UTF-8?q?md?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- ...\320\223\320\260\320\261\320\276\320\262.md" | 17 +++++++++++++++++ 1 file changed, 17 insertions(+) create mode 100644 "review-\320\223\320\260\320\261\320\276\320\262.md" 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, думаю, что это не так критично + +В целом игра понравилась, судя по коду, всё продумано до мелочей :)