![]() |
|
Модераторы: LSD, AntonSaburov |
![]()
|
|
| Platon |
|
|||
![]() Эксперт ![]() ![]() ![]() Профиль Группа: Завсегдатай Сообщений: 1801 Регистрация: 25.4.2006 Репутация: 16 Всего: 40 |
Опять видимо сказал неявно...
Надумал то надумал, но видимо неправильно... |
|||
|
||||
| Shaggie |
|
|||
![]() Опытный ![]() ![]() Профиль Группа: Завсегдатай Сообщений: 570 Регистрация: 21.12.2006 Где: outer space Репутация: 4 Всего: 72 |
Platon, классные шахматы! Красивые, умеют думать (не особо сильно, но откровенных ляпов не допускают и моими косяками завсегда пользуются), особенно подсветка движения фигур при шахованом короле порадовала.
Но позволю дать один совет - остановись пока! Тебе стоит причесать код, прежде чем двигаться дальше. Я его просмотрел достаточно бегло, но многое само бросается в глаза. Вот например:
Ты и сам всё прекрасно понимаешь, отделил код заметными комментариями. Осталось немногое - раскидать эти куски по отдельным методам, чтобы каждый выполнял какое-то цельное действие. Логика отдельно, представление отдельно, сеть отдельно. Это и к main() относится! У одного хорошего программиста есть замечание о том, как определить качество программы/библиотеки. Первый способ - медленный. Посчитать магические числа и строки (кстати их у тебя тоже немало Почему бы не воспользоваться enumerations? В классе ChessBoard, например, они прямо просятся в код вместо интовых костант. Это всё не так важно для работоспособности программы Если хочешь создать увлекательные шахматы (имеется в виду вменяемый компьютерный оппонент), рекомендую к прочтению книгу "Программирование шахмат и других логических игр", автор Евгений Корнилов, издательство "Санкт-Петербугр", 2005. |
|||
|
||||
| Platon |
|
||||
![]() Эксперт ![]() ![]() ![]() Профиль Группа: Завсегдатай Сообщений: 1801 Регистрация: 25.4.2006 Репутация: 16 Всего: 40 |
Это что такое? Константы? или повторяющиеся куски кода?
Ну, потому что, ими работать не удобно.
У меня проект в UTF-8 если что. Абсолютно согласен, с этой целью и выложил исходники, чтоб проконсультироваться по поводу оформления кода. Не особо сильно, потому что глубина просмотра 2 хода. на 3 уже туго думает. |
||||
|
|||||
| Shaggie |
|
|||
![]() Опытный ![]() ![]() Профиль Группа: Завсегдатай Сообщений: 570 Регистрация: 21.12.2006 Где: outer space Репутация: 4 Всего: 72 |
||||
|
||||
| Platon |
|
|||
![]() Эксперт ![]() ![]() ![]() Профиль Группа: Завсегдатай Сообщений: 1801 Регистрация: 25.4.2006 Репутация: 16 Всего: 40 |
Хм, ну у меня вроде почти так реализовано, просто в main метод и вызываемых им методах идет сборка приложения с определенными параметрами, ничего умного придумать не могу. А магические числа, тоже интересно, я к сожалению не могу понять какие константы к ним отнести. |
|||
|
||||
![]()
|
| Правила форума "Java" | |
|
|
Если Вам помогли, и атмосфера форума Вам понравилась, то заходите к нам чаще! С уважением, LSD, AntonSaburov, powerOn, tux, javastic. |
| 0 Пользователей читают эту тему (0 Гостей и 0 Скрытых Пользователей) | |
| 0 Пользователей: | |
| « Предыдущая тема | Java: Общие вопросы | Следующая тема » |
|
|
По вопросам размещения рекламы пишите на vladimir(sobaka)vingrad.ru
Отказ от ответственности Powered by Invision Power Board(R) 1.3 © 2003 IPS, Inc. |