Feature/improve chess piece

.gitignore

@@ -112,4 +112,5 @@ buildNumber.properties
 # JDT-specific (Eclipse Java Development Tools)
 .classpath
 
+.idea
 # End of https://www.toptal.com/developers/gitignore/api/java,maven,eclipse

pom.xml

@@ -100,19 +100,6 @@
                     <artifactId>maven-project-info-reports-plugin</artifactId>
                     <version>3.0.0</version>
                 </plugin>
-
-                <plugin>
-                    <groupId>org.pitest</groupId>
-                    <artifactId>pitest-maven</artifactId>
-                    <version>1.15.2</version>
-                    <dependencies>
-                        <dependency>
-                            <groupId>org.pitest</groupId>
-                            <artifactId>pitest-junit5-plugin</artifactId>
-                            <version>1.2.1</version>
-                        </dependency>
-                    </dependencies>
-                </plugin>
             </plugins>
         </pluginManagement>
     </build>

src/main/java/com/rogeriofrsouza/app/Program.java

@@ -1,17 +1,17 @@
 package com.rogeriofrsouza.app;
 
-import com.rogeriofrsouza.app.chess.ChessException;
-import com.rogeriofrsouza.app.chess.ChessMatch;
-import com.rogeriofrsouza.app.chess.ChessPiece;
-import com.rogeriofrsouza.app.chess.ChessPosition;
-
 import java.util.ArrayList;
 import java.util.InputMismatchException;
 import java.util.List;
 import java.util.Locale;
 import java.util.Optional;
 import java.util.Scanner;
 
