Оцените написанную игру

@wikkk · Регистрация: 26.09.2015

Студворк — интернет-сервис помощи студентам

Добрый день, написал игру - сетевые шахматы между игроками. Выложил на github, было бы круто узнать что скажут хорошие программисты, особенно по архитектуре и пр. моментам)

Игра: https://github.com/n-demidov/NetChess

@Aassians · 25.04.2016, 05:33

ты всё сам писал или использовал готовый движок?

Добавлено через 2 минуты
где изображения фигурок взял?

@wikkk · 25.04.2016, 11:05 **[ТС]**

всё сам, ну по Netty - смотрел обучалки, искал примеры, по JavaFX также, чё-то там на форумах пару раз спрашивал + гуглил вопросы, попутно продолжал обучаться.

Архитектуру\логику было интересно самому написать, а потом посмотреть как сделали другие. Вопросов потому что куча) Но руки так и не дошли посмотреть как в других приложениях.

Картинки с какого-то шахматного приложения на гитхабе взял

Добавлено через 2 часа 50 минут
Но шахматного бота я не писал - у меня только игроки могут играть друг с другом по сети.

@Aassians · 25.04.2016, 17:39

видел на андроиде, как много шахмат около сотни.

@turbanoff · 25.04.2016, 18:39

wikkk, сделайте мултимодульный проект, вместо трёх отдельных. Там будет намного проще смотреть.

@wikkk · 14.05.2016, 17:25 **[ТС]**

Сообщение от turbanoff

wikkk, сделайте мултимодульный проект, вместо трёх отдельных. Там будет намного проще смотреть.

Взгляните, пожалуйста, правильно ли сделал мультимодульный проект?

@dzrkot · 15.05.2016, 10:13

Ну не совсем логично раскидано по пакетам, это моё субъективное мнение.

А вот чисто по коду могу сказать следующее;

1) штуки типа

Java
1
if (nextLaunch.before(Calendar.getInstance().getTime()))

выносят в отдельный метод, с понятным названием

2) собственно не советую выносить блоки так, как вы это делаете, т.е. вместо

Java
1
2
3
4
if (nextLaunch.before(Calendar.getInstance().getTime()))
{
     // something
}

лучше

Java
1
2
3
if (nextLaunch.before(Calendar.getInstance().getTime())) {
     // something
}

так просто принято, это приятнее читается, экономит строку

3) у вас было что-то вроде

Java
1
2
for (final User playingUser : gameManager.getPlayingUsers(game))
            connectionManager.sendToUser(playingUser, gameMsg);

и примерно это же было где-то вообще в 1 строку.
Всегда заворачивайте блоки в фигурные скобки, не важно if это или for, однажды вы возможно допустите ошибку и будете её долго искать
Аналогично;

Java
1
if (isSendCurrentGame) messages.add(getCurrentGameMsg(user));

Собственно если это java8 то можно вообще streams использовать

4) вот это трешово смотрится;

Java
1
2
3
4
return new UserProfile(user.getName(), user.getRank(),
                user.getWins(), user.getDefeats(), user.getDraws(),
                user.getTotalTimeOnServer() + connectionManager.getUserCurrentTimeOnServer(user),
                isInvited, isPlaying

можно же было конструктор сделать отдельный

5) какие-то вещи повторяются в разных классах, т.е. их вообще следует вынести в отдельный класс, или если java8 то мб интерфейс с default поведением

@wikkk · 16.05.2016, 08:39 **[ТС]**

Спасибо!

Сообщение от dzrkot

4) вот это трешово смотрится;

Java
1
2
3
4
return new UserProfile(user.getName(), user.getRank(),
                user.getWins(), user.getDefeats(), user.getDraws(),
                user.getTotalTimeOnServer() + connectionManager.getUserCurrentTimeOnServer(user),
                isInvited, isPlaying

можно же было конструктор сделать отдельный

Тут как было: У меня есть класс AbstractUser, от которого наследуются три класса: User, UserProfile и Player.
Изначально встал вопрос: отправлять ли на клиент тот же самый класс User (и занулять пароли\сделать transient), либо создать отдельный класс и отправлять его. Выбрал второй вариант, т.к. в UserProfile добавляются еще дополнительно два свойства: находится ли игрок в сети и был ли приглашён в игру. И конструктор в классе UserProfile не могу сделать, потому что в общая библиотека и клиент даже не знают о существовании класса User. А засорять класс User мне тоже как-то не захотелось.
Возможно стоило бы создать отдельный класс Converter...

Аналогично у меня сделан класс Player, которому добавились чисто игровые свойства: цвет игрока, очки, оставшееся время на ход и пр. Также благодаря этому можно знать какое состояние было у игрока на начало игры (для истории).

Сообщение от dzrkot

Ну не совсем логично раскидано по пакетам, это моё субъективное мнение.
...
5) какие-то вещи повторяются в разных классах, т.е. их вообще следует вынести в отдельный класс, или если java8 то мб интерфейс с default поведением

А как их лучше раскидать?))

@dzrkot · 16.05.2016, 10:38

Сообщение от wikkk

И конструктор в классе UserProfile не могу сделать, потому что в общая библиотека и клиент даже не знают о существовании класса User.

