bennysim / tictactoe

tictactoe

tictactoe's Issues

Usage of unwrap_or()

tictactoe/src/main.rs

Line 69 in 3e06cff

self.your_turn.is_some() && self.your_turn.unwrap()

Dalo by se použít hezčí zápis s unwrap_or(). Řádek 69 si to zaslouží 🤭.
self.your_turn.unwrap_or(false)

Usage of struct MyTurn

tictactoe/src/main.rs

Line 344 in 3e06cff

struct MyTurn {

Struktura byla vytvořena na serializaci, ale možná by nebylo špatné ji používat i v jiných funkcích jako vstupní parametr. Například make_one_turn() a resolve_opponent_turn() berou vstupní parametry x a y, představující právě MyTurn; ubyl by tak jeden vstupní parametr. I když by v takovém případě bylo lepší asi použít obecnější název Turn.

Doplnit doc stringy

Vyhnutí se is_err - unwrap

tictactoe/src/main.rs

Lines 396 to 403 in 3e06cff

	if x.is_err() || y.is_err() {
	return Err(CoordinatesError::InvalidFormat);
	}
	match convert_coords(x.unwrap(), y.unwrap()) {
	Some(coords) => Ok(coords),
	None => Err(CoordinatesError::InvalidValue),
	}

Tento blok kódu bych navrhoval nahradit tímto. Tato implementace má výhodu, že nepoužívá unwrap a není tedy závislá na předchozí kontrole is_err()

    match (x, y) {
        (Ok(x), Ok(y)) => convert_coords(x, y).ok_or(CoordinatesError::InvalidValue),
        (_, _) => Err(CoordinatesError::InvalidFormat)
    }

Napisat testy

Return from match

tictactoe/src/main.rs

Lines 422 to 433 in 3e06cff

	let x= match x {
	'A' => Some(0),
	'B' => Some(1),
	'C' => Some(2),
	_ => None,
	};
	if x.is_none() || !(1..=3).contains(&y) {
	None
	} else {
	Some((x.unwrap(), (y-1) as usize))
	}

Uvedený zápis je možné následně zjednodušit

    let x= match x {
        'A' => 0,
        'B' => 1,
        'C' => 2,
        _ => return None,
    };

    match (1..=3).contains(&y)  {
        true => Some((x, (y-1) as usize)),
        false => None
    }

Failed invariant of GameSession due to initiation outside of the constructor

tictactoe/src/main.rs

Line 59 in 3e06cff

fn is_initiated(&self) -> bool {

Volání make_opponent_turn() či make_my_turn() před initiate() by způsobilo nesprávné chování funkce is_initiated(). Pořadí volání operací není vynuceno, musí být hlídáno programátorem, a tím je porušen invariant objektu. Řešením by mohlo být vytvářet GameSession až v momentu, kdy ji mohu rovnou nainicializovat a udělat tuto operaci v konstruktoru objektu.

Využít Generic.

Bylo by vhodné kde je to možné použít Generic. Například nadefinovat Trait pro Gameboard a následně použít tuto konstrukci

trait Gameboard {}

fn play_game<T>(gameboard: T, ...)
where
  T: Gameboard
{}

Name of function is_corner()

tictactoe/src/tictactoe.rs

Line 140 in 3e06cff

fn is_corner(x: usize, y: usize) -> Option<Diagonal> {

Při volání funkce is_corner() by bylo intuitivní předpokládat, že dojde k navrácení booleovské proměnné vyjadřující, zda se jedná o rohové políčko. Navracení diagonály podle mě úplně neodpovídá názvu funkce, možná by bylo lepší zvolit název například get_diagonal_type() nebo něco podobného. Nicméně chápu myšlenku.

Parse to usize

tictactoe/src/main.rs

Line 394 in 3e06cff

let y = coords[1].parse::<u8>();

Proč se tu již rovnou neparsuje na usize, když se v další funkci následně u8 -> usize?

File main.rs and function main reorganization

Bylo by dobré pro zvýšení přehlednosti rozdělit soubor main.rs do jednotlivých logických celků. Funkce main() je také příliš rozsáhlá, například obsah cyklu loop by mohl jít do samostané funkce; což je obecně dobrá praktika pro cykly bez podmínky, jelikož by to teoreticky umožňovalo testování alespoň jedné iterace.