0

我目前正在尝试构建一个简单的棋子移动程序。我从文本文件中读取了国际象棋运动符号。国际象棋符号是这样的：

plb1 c3 = (p)pawn(l)白棋从位置 b1(b1) 移动到位置 c3(c3)。然后我的正则表达式解析移动。

在代码中，我匹配不同的部分并打印出它们的各种动作。我想要做的是将moveBetter()方法分解为多个较小的方法，例如findPiece()或findColor()方法。

问题是，每次我尝试分解方法时，它们都不起作用，或者我被告知我必须使方法静态并且进一步破坏它。如何将 moveBetter() 拆分为多个方法？

import java.io.IOException;
import java.util.regex.Matcher;
import java.util.regex.Pattern;

public class SuperiorFileIO {

public static void main(String[] args) throws IOException {
    moveBetter();
}

public static void moveBetter() throws IOException
{
    String piece = " "; //group1
    String color = " "; //group2
    String position = " "; //group3
    String position2 = " "; //group4
    String capture = " "; //group5
    //or 
    String kingPos1 = "";//6
    String kingPos2 = "";//7
    String rookPos1 = "";//8
    String rookPos2 = "";//9

    ReadFile readFile = new ReadFile();     
    String input = readFile.readTextFile("TestMoves");


    Pattern matchingMoves = Pattern.compile("(q|k|b|p|n|r+)(l|d)(\\w\\d) ?(\\w\\d)?([*])?|(\\w\\d) (\\w\\d) (\\w\\d) (\\w\\d)");
    Matcher m1 = matchingMoves.matcher(input);

    while (m1.find())
    {
        //find piece
            if(m1.group(6) != null && m1.group(6).matches("(\\w\\d)")) //castling part 1
                kingPos1 = m1.group(6);
            else if (m1.group(1).equals("q"))
                piece = "Queen";
            else if (m1.group(1).equals("k"))
                piece = "King";
            else if (m1.group(1).equals("b"))
                piece = "Bishop";
            else if (m1.group(1).equals("p"))
                piece = "Pawn";
            else if (m1.group(1).equals("n"))
                piece = "Knight";
            else if (m1.group(1).matches("(q|k|b|p|n|r+)") &&  m1.group(1).equals("r"))
                piece = "Rook";
            else {
                System.out.println("PIECE FAILED");
                piece = "piece Failed";
            }
        //find color
            if(m1.group(7) != null && m1.group(7).matches("(\\w\\d)")) //castling part 2
                kingPos2 = m1.group(7);
            else if(m1.group(2).equals("d"))
                color = "Black";
            else if(m1.group(2).matches("(l|d)") && m1.group(2).equals("l"))
                color = "White";
            else {
                System.out.println("COLOR FAILED");
                piece = "color Failed";
            }
        //set starting position
            if (m1.group(7) != null && m1.group(7).matches("(\\w\\d)") )
            rookPos1 = m1.group(7); //castling part 3
            else
                position = m1.group(3);
        //Castling
            if (m1.group(8) != null && m1.group(9) != null && m1.group(8).matches("(\\w\\d)") && m1.group(9).matches("(\\w\\d)"))
            {
                rookPos1 = m1.group(8);
                rookPos2 = m1.group(9);
                System.out.println("The king moves from " + kingPos1 + " to " + kingPos2 + " while the rook moves from " + rookPos1 + " to " + rookPos2);   
            }
        //if the player does not move // finds a piece
            else if (m1.group(4) == null)
                System.out.println("The " + color + " " + piece + " resides at " + position);           

        //set ending position
            else if (m1.group(4) != null && m1.group(5) == null)
            {
                position2 = m1.group(4);
                System.out.println("The " + color + " " + piece + " moves from " + position + " to " + position2);
            }
        //set ending position and capture
            else if (m1.group(4) != null && m1.group(5) != null)
            {
                //System.out.println("capture occured");
                position2 = m1.group(4);
                capture = m1.group(5); //unused
                System.out.println("The " + color + " " + piece + " moves from " + position + " to " + position2  + " and captures the piece on " + position2);
            }

            else {
                System.out.println("PARSE ERROR");
            }
    }


}

}

4

3 回答 3

1

不要movePiece从您的静态main方法中调用。定义一个ChessEngine类或其他东西，在您的 main 方法中实例化一个，然后让该ChessEngine对象处理所有逻辑。然后，您可以按照其他答案/评论中的建议拆分您的方法。

于 2013-11-14T20:35:24.150 回答
0

你可以这样做：

public static void main(String[] args) throws IOException
{
    findPiece();
    findColor();
}

private static void findPiece()
{
    //Code
}

private static void findColor()
{
    //Code
}

方法可见性可以是私有的或公开的，具体取决于您要如何使用它，对于这些方法，我更喜欢私有。

找件（）：

while(m1.find())
{
    if(m1.group(6) != null && m1.group(6).matches("(\\w\\d)")) //castling part 1
        kingPos1 = m1.group(6);
    else if (m1.group(1).equals("q"))
        piece = "Queen";
    else if (m1.group(1).equals("k"))
        piece = "King";
    else if (m1.group(1).equals("b"))
        piece = "Bishop";
    else if (m1.group(1).equals("p"))
        piece = "Pawn";
    else if (m1.group(1).equals("n"))
        piece = "Knight";
    else if (m1.group(1).matches("(q|k|b|p|n|r+)") &&  m1.group(1).equals("r"))
        piece = "Rook";
    else {
        System.out.println("PIECE FAILED");
        piece = "piece Failed";
    }
}

其他方法几乎相同，循环和复制粘贴您要执行的部分。虽然如果每个方法中的代码在另一个方法中需要，那么拆分它不会简单地工作

于 2013-11-14T20:19:40.483 回答
0

考虑这个基本示例：

public static void moveBetter() throws IOException
{
    String piece = "pluto";
    piece = "goofy";
}

为了将其拆分为两种方法，您必须：

1.提取变量并将其放在moveBetter之外

string piece;

public static void moveBetter() throws IOException
{
    piece = "pluto";
    piece = "goofy";
}

2.define the variable as static since a static method (like moveBetter) can't access a non static variable or a non static method

static string piece;

3.create a the new method and move the code inside it. remember the new method must be static too in order to be accessed by moveBetter

string static piece;

public static void moveBetter() throws IOException
{
    piece = "pluto";
    assignGoofy();
}

public static void assignGoofy()
{
    piece = "goofy";
}
于 2013-11-14T20:37:59.347 回答