+import com.rogeriofrsouza.app.chess.ChessException;
+import com.rogeriofrsouza.app.chess.ChessMatch;
+import com.rogeriofrsouza.app.chess.ChessPiece;
+import com.rogeriofrsouza.app.chess.ChessPosition;
+
 public class Program {
 
     public static void main(String[] args) {
@@ -42,14 +42,14 @@ public static void main(String[] args) {
 
                 if (chessMatch.getPromoted() != null) {
                     System.out.print("Enter piece for promotion (B/N/R/Q): ");
-                    String type = scanner.nextLine().substring(0, 1).toUpperCase();
+                    ChessPiece.Name pieceName = ui.readPromotedPiece(scanner);
 
-                    while (!List.of("B", "N", "R", "Q").contains(type)) {
+                    while (pieceName == null) {
                         System.err.print("Invalid value! Enter piece for promotion (B/N/R/Q): ");
-                        type = scanner.nextLine().substring(0, 1).toUpperCase();
+                        pieceName = ui.readPromotedPiece(scanner);
                     }
 
-                    chessMatch.replacePromotedPiece(type);
+                    chessMatch.replacePromotedPiece(pieceName);
                 }
             } catch (ChessException | InputMismatchException exception) {
                 System.err.println(exception.getMessage());

src/main/java/com/rogeriofrsouza/app/UI.java

@@ -3,12 +3,10 @@
 import com.rogeriofrsouza.app.chess.ChessMatch;
 import com.rogeriofrsouza.app.chess.ChessPiece;
 import com.rogeriofrsouza.app.chess.ChessPosition;
-import com.rogeriofrsouza.app.chess.Color;
 
 import java.util.InputMismatchException;
 import java.util.List;
 import java.util.Scanner;
-import java.util.stream.Collectors;
 
 public class UI {
 
@@ -30,21 +28,18 @@ public void clearScreen() {
     public void printMatch(ChessMatch chessMatch, List<ChessPiece> captured) {
         printBoard(chessMatch.getPieces(), null);
 
-        List<ChessPiece> white =
-                captured.stream()
-                        .filter(piece -> piece.getColor() == Color.WHITE)
-                        .collect(Collectors.toList());
+        List<ChessPiece> white = captured.stream()
+            .filter(piece -> piece.getColor() == ChessPiece.Color.WHITE)
+            .toList();
 
-        List<ChessPiece> black =
-                captured.stream()
-                        .filter(piece -> piece.getColor() == Color.BLACK)
-                        .collect(Collectors.toList());
+        List<ChessPiece> black = captured.stream()
+            .filter(piece -> piece.getColor() == ChessPiece.Color.BLACK)
+            .toList();
 
         System.out.println("\nCaptured pieces");
 
-        System.out.printf(
-                "White: %s%s%n%sBlack: %s%s%n%s",
-                ANSI_WHITE, white, ANSI_RESET, ANSI_YELLOW, black, ANSI_RESET);
+        System.out.printf("White: %s%s%n%sBlack: %s%s%n%s",
+            ANSI_WHITE, white, ANSI_RESET, ANSI_YELLOW, black, ANSI_RESET);
 
         System.out.println("\nTurn: " + chessMatch.getTurn());
 
@@ -64,7 +59,7 @@ public void printBoard(ChessPiece[][] pieces, boolean[][] possibleMoves) {
         StringBuilder stringBuilder = new StringBuilder();
 
         for (int i = 0; i < pieces.length; i++) {
-            stringBuilder.append((8 - i) + " ");
+            stringBuilder.append((8 - i)).append(" ");
 
             for (int j = 0; j < pieces[i].length; j++) {
                 if (possibleMoves != null && possibleMoves[i][j]) {
@@ -74,9 +69,10 @@ public void printBoard(ChessPiece[][] pieces, boolean[][] possibleMoves) {
                 if (pieces[i][j] == null) {
                     stringBuilder.append("-");
                 } else {
-                    String color =
-                            pieces[i][j].getColor() == Color.WHITE ? ANSI_WHITE : ANSI_YELLOW;
-                    stringBuilder.append(color + pieces[i][j]);
+                    String color = pieces[i][j].getColor() == ChessPiece.Color.WHITE
+                        ? ANSI_WHITE
+                        : ANSI_YELLOW;
+                    stringBuilder.append(color).append(pieces[i][j]);
                 }
 
                 stringBuilder.append(ANSI_RESET + " ");
@@ -86,20 +82,40 @@ public void printBoard(ChessPiece[][] pieces, boolean[][] possibleMoves) {
         }
 
         stringBuilder.append("  a b c d e f g h");
-        System.out.println(stringBuilder.toString());
+        System.out.println(stringBuilder);
     }
 
-    public ChessPosition readChessPosition(Scanner scanner) {
+    private String readInput(Scanner scanner) {
         String input = scanner.nextLine();
 
+        if (input == null || input.trim().isBlank()) {
+            throw new InputMismatchException("Error reading input: cannot be null or blank.");
+        }
+
+        return input.trim();
+    }
+
+    public ChessPosition readChessPosition(Scanner scanner) {
+        String input = readInput(scanner);
+
         if (!input.matches("[a-h][1-8]")) {
             throw new InputMismatchException(
-                    "Error reading ChessPosition. Valid values are from a1 to h8.");
+                "Error reading ChessPosition. Valid values are from a1 to h8.");
         }
 
         char column = input.charAt(0);
         int row = Character.getNumericValue(input.charAt(1));
 
         return new ChessPosition(column, row);
     }
+
+    public ChessPiece.Name readPromotedPiece(Scanner scanner) {
+        String input = readInput(scanner).substring(0, 1);
+
+        return ChessMatch.possiblePromotedPieces
+            .stream()
+            .filter(pieceName -> pieceName.getLetter().equalsIgnoreCase(input))
+            .findFirst()
+            .orElse(null);
+    }
 }

src/main/java/com/rogeriofrsouza/app/boardgame/Piece.java

@@ -1,45 +1,31 @@
 package com.rogeriofrsouza.app.boardgame;
 
 import lombok.EqualsAndHashCode;
+import lombok.Getter;
 
+@Getter
 @EqualsAndHashCode
 public abstract class Piece {
 
-    // Não é ainda a posição do xadrez, é uma posição simples de matriz. Não deve ser visível na
-    // camada de xadrez
     protected Position position;
+    private final Board board;
 
-    private Board board;
-
-    // A posição de uma peça recém criada será nula, pois ainda não foi colocada no tabuleiro
-    public Piece(Board board) {
+    protected Piece(Board board) {
         this.board = board;
-        // position = null;
-    }
-
-    // Somente classes do mesmo pacote e subclasses podem acessar o Board
-    protected Board getBoard() {
-        return board;
     }
 
-    // Não permitir que o Board seja alterado. Remover o setBoard()
-
-    public abstract boolean[][] possibleMoves();
+    public abstract boolean[][] computePossibleMoves();
 
-    /*
-     * Hook method: método que faz um gancho com a subclasse.
-     * Chama uma possível implementação de alguma subclasse concreta da classe Piece
-     */
-    public boolean possibleMove(Position position) {
-        return possibleMoves()[position.getRow()][position.getColumn()];
+    public boolean isTargetPossibleMove(Position target) {
+        return computePossibleMoves()[target.getRow()][target.getColumn()];
     }
 
     public boolean isThereAnyPossibleMove() {
-        boolean[][] mat = possibleMoves();
+        boolean[][] possibleMoves = computePossibleMoves();
 
-        for (int i = 0; i < mat.length; i++) {
-            for (int j = 0; j < mat.length; j++) {
-                if (mat[i][j]) {
+        for (boolean[] row : possibleMoves) {
+            for (boolean move : row) {
+                if (move) {
                     return true;
                 }
             }