Update README.MD

Reviewed-on: #4

README.MD

@@ -14,6 +14,7 @@ A game server for sharing gamestate of azul boards
   * [x] Gjøre trekk
     * [x] strict
     * [x] loose
+    * [x] validate
     * [ ] random
   * [ ] Serialization/Deserialization regler - serde tags
   * [ ] HTTP Stack

src/azul.rs

@@ -255,14 +255,7 @@ impl GameState {
             return Err(MoveErr::Other("Game is over"));
         }
 
-        self.do_move(game_move.clone()).map_err(|e| e)?;
-        match game_move.policy {
-            //if validate return ok and do not iterate 
-            Policy::Validate => return Ok(()),
-            //else continue the function
-            _ => (),
-        }
-
+        self.do_move(game_move).map_err(|e| e)?;
 
         if self.is_empty() {
             self.score_unchecked().map_err(|e| MoveErr::Other(e))?;
@@ -292,18 +285,30 @@ impl GameState {
         match game_move.policy {
             Policy::Strict => self.do_move_strict(game_move),
             Policy::Loose => self.do_move_loose(game_move),
-            Policy::Validate => self.do_move_validate(game_move),
         }
     }
 
+    /// Does a move, if the move is placed where it can't fit, places it on the floor
+    fn do_move_loose(&mut self, game_move: GameMove) -> Result<(), MoveErr> {
+        match self.do_move_strict(game_move.clone()) {
+            Ok(()) => Ok(()),
+            Err(MoveErr::Dst(_)) => {
+                let mut new_game_move = game_move;
+                new_game_move.destination = Destination::Floor;
+                self.do_move_strict(new_game_move)
+            }
+            e => e,
+        }
+    }
 
-
-    /// Validates a move, errors out if the move is weird in any way
-    fn do_move_validate(&mut self, game_move: GameMove) -> Result<(), MoveErr> {
+    /// Does a move, errors out if the move is weird in any way
+    fn do_move_strict(&mut self, game_move: GameMove) -> Result<(), MoveErr> {
         if self.current_player != game_move.player {
             return Err(MoveErr::Player("Not this player's turn"));
         }
 
+        // let player = &mut self.players[current_player];
+
         let color = game_move.color;
         let src = game_move.source;
         let dst = game_move.destination;
@@ -342,32 +347,6 @@ impl GameState {
                     "That pattern line and color is already filled on the wall",
                 ))
             }
-            _ => Ok(()),
-        }
-    }
-
-    /// Does a move, if the move is placed where it can't fit, places it on the floor
-    fn do_move_loose(&mut self, game_move: GameMove) -> Result<(), MoveErr> {
-        match self.do_move_strict(game_move.clone()) {
-            Ok(()) => Ok(()),
-            Err(MoveErr::Dst(_)) => {
-                let mut new_game_move = game_move;
-                new_game_move.destination = Destination::Floor;
-                self.do_move_strict(new_game_move)
-            }
-            e => e,
-        }
-    }
-
-    /// Does a move, errors out if the move is weird in any way
-    fn do_move_strict(&mut self, game_move: GameMove) -> Result<(), MoveErr> {
-        self.do_move_validate(game_move.clone())?;
-        
-        let color = game_move.color;
-        let src = game_move.source;
-        let dst = game_move.destination;
-
-        match (color, src, dst) {
             (c, s, Destination::PatternLine(p)) => {
                 let amount = self.take_tiles(s, c);
 
@@ -399,7 +378,6 @@ impl GameState {
                     .add_color(c, amount as isize)
                     .map_err(|e| MoveErr::Other(e))?;
             }
-            _ => (),
         };
 
         self.current_player = self
@@ -892,14 +870,9 @@ enum Policy {
     /// Anything weird will return an error
     #[serde(rename = "strict")]
     Strict,
-    /// If you place tiles wrongly, they will fall on the floor, anything else will return an error
     #[serde(rename = "loose")]
+    /// If you place tiles wrongly, they will fall on the floor, anything else will return an error
     Loose,
-
-    /// Will validate the move, but not execute it
-    #[serde(rename = "validate")]
-    Validate,
-
     // Anything weird will instead play a random move
     // #[serde(rename = "random")]
     // Random,