Skip to content

review-Федоров.md #3

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
fed1v wants to merge 1 commit into
base: main
Choose a base branch
from
Open

review-Федоров.md #3

Changes from all commits
Commits
Show all changes
1 commit
Select commit
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
11 changes: 11 additions & 0 deletions review-Федоров.md
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,11 @@
+ Идея интересная, реализация тоже понравилась. Приятная графика, интуитивно понятный ход игры, ничего лишнего, за это лайк.
+ Почти весь код сопровождается комментариями, поэтому, при желании, не составит труда в нем разобраться.
Хотя в классе Counter.jack вообще нет комментариев, думаю, они там тоже не помешают :)
+ Написали собственный BubbleSort, функцию для деления с остатком, Tuple. И даже придумали, как реализовать Random!
В общем, всеми силами пытались адаптировать Jack к нашему любимому C#, за это тоже лайк
+ Кода достаточно много, особенно выделяется вышеупомянутый Counter.jack - 1200 строк кода! Но, думаю, сократить его не было возможности
из-за особенностей языка Jack и работы с графикой, поэтому, на мой взгляд, это не критично.
+ Возможно, метод shuffleDeck из CardGame можно было бы перенести в ArrayExtensions и переименовать в shuffleArray,
ведь этот метод, в теории, мог бы понадобиться вам в будущем в других классах. А метод getRandomNumbers - в класс Random.

+ Игра понравилась, авторам удалось сделать качественный проект, Егоры молодцы