не понял... вы же на сервере создаёте UserProfile... и вам ничего не мешает создать констурктор или статический метод-инициализатор в userProfile, который принимает User и читает всё необходимое... Если это уц вас только в 1 месте встречается, и вы ничего не планируете менять, то можно конечно и оставить, но практика показывает, что такие вещи встречаются не в 1 месте, и засорять такими вызовами логику не стоит, лучше создать конструктор или метод...

Сообщение от wikkk

А как их лучше раскидать?))

чтобы ответить, мне надо изучить проект, я не особо его смотрел... но пакеты game , users , network внутри model для меня не очень понятны...

@wikkk · 16.05.2016, 11:19 **[ТС]**

Сообщение от dzrkot

не понял... вы же на сервере создаёте UserProfile... и вам ничего не мешает создать констурктор или статический метод-инициализатор в userProfile, который принимает User и читает всё необходимое... Если это уц вас только в 1 месте встречается, и вы ничего не планируете менять, то можно конечно и оставить, но практика показывает, что такие вещи встречаются не в 1 месте, и засорять такими вызовами логику не стоит, лучше создать конструктор или метод...

Создаю на сервере, да, но сам класс UserProfile находится в проекте NetChessCommon. И эта общая библиотека ничего не знает о классе User - т.к. класс User находится в проекте NetChessServer. Это чтобы клиент ничего не знал о классе User и его туда случайно нельзя было передать.

Новые блоги и статьи Все статьи Все блоги /
Первый деплой lagorue 16.01.2026 Не спеша развернул своё 1ое приложение в kubernetes. А дальше мне интересно создать 1фронтэнд приложения и 2 бэкэнд приложения развернуть 2 деплоя в кубере получится 2 сервиса и что-бы они. . .	Расчёт переходных процессов в цепи постоянного тока igorrr37 16.01.2026 / * Дана цепь постоянного тока с R, L, C, k(ключ), U, E, J. Программа составляет систему уравнений по 1 и 2 законам Кирхгофа, решает её и находит токи на L и напряжения на C в установ. режимах до и. . .	Восстановить юзерскрипты Greasemonkey из бэкапа браузера damix 15.01.2026 Если восстановить из бэкапа профиль Firefox после переустановки винды, то список юзерскриптов в Greasemonkey будет пустым. Но восстановить их можно так. Для этого понадобится консольная утилита. . .	Изучаю kubernetes lagorue 13.01.2026 А пригодятся-ли мне знания kubernetes в России?
Сукцессия микоризы: основная теория в виде двух уравнений. anaschu 11.01.2026 https:/ / rutube. ru/ video/ 7a537f578d808e67a3c6fd818a44a5c4/	WordPad для Windows 11 Jel 10.01.2026 WordPad для Windows 11 — это приложение, которое восстанавливает классический текстовый редактор WordPad в операционной системе Windows 11. После того как Microsoft исключила WordPad из. . .	Classic Notepad for Windows 11 Jel 10.01.2026 Old Classic Notepad for Windows 11 Приложение для Windows 11, позволяющее пользователям вернуть классическую версию текстового редактора «Блокнот» из Windows 10. Программа предоставляет более. . .	Почему дизайн решает? Neotwalker 09.01.2026 В современном мире, где конкуренция за внимание потребителя достигла пика, дизайн становится мощным инструментом для успеха бренда. Это не просто красивый внешний вид продукта или сайта — это. . .

@wikkk 1 / 1 / 1 Регистрация: 26.09.2015 Сообщений: 40

	Оцените написанную игру 19.04.2016, 09:16. Показов 1031. Ответов 9 Метки нет (Все метки) Добрый день, написал игру - сетевые шахматы между игроками. Выложил на github, было бы круто узнать что скажут хорошие программисты, особенно по архитектуре и пр. моментам) Игра: https://github.com/n-demidov/NetChess 0

@Aassians 12 / 4 / 0 Регистрация: 23.11.2014 Сообщений: 390
	25.04.2016, 05:33
	ты всё сам писал или использовал готовый движок? Добавлено через 2 минуты где изображения фигурок взял? 0

@wikkk 1 / 1 / 1 Регистрация: 26.09.2015 Сообщений: 40
	25.04.2016, 11:05 [ТС]
	всё сам, ну по Netty - смотрел обучалки, искал примеры, по JavaFX также, чё-то там на форумах пару раз спрашивал + гуглил вопросы, попутно продолжал обучаться. Архитектуру\логику было интересно самому написать, а потом посмотреть как сделали другие. Вопросов потому что куча) Но руки так и не дошли посмотреть как в других приложениях. Картинки с какого-то шахматного приложения на гитхабе взял Добавлено через 2 часа 50 минут Но шахматного бота я не писал - у меня только игроки могут играть друг с другом по сети. 0

@Aassians 12 / 4 / 0 Регистрация: 23.11.2014 Сообщений: 390
	25.04.2016, 17:39
	видел на андроиде, как много шахмат около сотни. 0

@turbanoff 4094 / 3828 / 745 Регистрация: 18.05.2010 Сообщений: 9,331 Записей в блоге: 12
	25.04.2016, 18:39
	wikkk, сделайте мултимодульный проект, вместо трёх отдельных. Там будет намного проще смотреть. 0