From 1be71bf2033cf5bae3aef83aa8c5465faf583844 Mon Sep 17 00:00:00 2001 From: Karma Riuk Date: Fri, 31 Jan 2025 18:27:44 +0100 Subject: [PATCH] fixed giga spelling mistake (it's castling right, not castling write) --- src/logic/board.py | 40 ++++++++++++++++++++-------------------- src/logic/pieces/king.py | 8 ++++---- 2 files changed, 24 insertions(+), 24 deletions(-) diff --git a/src/logic/board.py b/src/logic/board.py index 167cbe9..105c8ca 100644 --- a/src/logic/board.py +++ b/src/logic/board.py @@ -15,8 +15,8 @@ class Board: self._white: dict[Position, Piece] = {} self._black: dict[Position, Piece] = {} self._turn = None - self._white_castling_write = set() - self._black_castling_write = set() + self._white_castling_rights = set() + self._black_castling_rights = set() self._en_passant_target = None @staticmethod @@ -88,13 +88,13 @@ class Board: sides = "kq" assert c in sides or c in sides.upper(), f"The FEN position is malformed, the castling rights should be either k or q (both either lower- or upper-case), instead is '{c}'" if c == "K": - ret._white_castling_write.add(CastleSide.King) + ret._white_castling_rights.add(CastleSide.King) if c == "Q": - ret._white_castling_write.add(CastleSide.Queen) + ret._white_castling_rights.add(CastleSide.Queen) if c == "k": - ret._black_castling_write.add(CastleSide.King) + ret._black_castling_rights.add(CastleSide.King) if c == "q": - ret._black_castling_write.add(CastleSide.Queen) + ret._black_castling_rights.add(CastleSide.Queen) # -- En passant target if position[index] != "-": @@ -152,8 +152,8 @@ class Board: return False return True - def castling_writes_for(self, colour: Colour) -> set[CastleSide]: - return self._white_castling_write if colour == Colour.WHITE else self._black_castling_write + def castling_rights_for(self, colour: Colour) -> set[CastleSide]: + return self._white_castling_rights if colour == Colour.WHITE else self._black_castling_rights def make_move(self, move: Move) -> "Board": dest_piece = self.piece_at(move.pos.x, move.pos.y) @@ -166,8 +166,8 @@ class Board: ret._white = self._white.copy() ret._black = self._black.copy() ret._turn = Colour.WHITE if self._turn == Colour.BLACK else Colour.BLACK - ret._white_castling_write = self._white_castling_write.copy() - ret._black_castling_write = self._black_castling_write.copy() + ret._white_castling_rights = self._white_castling_rights.copy() + ret._black_castling_rights = self._black_castling_rights.copy() piece = move.piece @@ -201,22 +201,22 @@ class Board: # -- Check for castling rights if piece.colour == Colour.WHITE: if type(piece) == King: - ret._white_castling_write = set() + ret._white_castling_rights = set() if type(piece) == Rook: - if piece.pos.x == 0 and CastleSide.Queen in ret._white_castling_write: - ret._white_castling_write.remove(CastleSide.Queen) - elif piece.pos.x == 7 and CastleSide.King in ret._white_castling_write: - ret._white_castling_write.remove(CastleSide.King) + if piece.pos.x == 0 and CastleSide.Queen in ret._white_castling_rights: + ret._white_castling_rights.remove(CastleSide.Queen) + elif piece.pos.x == 7 and CastleSide.King in ret._white_castling_rights: + ret._white_castling_rights.remove(CastleSide.King) else: if type(piece) == King: - ret._black_castling_write = set() + ret._black_castling_rights = set() if type(piece) == Rook: - if piece.pos.x == 0 and CastleSide.Queen in ret._black_castling_write: - ret._black_castling_write.remove(CastleSide.Queen) - elif piece.pos.x == 7 and CastleSide.King in ret._black_castling_write: - ret._black_castling_write.remove(CastleSide.King) + if piece.pos.x == 0 and CastleSide.Queen in ret._black_castling_rights: + ret._black_castling_rights.remove(CastleSide.Queen) + elif piece.pos.x == 7 and CastleSide.King in ret._black_castling_rights: + ret._black_castling_rights.remove(CastleSide.King) return ret diff --git a/src/logic/pieces/king.py b/src/logic/pieces/king.py index 006090c..be478e3 100644 --- a/src/logic/pieces/king.py +++ b/src/logic/pieces/king.py @@ -23,11 +23,11 @@ class King(Piece): return self.keep_only_blocking(ret, board) # -- Castles - castling_writes = board.castling_writes_for(self.colour) - if len(castling_writes) == 0: + castling_rights = board.castling_rights_for(self.colour) + if len(castling_rights) == 0: return ret - if CastleSide.King in castling_writes: + if CastleSide.King in castling_rights: clear = True for dx in range(1, 3): x = self.pos.x + dx @@ -45,7 +45,7 @@ class King(Piece): if clear: ret.append(Move(self, Position(6, self.pos.y), castle_side=CastleSide.King)) - if CastleSide.Queen in castling_writes: + if CastleSide.Queen in castling_rights: clear = True for dx in range(1, 3): x = self.pos.x - dx