Исправление логики движения в Java

Question

Исправление логики движения в Java

1

В настоящее время я разрабатываю шахматную игру в java, используя Eclipse IDE,

У меня есть все перемещение кусочков отдельно от Пешки, что является самым сложным, потому что Пешка должна быть способна совершать разные движения

Пешка должна иметь возможность перемещаться дважды в начале, а затем только один после этого

В настоящее время я заставил пешку двигаться только дважды, но я застрял на том, чтобы заставить остальную логику работать

Я работал с идеей выражения if/else

Я мог бы использовать некоторую помощь, написав его

Вот код до сих пор для Пешки, и я включил комментарий для вашего использования

Обновление до проблемы только с черной пешкой не движется вправо Я был в состоянии установить белый один правильный путь, но не черный Я не знаю, почему он не работает

 //Pawn Movement
    private boolean isValidPawnMove(int sourceRow, int sourceColumn, int targetRow, int targetColumn) {

        boolean isValid = false;
        if( isTargetLocationFree() ){
            if( sourceColumn == targetColumn){
                if(  sourcePiece.getColor() == Piece.COLOR_WHITE ){

                    // White Pawn
                    if( sourceRow+1 == targetRow || sourceRow == 1 && targetRow == 3){//Pawns can move to at the start then only 1 after that 
                        isValid = true;
                    }else{
                        isValid = false;
                    }
                }
                else{

                    // Black Pawn
                    if( sourceRow-1 == targetRow || sourceRow == -1 && targetRow == -3){
                        isValid = true;
                    }else{
                        isValid =  false;
                    }
                }
            }else{

                //If you try to move left or right into a different Column 
                isValid = false;

            }

        //Take square occupied by an opponents piece, which is diagonally in front 
        }else if( isTargetLocationCaptureable() ){

            if( sourceColumn+1 == targetColumn || sourceColumn-1 == targetColumn){
                //One column to the right or left
                if(  sourcePiece.getColor() == Piece.COLOR_WHITE ){
                    //White Piece 
                    if( sourceRow+1 == targetRow ){
                        //Move one up
                        isValid = true;
                    }else{
                        //Not moving one up
                        isValid = false;
                    }
                }else{
                    //Black Piece
                    if( sourceRow-1 == targetRow ){
                        //Move one down
                        isValid = true;
                    }else{
                        //Not moving one down
                        isValid = false;
                    }
                }
            }else{
                //Not one column to the left or right
                isValid = false;
            }
        }
        return isValid;
    }

Спасибо за любую помощь, которую вы можете предоставить

ClarkPamler93 25 нояб. 2014, в 23:56

Источник

Теги:

java

logic

jframe

chess

3 ответа

0

В шахматах пешка может выбрать, хочет ли она идти на один или два квадрата вперед, если она никогда не двигалась и, следовательно, все еще стоит во втором ряду. Вы должны спросить, есть ли это на второй строке и на двух квадратах перед ней пустые и не только "целевое местоположение".

if( sourcePiece.getColor() == Piece.COLOR_WHITE ){

      // White Pawn
      if( sourceRow+1 == targetRow ){
          isValid = true;
      } else if (sourceRow+2 == targetRow && sourceRow == ROW_2) {
          if ((isFreeSquare(sourceColumn+1) && isFreeSquare(sourceColumn+2)) {
             isValid = true;
          } else {
              isValid = false;
          }
      } else {
          isValid = false;
      }
  }
  else{

      // Black Pawn
      ...
  }

Вы можете оставить код isFreeSquare (sourceColumn + 2), потому что вы уже запросили его с помощью isTargetLocationFree(). Для черного вы должны спросить, находится ли пешка на ROW_7.

TheSlater 28 нояб. 2014, в 12:16

0

У меня есть вопрос о свободном доступе. Можете ли вы показать мне, где я мог бы добавить этот метод в настоящее время, если я пытаюсь добавить его, я получаю две ошибки, а также у меня проблема с && sourceRow == ROW_2
ClarkPamler93 28 нояб. 2014, в 21:27
0

Если ROW_2 не отображается, вы можете написать Piece.ROW_2 или просто «1» без кавычек. У вас есть метод isTargetLocationFree (). Может быть, вы можете расширить его, чтобы вернуть, если данный квадрат свободен, и передать координаты в качестве аргументов.
TheSlater 29 нояб. 2014, в 09:32

0

В вашем классе для пешек вы можете иметь экземпляр boolean, скажем,

boolean hasMoved

и изначально сделать это ложным. Если это логическое значение ложно, тогда пешка может перемещать один или два блока. Всякий раз, когда вы перемещаете пешку, вы проверяете это логическое значение, и если оно ложно, установите его значение true после его перемещения. Это должно сработать.

kirbyquerby 25 нояб. 2014, в 21:25

0

Будет ли счетчик более практичным?
MadProgrammer 25 нояб. 2014, в 23:32
0

Нет нужды. Это одноразовая вещь, и вам придется использовать логическое выражение, если в любом случае вы используете счетчик (который вы все равно увеличиваете только один раз). Кроме того, учитывая, что логические значения занимают гораздо меньше места, чем числа, логические значения определенно лучше.
kirbyquerby 26 нояб. 2014, в 00:17
0

Я думал, что это должно было бы увеличиться вдвое, но я не играю в шахматы, так что я знаю - ваш ответ, тем не менее, на правильном пути;)
MadProgrammer 26 нояб. 2014, в 00:19
0

