Сделал шахматы: прошу критики

@kirbornu · Регистрация: 24.12.2021

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

Я - начинающий программист-самоучка, программа в архиве - моя первая более-менее серьезная работа, и я хочу чтобы опытные люди указали мне на мои ошибки и, может быть, даже дали советы насчёт развития моего навыка. Я не стал реализовывать все тонкости правил шахмат, как например то, что короля нельзя рубить и ему нельзя подставляться под удар. Спустя 5 дней(суммарно примерно 12 часов) работы я решил остановиться на этом и теперь прошу критики.

P.S. Заранее большое спасибо

@Ципихович Эндрю · 24.12.2021, 10:43

ну пешка с начальной позиции через одну, одна может сходить на 2 шага вперёд, другая нет
и виснут, долго вызывал Контрол+Альт+Делит, чтобы убить процесс

@Arsegg · 24.12.2021, 11:06

kirbornu, бегло посмотрел код:
1) Покури про паттерн MVC - отдели логику работы движка от их отображения и взаимодействия с пользователем;
2) Избавься от тонны if'ов в методах и замени, например, на перебор возможных ходов через массивы;
3) Ну и поработай с правилами шахмат: поставил черному королю двойной шах и повесил программу - черные вместо того, чтобы спасать короля, ходят "левыми" фигурами, обрекая черного монарха на смерть.

enx · 24.12.2021, 11:08

Не по теме:

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

обрекая черного монарха на смерть.

у них хитрый план

Логика плохая, все остальное по делу сказали.

@Welemir1 · 24.12.2021, 11:12

kirbornu, работа проведена большая, молодец! С медом закончено, перейдем к дегтю
- почитать про ф-строки и использовать только их
- не использовать глобал
- почитать про аннотации типов
- почитать про ООП
- почитать про гит и следующий проект выкладывать не архивом, а ссылкой
- не стесняйся создавать много модулей, программа не должна быть в одном

@Михалыч · 24.12.2021, 14:34

kirbornu, здорово!

я бы еще, в дополнение к вышесказанному, добавил бы про документацию (документ строка). Пока проект не большой, то вроде все в голове, а потом не поймешь что за что отвечает... Сначала кажется зачем столько времени на это убивать, но потом это себя окупает.

Новые блоги и статьи Все статьи Все блоги /
Символьное дифференцирование igorrr37 13.02.2026 / * Программа принимает математическое выражение в виде строки и выдаёт его производную в виде строки и вычисляет значение производной при заданном х Логарифм записывается как: (x-2)log(x^2+2) -. . .	Камера Toupcam IUA500KMA Eddy_Em 12.02.2026 Т. к. у всяких "хикроботов" слишком уж мелкий пиксель, для подсмотра в ESPriF они вообще плохо годятся: уже 14 величину можно рассмотреть еле-еле лишь на экспозициях под 3 секунды (а то и больше),. . .	И ясному Солнцу zbw 12.02.2026 И ясному Солнцу, и светлой Луне. В мире покоя нет и люди не могут жить в тишине. А жить им немного лет.	«Знание-Сила» zbw 12.02.2026 «Знание-Сила» «Время-Деньги» «Деньги -Пуля»
SDL3 для Web (WebAssembly): Подключение Box2D v3, физика и отрисовка коллайдеров 8Observer8 12.02.2026 Содержание блога Box2D - это библиотека для 2D физики для анимаций и игр. С её помощью можно определять были ли коллизии между конкретными объектами и вызывать обработчики событий столкновения. . . .	SDL3 для Web (WebAssembly): Загрузка PNG с прозрачным фоном с помощью SDL_LoadPNG (без SDL3_image) 8Observer8 11.02.2026 Содержание блога Библиотека SDL3 содержит встроенные инструменты для базовой работы с изображениями - без использования библиотеки SDL3_image. Пошагово создадим проект для загрузки изображения. . .	SDL3 для Web (WebAssembly): Загрузка PNG с прозрачным фоном с помощью SDL3_image 8Observer8 10.02.2026 Содержание блога Библиотека SDL3_image содержит инструменты для расширенной работы с изображениями. Пошагово создадим проект для загрузки изображения формата PNG с альфа-каналом (с прозрачным. . .	Установка Qt-версии Lazarus IDE в Debian Trixie Xfce volvo 10.02.2026 В общем, достали меня глюки IDE Лазаруса, собранной с использованием набора виджетов Gtk2 (конкретно: если набирать текст в редакторе и вызвать подсказку через Ctrl+Space, то после закрытия окошка. . .

@Ципихович Эндрю 1712 / 579 / 76 Регистрация: 10.04.2009 Сообщений: 9,323
	24.12.2021, 10:43
	ну пешка с начальной позиции через одну, одна может сходить на 2 шага вперёд, другая нет и виснут, долго вызывал Контрол+Альт+Делит, чтобы убить процесс 0

@Arsegg 3582 / 2182 / 571 Регистрация: 02.09.2015 Сообщений: 5,510
	24.12.2021, 11:06
	kirbornu, бегло посмотрел код: 1) Покури про паттерн MVC - отдели логику работы движка от их отображения и взаимодействия с пользователем; 2) Избавься от тонны if'ов в методах и замени, например, на перебор возможных ходов через массивы; 3) Ну и поработай с правилами шахмат: поставил черному королю двойной шах и повесил программу - черные вместо того, чтобы спасать короля, ходят "левыми" фигурами, обрекая черного монарха на смерть. 1

@Welemir1 Автоматизируй это! 7391 / 4818 / 1246 Регистрация: 30.03.2015 Сообщений: 13,687 Записей в блоге: 29
	24.12.2021, 11:12
	Сообщение было отмечено kirbornu как решение Решение kirbornu, работа проведена большая, молодец! С медом закончено, перейдем к дегтю - почитать про ф-строки и использовать только их - не использовать глобал - почитать про аннотации типов - почитать про ООП - почитать про гит и следующий проект выкладывать не архивом, а ссылкой - не стесняйся создавать много модулей, программа не должна быть в одном 2

@Михалыч 1011 / 355 / 59 Регистрация: 28.02.2013 Сообщений: 937
	24.12.2021, 14:34
	kirbornu, здорово! я бы еще, в дополнение к вышесказанному, добавил бы про документацию (документ строка). Пока проект не большой, то вроде все в голове, а потом не поймешь что за что отвечает... Сначала кажется зачем столько времени на это убивать, но потом это себя окупает. 1

Сделал шахматы: прошу критики

Решение