fixed some issues, now showing legal moves of

selected piece

.github/workflows/automatic-prerelease.yml

@@ -17,3 +17,4 @@ jobs:
           repo_token: "${{ secrets.GITHUB_TOKEN }}"
           prerelease: true
           automatic_release_tag: "latest"
+          title: "Development Build"

src/logic/board.py

@@ -16,8 +16,8 @@ class Board:
             pos_w_pawn = Position(x, 1)
             pos_b_pawn = Position(x, 6)
 
-            self._white[pos_w_pawn] = Pawn(pos_w_pawn)
-            self._black[pos_b_pawn] = Pawn(pos_b_pawn)
+            self._white[pos_w_pawn] = Pawn(pos_w_pawn, Piece.WHITE)
+            self._black[pos_b_pawn] = Pawn(pos_b_pawn, Piece.BLACK)
 
             pos_w_piece = Position(x, 0)
             pos_b_piece = Position(x, 7)
@@ -34,9 +34,19 @@ class Board:
             elif x == 4:
                 piece = King
             assert piece != None, f"Didn't know which piece to assign for {x = }"
-            self._white[pos_w_piece] = piece(pos_w_piece)
-            self._black[pos_b_piece] = piece(pos_b_piece)
+            self._white[pos_w_piece] = piece(pos_w_piece, Piece.WHITE)
+            self._black[pos_b_piece] = piece(pos_b_piece, Piece.BLACK)
 
+    def piece_at(self, x: int, y: int) -> Piece | None:
+        pos = Position(x, y)
+        white_piece = self._white.get(pos, None)
+        black_piece = self._black.get(pos, None)
+
+        assert white_piece == None or black_piece == None, f"There are two pieces at the same position {pos}, this shouldn't happen!"
+
+        if white_piece != None:
+            return white_piece
+        return black_piece
 
 def create_board():
     return Board()

src/logic/pieces/pawn.py

@@ -1,7 +1,31 @@
-from .piece import Piece
+from logic.position import Position
+from logic.pieces.piece import Piece
 
 class Pawn(Piece):
-    def __init__(self, pos) -> None:
-        super().__init__(pos)
-        self.already_moved = False
+    def legal_moves(self, board) -> list[Position]:
+        ret = []
 
+        # can we capture to the left?
+        if self.pos.x > 0 and (
+            (self.colour == self.WHITE and (capturable_piece := board.piece_at(self.pos.x - 1, self.pos.y + 1)))
+            or
+            (self.colour == self.BLACK and (capturable_piece := board.piece_at(self.pos.x - 1, self.pos.y - 1)))
+        ):
+            if capturable_piece.colour != self.colour:
+                ret.append(capturable_piece.pos)
+
+        # can we capture to the right?
+        if self.pos.x < 7 and (
+            (self.colour == self.WHITE and (capturable_piece := board.piece_at(self.pos.x + 1, self.pos.y + 1)))
+            or
+            (self.colour == self.BLACK and (capturable_piece := board.piece_at(self.pos.x + 1, self.pos.y - 1)))
+        ):
+            if capturable_piece.colour != self.colour:
+                ret.append(capturable_piece.pos)
+
+        for dy in range(1, 3 if self.pos.y == 1 else 2):
+            if self.pos.y + dy > 7 or board.piece_at(self.pos.x, self.pos.y + dy):
+                break
+            ret.append(Position(self.pos.x, self.pos.y + dy))
+
+        return ret

src/logic/pieces/piece.py

@@ -2,8 +2,13 @@ from logic.position import Position
 
 
 class Piece:
-    def __init__(self, pos) -> None:
+    WHITE = "white"
+    BLACK = "black"
+
+    def __init__(self, pos, colour) -> None:
         self.pos = pos
+        assert colour == self.WHITE or colour == self.BLACK, "The colour of the piece must be either Piece.WHITE or Piece.BLACK"
+        self.colour = colour
 
     def position(self) -> Position:
         return self.pos

src/main.py

@@ -1,9 +1,10 @@
 from logic.board import create_board
+from view.gui import GUI
 from view.tui import TUI
 
 if __name__ == "__main__":
     board = create_board()
 
-    view = TUI(board)
+    view = GUI(board)
 
     view.show()

src/view/gui.py

@@ -0,0 +1,94 @@
+import tkinter as tk
+
+from logic.board import Board
+from logic.pieces.piece import Piece
+from logic.position import Position
+from view.view import View
+
+class GUI(View):
+    def __init__(self, board: Board) -> None:
+        super().__init__(board)
+
+        self.root = tk.Tk()
+        self.root.title("Chess Board")
+
+        self.tile_size = 80
+        board_size = self.tile_size * 8
+
+        self.canvas = tk.Canvas(self.root, width=board_size, height=board_size)
+        self.canvas.pack()
+
+        self.state = {"selected_piece": None, "legal_moves": []}
+
+        self.canvas.bind("<Button-1>", self._on_canvas_click)
+        self._draw_chess_board()
+
+
+    def _draw_chess_board(self):
+        colours = ["#F0D9B5", "#B58863"]  # Light and dark squares
+
+        for y in range(8):
+            for x in range(8):
+                colour = colours[(x + y) % 2]
+                if self.state["selected_piece"] and Position(x, 7-y) in self.state["legal_moves"]:
+                    colour = "#ADD8E6"  # Highlight legal moves
+
+                self.canvas.create_rectangle(
+                    x * self.tile_size, 
+                    y * self.tile_size, 
+                    (x + 1) * self.tile_size, 
+                    (y + 1) * self.tile_size, 
+                    fill=colour
+                )
+
+                piece = self.board.piece_at(x, 7-y)
+
+                if piece:
+                    text_colour = "white" if piece.colour == Piece.WHITE else "black"
+                    self.canvas.create_text(
+                        (x + 0.5) * self.tile_size, 
+                        (y + 0.5) * self.tile_size, 
+                        text=piece.__class__.__name__[0], 
+                        fill=text_colour, 
+                        font=("Arial", 32, "bold")
+                    )
+
+                # Cell annotations
+                text_colour = colours[(x + y + 1) % 2] # the other colour
+
+                if x == 0: # numbers in the top left of the first column
+                    self.canvas.create_text(
+                        (x + 0.15) * self.tile_size, 
+                        (y + 0.15) * self.tile_size, 
+                        text=8-y, 
+                        fill=text_colour, 
+                        font=("Arial", 10, "bold")
+                    )
+                if y == 7: # numbers in the top left of the first column
+                    self.canvas.create_text(
+                        (x + 0.85) * self.tile_size, 
+                        (y + 0.85) * self.tile_size, 
+                        text="abcdefgh"[x], 
+                        fill=text_colour, 
+                        font=("Arial", 10, "bold")
+                    )
+                    
+
+    def _on_canvas_click(self, event):
+        x, y = event.x // self.tile_size, event.y // self.tile_size
+        y = 7 - y
+        piece = self.board.piece_at(x, y)
+        print(f"Clicked on {x, y}, {piece = }")
+
+        if piece:
+            self.state["selected_piece"] = piece
+            self.state["legal_moves"] = piece.legal_moves(self.board)
+        else:
+            self.state["selected_piece"] = None
+            self.state["legal_moves"] = []
+
+        self.canvas.delete("all")
+        self._draw_chess_board()
+
+    def show(self) -> None:
+        self.root.mainloop()