szymon/checkers
1
0
Fork 0
mirror of https://github.com/szymon-jozef/checkers.git synced 2026-10-27 19:20:44 +01:00
Code Issues Packages Projects Releases Wiki Activity

refactor: remove position from pawn

Browse Source
This commit is contained in:
szymon
2026-08-12 13:49:02 +02:00
parent 80c7d734d2
commit be542d9b72
2 changed files with 8 additions and 13 deletions
Download Patch File Download Diff File Expand all files Collapse all files

13
src/logic/board.rs
View File

@@ -46,7 +46,7 @@ impl Board {
debug!("Placing new pawn at: {}, owned by: {}", position, player);
Field {
position,
pawn: Some(Pawn::new(position, player)),
pawn: Some(Pawn::new(player)),
}
} else {
debug!("Not placing any pawn at: {}", position);
@@ -96,10 +96,8 @@ impl Board {
// we can safely unwrap, because if there's not pawn function returns false earlier
let mut mowing_pawn: Pawn = self[from].pawn.take().unwrap();
if to.row == player.end_row || matches!(mowing_pawn.state, PawnState::Dame(_)) {
mowing_pawn.state = super::pawn::PawnState::Dame(to);
} else {
mowing_pawn.state = super::pawn::PawnState::Man(to);
if to.row == player.end_row {
mowing_pawn.state = super::pawn::PawnState::Dame;
}
self[to.row][to.column].pawn = Some(mowing_pawn);
@@ -178,10 +176,7 @@ mod tests {
assert!(test_board[row][column].pawn.is_some());
// we unwrap, because some value has to be here, if it's not then something is wrong
// beside we check for that in the assert above
assert!(
test_board[row][column].pawn.as_ref().unwrap().state
== PawnState::Man(Position { row, column })
);
assert!(test_board[row][column].pawn.as_ref().unwrap().state == PawnState::Man);
if row < 3 {
assert_eq!(

8
src/logic/pawn.rs
View File

@@ -4,8 +4,8 @@ use crate::logic::{player::Player, utils::Position};
#[derive(PartialEq, Eq, Debug)]
pub enum PawnState {
Man(Position),
Dame(Position),
Man,
Dame,
Captured,
}
@@ -17,9 +17,9 @@ pub struct Pawn {
}
impl Pawn {
pub fn new(start_pos: Position, owner: &Player) -> Self {
pub fn new(owner: &Player) -> Self {
Self {
state: PawnState::Man(start_pos),
state: PawnState::Man,
owner: owner.id,
}
}