azatnv / ruscheckers Goto Github PK

1. RusCheckers/src/main/MainController.java

   Lines 22 to 45 in 553f065

   	public ImageView b1;
   	public ImageView b2;
   	public ImageView b3;
   	public ImageView b4;
   	public ImageView b5;
   	public ImageView b6;
   	public ImageView b7;
   	public ImageView b8;
   	public ImageView b9;
   	public ImageView b10;
   	public ImageView b11;
   	public ImageView b12;
   	public ImageView w1;
   	public ImageView w2;
   	public ImageView w3;
   	public ImageView w4;
   	public ImageView w5;
   	public ImageView w6;
   	public ImageView w7;
   	public ImageView w8;
   	public ImageView w9;
   	public ImageView w10;
   	public ImageView w11;
   	public ImageView w12;

Такого кода быть точно не должно. Подумайте, как сделать нормально.

2. RusCheckers/src/main/MainController.java

   Lines 61 to 62 in 553f065

   	private static int[][] board = new int[8][8]; //[X][Y]
   	private static int[][] boardCopy = new int[8][8]; //[X][Y]

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

3. Постарайтесь избавиться от дублирования кода, хотя бы там, где об этом предупреждает IDE.

4. Магические числа лучше заменить константами и перечислениями.

5. Нет проверки обязательного взятия (вернее, его начала).

6. Длинное взятие прерывается, если шашка превратилась в дамку.

1. То, что в классе Functions как-то совсем не соответствует объектно-ориентированному стилю. Сделайте лучше класс, представляющий поле и методы на нём.
2. Во всём коде куча магических чисел (8, 7, 3). Вынесите их в статические константы с осмысленными именами.
3. В findAllPossibleMoves местами копипаста.
4. От десятков одинаковых ImageView тоже лучше бы избавиться.
5. Cells представляет и тип ячейки на поле и текущего игрока, что не очень хорошо.
6. curDepth в createNodes вычисляется неверно. Там получается не текущая глубина, а просто число рассматриваемых ходов. На мысль об этом наводит следующее: поставьте depth в 2000 и запустите бота. Он работает почти так же, как и с depth == 3, то есть получается какой-то ultimate решатель. Посмотрите на position в дебаггере при первом вызове miniMax и увидите в чём, косяк.
7. Почти вся польза от альфа-бета отсечения убивается за счёт того, что изначально вы строите дерево, ничего не отсекая, а отсечение используете только при оценке позиций.