promote gui

app/src/main/java/chess/io/CommandExecutor.java

@@ -1,6 +1,7 @@
 package chess.io;
 
 import chess.io.commands.MoveCommand;
+import chess.io.commands.PromoteCommand;
 import chess.model.Game;
 
 public class CommandExecutor {
@@ -18,12 +19,24 @@ public class CommandExecutor {
 		assert this.outputSystem != null : "No output system specified !";
 
 		CommandResult result = command.execute(this.game, this.outputSystem);
-		if (result == CommandResult.Moved) {
+		processResult(command, result);
-			this.game.checkPromotion(((MoveCommand) command).getMove().getFinish());
+		return result;
-			this.game.checkGameStatus();
+	}
+
+	private void processResult(Command command, CommandResult result) {
+		if (result != CommandResult.Moved)
+			return;
+
+		if (command instanceof MoveCommand move) {
+			boolean needsPromote = this.game.checkPromotion(move.getMove().getFinish());
+			if (this.game.checkGameStatus())
+				return;
+
+			if (!needsPromote)
+				this.game.switchPlayerTurn();
+		} else if (command instanceof PromoteCommand) {
 			this.game.switchPlayerTurn();
 		}
-		return result;
 	}
 
 	public void setOutputSystem(OutputSystem outputSystem) {
@@ -60,6 +73,4 @@ public class CommandExecutor {
 		this.game.OnPlayerTurn.connect(outputSystem::playerTurn);
 	}
 
-	
-
 }

app/src/main/java/chess/io/commands/MoveCommand.java

@@ -24,6 +24,10 @@ public class MoveCommand extends Command {
 	public CommandResult execute(Game game, OutputSystem outputSystem) {
 		final ChessBoard board = game.getBoard();
 
+		// we must promote the pending pawn before
+		if (game.pawnShouldBePromoted())
+			return CommandResult.NotAllowed;
+
 		Piece piece = board.pieceAt(move.getStart());
 		if (piece == null)
 			return CommandResult.NotAllowed;

app/src/main/java/chess/io/commands/NewGameCommand.java

@@ -19,33 +19,33 @@ public class NewGameCommand extends Command {
 		ChessBoard board = game.getBoard();
 
 		for (int i = 0; i < 8; i++) {
-			board.pieceComes(new Pawn(Color.Black), new Coordinate(i, 0));
+			board.pieceComes(new Pawn(Color.Black), new Coordinate(i, 1));
-			board.pieceComes(new Pawn(Color.White), new Coordinate(i, Coordinate.VALUE_MAX - 1));
+			board.pieceComes(new Pawn(Color.White), new Coordinate(i, Coordinate.VALUE_MAX - 2));
 		}
 
-		board.pieceComes(new Rook(Color.Black), new Coordinate(0, 1));
+		board.pieceComes(new Rook(Color.Black), new Coordinate(0, 0));
-		board.pieceComes(new Rook(Color.Black), new Coordinate(Coordinate.VALUE_MAX - 1, 1));
+		board.pieceComes(new Rook(Color.Black), new Coordinate(Coordinate.VALUE_MAX - 1, 0));
 
-		board.pieceComes(new Rook(Color.White), new Coordinate(0, Coordinate.VALUE_MAX - 2));
+		board.pieceComes(new Rook(Color.White), new Coordinate(0, Coordinate.VALUE_MAX - 1));
-		board.pieceComes(new Rook(Color.White), new Coordinate(Coordinate.VALUE_MAX - 1, Coordinate.VALUE_MAX - 2));
+		board.pieceComes(new Rook(Color.White), new Coordinate(Coordinate.VALUE_MAX - 1, Coordinate.VALUE_MAX - 1));
 
-		board.pieceComes(new Knight(Color.Black), new Coordinate(1, 1));
+		board.pieceComes(new Knight(Color.Black), new Coordinate(1, 0));
-		board.pieceComes(new Knight(Color.Black), new Coordinate(Coordinate.VALUE_MAX - 2, 1));
+		board.pieceComes(new Knight(Color.Black), new Coordinate(Coordinate.VALUE_MAX - 2, 0));
 
-		board.pieceComes(new Knight(Color.White), new Coordinate(1, Coordinate.VALUE_MAX - 2));
+		board.pieceComes(new Knight(Color.White), new Coordinate(1, Coordinate.VALUE_MAX - 1));
-		board.pieceComes(new Knight(Color.White), new Coordinate(Coordinate.VALUE_MAX - 2, Coordinate.VALUE_MAX - 2));
+		board.pieceComes(new Knight(Color.White), new Coordinate(Coordinate.VALUE_MAX - 2, Coordinate.VALUE_MAX - 1));
 
-		board.pieceComes(new Bishop(Color.Black), new Coordinate(2, 1));
+		board.pieceComes(new Bishop(Color.Black), new Coordinate(2, 0));
-		board.pieceComes(new Bishop(Color.Black), new Coordinate(Coordinate.VALUE_MAX - 3, 1));
+		board.pieceComes(new Bishop(Color.Black), new Coordinate(Coordinate.VALUE_MAX - 3, 0));
 
-		board.pieceComes(new Bishop(Color.White), new Coordinate(2, Coordinate.VALUE_MAX - 2));
+		board.pieceComes(new Bishop(Color.White), new Coordinate(2, Coordinate.VALUE_MAX - 1));
-		board.pieceComes(new Bishop(Color.White), new Coordinate(Coordinate.VALUE_MAX - 3, Coordinate.VALUE_MAX - 2));
+		board.pieceComes(new Bishop(Color.White), new Coordinate(Coordinate.VALUE_MAX - 3, Coordinate.VALUE_MAX - 1));
 
-		board.pieceComes(new Queen(Color.Black), new Coordinate(4, 1));
+		board.pieceComes(new Queen(Color.Black), new Coordinate(3, 0));
-		board.pieceComes(new King(Color.Black), new Coordinate(3, 1));
+		board.pieceComes(new King(Color.Black), new Coordinate(4, 0));
 
-		board.pieceComes(new Queen(Color.White), new Coordinate(4, Coordinate.VALUE_MAX - 2));
+		board.pieceComes(new Queen(Color.White), new Coordinate(3, Coordinate.VALUE_MAX - 1));
-		board.pieceComes(new King(Color.White), new Coordinate(3, Coordinate.VALUE_MAX - 2));
+		board.pieceComes(new King(Color.White), new Coordinate(4, Coordinate.VALUE_MAX - 1));
 
 		game.resetPlayerTurn();
 

app/src/main/java/chess/io/commands/PromoteCommand.java

@@ -49,7 +49,7 @@ public class PromoteCommand extends Command {
 		board.pieceLeaves(this.pieceCoords);
 		board.pieceComes(createPiece(this.promoteType, pawn.getColor()), this.pieceCoords);
 
-		return CommandResult.NotMoved;
+		return CommandResult.Moved;
 	}
 
 	private Piece createPiece(PromoteType promoteType, Color color) {

app/src/main/java/chess/model/Game.java

@@ -16,7 +16,6 @@ public class Game {
 	public final Signal0 OnMat = new Signal0();
 	public final Signal0 OnPat = new Signal0();
 
-
 	public Game(ChessBoard board) {
 		this.board = board;
 	}
@@ -31,6 +30,7 @@ public class Game {
 
 	public void resetPlayerTurn() {
 		this.playerTurn = Color.White;
+		this.OnPlayerTurn.emit(playerTurn);
 	}
 
 	public void switchPlayerTurn() {
@@ -38,23 +38,30 @@ public class Game {
 		this.OnPlayerTurn.emit(playerTurn);
 	}
 
-	public void checkPromotion(Coordinate pieceCoords) {
+	public boolean checkPromotion(Coordinate pieceCoords) {
 
 		Piece piece = getBoard().pieceAt(pieceCoords);
 
 		if (piece == null || !(piece instanceof Pawn))
-			return;
+			return false;
 
 		int destY = pieceCoords.getY();
 
 		int enemyLine = piece.getColor() == Color.White ? 0 : 7;
 
-		if (destY == enemyLine) {
+		if (destY != enemyLine)
-			OnPromote.emit(pieceCoords);
+			return false;
-		}
+
+		OnPromote.emit(pieceCoords);
+		return true;
+
 	}
 
-	public void checkGameStatus() {
+	/**
+	 * 
+	 * @return true if game should end
+	 */
+	public boolean checkGameStatus() {
 		final ChessBoard board = getBoard();
 
 		final Color enemy = Color.getEnemy(getPlayerTurn());
@@ -65,10 +72,30 @@ public class Game {
 			} else {
 				OnMat.emit();
 				OnWin.emit(getPlayerTurn());
+				return true;
 			}
 		} else if (!board.hasAllowedMoves(enemy)) {
 			OnPat.emit();
+			return true;
 		}
+		return false;
+	}
+
+	public boolean pawnShouldBePromoted() {
+		if (pawnShouldBePromoted(Color.White))
+			return true;
+		return pawnShouldBePromoted(Color.Black);
+	}
+
+	private boolean pawnShouldBePromoted(Color color) {
+		int enemyLineY = color == Color.White ? 0 : 7;
+
+		for (int x = 0; x < Coordinate.VALUE_MAX; x++) {
+			if(checkPromotion(new Coordinate(x, enemyLineY)))
+				return true;
+		}
+
+		return false;
 	}
 
 }

app/src/main/java/chess/simplerender/Window.java

@@ -8,7 +8,9 @@ import java.util.List;
 
 import javax.swing.JFrame;
 import javax.swing.JLabel;
+import javax.swing.JOptionPane;
 import javax.swing.JPanel;
+import javax.swing.SwingUtilities;
 
 import chess.io.Command;
 import chess.io.CommandExecutor;
@@ -154,27 +156,37 @@ public class Window extends JFrame implements OutputSystem {
 
 	@Override
 	public void winnerIs(chess.model.Color color) {
-		System.out.println("Victoire de " + color);
+		SwingUtilities.invokeLater(() -> {
+			JOptionPane.showMessageDialog(this, "Victoire de " + color);
+		});
 	}
 
 	@Override
 	public void kingIsInCheck() {
-		System.out.println("Échec !");
+		SwingUtilities.invokeLater(() -> {
+			JOptionPane.showMessageDialog(this, "Échec !");
+		});
 	}
 
 	@Override
 	public void kingIsInMat() {
-		System.out.println("Échec et mat !");
+		SwingUtilities.invokeLater(() -> {
+			JOptionPane.showMessageDialog(this, "Échec et mat !");
+		});
 	}
 
 	@Override
 	public void patSituation() {
-		System.out.println("Pat. Égalité !");
+		SwingUtilities.invokeLater(() -> {
+			JOptionPane.showMessageDialog(this, "Pat. Égalité !");
+		});
 	}
 
 	@Override
 	public void hasSurrendered(chess.model.Color color) {
-		System.out.println("Abandon de " + color);
+		SwingUtilities.invokeLater(() -> {
+			JOptionPane.showMessageDialog(this, "Abandon de " + color);
+		});
 	}
 
 	@Override
@@ -184,8 +196,42 @@ public class Window extends JFrame implements OutputSystem {
 
 	@Override
 	public void promotePawn(Coordinate pieceCoords) {
-		System.out.println("PROMOTE");
+		SwingUtilities.invokeLater(() -> {
-		sendCommand(new PromoteCommand(PromoteType.Queen, pieceCoords));
+			// String result = JOptionPane.showInputDialog(this, "Choisissez le type !");
+			// sendCommand(new PromoteCommand(PromoteType.Queen, pieceCoords));
+
+			String result = null;
+
+			Object[] possibilities = new Object[PromoteType.values().length];
+			int i = 0;
+			for (PromoteType type : PromoteType.values()) {
+				possibilities[i] = type.name();
+				i++;
+			}
+
+			while (result == null || result.isEmpty()) {
+				result = (String) JOptionPane.showInputDialog(
+						this,
+						"Choose the type of piece to upgrade the pawn",
+						"Promote Dialog",
+						JOptionPane.PLAIN_MESSAGE,
+						null,
+						possibilities,
+						possibilities[0]);
+			}
+
+			PromoteType choosedType = null;
+
+			for (PromoteType type : PromoteType.values()) {
+				if (type.name().equals(result)) {
+					choosedType = type;
+					break;
+				}
+			}
+
+			if (choosedType != null)
+				sendCommand(new PromoteCommand(choosedType, pieceCoords));
+		});
 	}
 
 }