Я знаю, что мне нужно логическое значение, чтобы пометить каждую перемещенную пешку, и мне нужно добавить это в мой текущий код. Можете ли вы помочь мне с этим?
ClarkPamler93 26 нояб. 2014, в 01:27
0

Все, что вы нам показываете, это код, который сообщает, разрешено ли пешке двигаться ... Я предполагал, что вы использовали класс для фигур, которые вы бы легко могли добавить в логическое значение, но если нет, то что Вы используете, чтобы представлять каждый кусок?
kirbyquerby 26 нояб. 2014, в 02:01
0

да, и приведенный выше код также позволяет установить ход пешки, как вы можете видеть из кода выше, а значение awn в настоящий момент установлено равным 1. Мне нужна помощь в написании кода, позволяющего ему сдвигаться дважды в начале, а затем один раз после этого. должно быть сделано в этом классе выше?
ClarkPamler93 26 нояб. 2014, в 23:39
0

Создайте логическое значение, указывающее, находится ли пешка в исходном положении. Если это так, то вы можете вернуть true, если позиция находится в пределах 1 или 2 единиц фигуры. В противном случае ограничьте пособие до 1 единицы.
kirbyquerby 27 нояб. 2014, в 01:22
0

Я обновил свой вопрос, добавив больше информации и кода для настройки частей. Надеюсь, что вы можете помочь с развитием событий, так как я до сих пор не могу
ClarkPamler93 27 нояб. 2014, в 13:02
0

Позвольте мне переформулировать. Разрешить ему перемещать 2 пробела тогда и только тогда, когда его текущая строка является строкой, с которой он начинается. Это то, что я пытался сказать в последнем комментарии. Или, если вы сделаете метод установки для своего поля типа, вы можете легко разделить пешку на два типа; перемещать и не двигать, и использовать это значение, чтобы определить, сколько он может двигаться.
kirbyquerby 27 нояб. 2014, в 22:14
0

Но как ? единственный код, который вы мне показали, это логическое значение
ClarkPamler93 27 нояб. 2014, в 23:20

Показать ещё 8 комментариев

Ещё вопросы

У меня есть вопрос о свободном доступе. Можете ли вы показать мне, где я мог бы добавить этот метод в настоящее время, если я пытаюсь добавить его, я получаю две ошибки, а также у меня проблема с && sourceRow == ROW_2
Если ROW_2 не отображается, вы можете написать Piece.ROW_2 или просто «1» без кавычек. У вас есть метод isTargetLocationFree (). Может быть, вы можете расширить его, чтобы вернуть, если данный квадрат свободен, и передать координаты в качестве аргументов.
Будет ли счетчик более практичным?
Нет нужды. Это одноразовая вещь, и вам придется использовать логическое выражение, если в любом случае вы используете счетчик (который вы все равно увеличиваете только один раз). Кроме того, учитывая, что логические значения занимают гораздо меньше места, чем числа, логические значения определенно лучше.
Я думал, что это должно было бы увеличиться вдвое, но я не играю в шахматы, так что я знаю - ваш ответ, тем не менее, на правильном пути;)
Я знаю, что мне нужно логическое значение, чтобы пометить каждую перемещенную пешку, и мне нужно добавить это в мой текущий код. Можете ли вы помочь мне с этим?
Все, что вы нам показываете, это код, который сообщает, разрешено ли пешке двигаться ... Я предполагал, что вы использовали класс для фигур, которые вы бы легко могли добавить в логическое значение, но если нет, то что Вы используете, чтобы представлять каждый кусок?
да, и приведенный выше код также позволяет установить ход пешки, как вы можете видеть из кода выше, а значение awn в настоящий момент установлено равным 1. Мне нужна помощь в написании кода, позволяющего ему сдвигаться дважды в начале, а затем один раз после этого. должно быть сделано в этом классе выше?
Создайте логическое значение, указывающее, находится ли пешка в исходном положении. Если это так, то вы можете вернуть true, если позиция находится в пределах 1 или 2 единиц фигуры. В противном случае ограничьте пособие до 1 единицы.
Я обновил свой вопрос, добавив больше информации и кода для настройки частей. Надеюсь, что вы можете помочь с развитием событий, так как я до сих пор не могу
Позвольте мне переформулировать. Разрешить ему перемещать 2 пробела тогда и только тогда, когда его текущая строка является строкой, с которой он начинается. Это то, что я пытался сказать в последнем комментарии. Или, если вы сделаете метод установки для своего поля типа, вы можете легко разделить пешку на два типа; перемещать и не двигать, и использовать это значение, чтобы определить, сколько он может двигаться.
Но как ? единственный код, который вы мне показали, это логическое значение

BarrySW19 · Accepted Answer · 2014-11-28T11-48-00.000Z

Я думаю, что самым простым решением является явная проверка исходных и целевых строк, поскольку белые пешки могут перемещать только два вперед со второго ранга, поэтому ваша логика становится (для белого):

if( sourceRow+1 == targetRow || sourceRow == 2 && targetRow == 4) {

Очевидно, вам также нужно будет проверить, что (sourceColumn, 3) также пуст.

Я добавил это, но в пешечном движении ничего не изменилось, я что-то упустил?
Индексирование строк начинается с 0 или 1? Этот код предполагает 1. Вы можете проверить, как мой шахматный движок выполняет генерацию пешек на github.com/BarrySW19/CalculonX/tree/master/src/main/java/nl/…
Был в состоянии исправить это просто пришлось настроить его, чтобы начать с 0, а не 1 спасибо за помощь, единственная проблема, это не относится к черной пешке?
Обновление получило исправление еще раз спасибо за вашу помощь