Let's set boundaries.

2 files changed, 36 insertions, 11 deletions

diff --git a/src/main.rs b/src/main.rs
index c8e3133..0351ffb 100644
--- a/src/main.rs
+++ b/src/main.rs
@@ -1,4 +1,5 @@
 extern crate crossterm;
+extern crate ignore_result;
 extern crate rand;
 extern crate text_io;
 
@@ -12,7 +13,8 @@ use crossterm::input::{input, InputEvent, KeyEvent};
 use crossterm::screen::{EnterAlternateScreen, LeaveAlternateScreen, RawScreen};
 use crossterm::terminal;
 use crossterm::{execute, Output};
-use entities::{Entity, Player};
+use entities::Player;
+use ignore_result::Ignore;
 use state::State;
 use std::env;
 use std::io::{stdout, Write};
@@ -59,15 +61,15 @@ fn main() {
                 InputEvent::Keyboard(KeyEvent::Char('?')) => {
                     execute!(stdout(), Output("q: quit")).unwrap()
                 }
-                InputEvent::Keyboard(KeyEvent::Char('j')) => state.player.move_by(DOWN).unwrap(),
-                InputEvent::Keyboard(KeyEvent::Char('k')) => state.player.move_by(UP).unwrap(),
-                InputEvent::Keyboard(KeyEvent::Char('h')) => state.player.move_by(LEFT).unwrap(),
-                InputEvent::Keyboard(KeyEvent::Char('l')) => state.player.move_by(RIGHT).unwrap(),
+                InputEvent::Keyboard(KeyEvent::Char('j')) => state.move_player(DOWN).ignore(),
+                InputEvent::Keyboard(KeyEvent::Char('k')) => state.move_player(UP).ignore(),
+                InputEvent::Keyboard(KeyEvent::Char('h')) => state.move_player(LEFT).ignore(),
+                InputEvent::Keyboard(KeyEvent::Char('l')) => state.move_player(RIGHT).ignore(),
                 // Arrow keys for noobs
-                InputEvent::Keyboard(KeyEvent::Down) => state.player.move_by(DOWN).unwrap(),
-                InputEvent::Keyboard(KeyEvent::Up) => state.player.move_by(UP).unwrap(),
-                InputEvent::Keyboard(KeyEvent::Left) => state.player.move_by(LEFT).unwrap(),
-                InputEvent::Keyboard(KeyEvent::Right) => state.player.move_by(RIGHT).unwrap(),
+                InputEvent::Keyboard(KeyEvent::Down) => state.move_player(DOWN).ignore(),
+                InputEvent::Keyboard(KeyEvent::Up) => state.move_player(UP).ignore(),
+                InputEvent::Keyboard(KeyEvent::Left) => state.move_player(LEFT).ignore(),
+                InputEvent::Keyboard(KeyEvent::Right) => state.move_player(RIGHT).ignore(),
                 _ => (),
             }
         }
diff --git a/src/state.rs b/src/state.rs
index 68d5250..d377784 100644
--- a/src/state.rs
+++ b/src/state.rs
@@ -3,8 +3,8 @@ use crossterm::{queue, Output};
 use std::io::{stdout, Write};
 
 use crate::entities::{Character, Entity, Player};
-use crate::tiling::{tile_to_str, TileGrid};
-use crate::world::{Dungeon, Generatable, Level};
+use crate::tiling::{tile_to_str, TileGrid, TileType};
+use crate::world::{apply_movement, Dungeon, Generatable, Level, Movement};
 
 pub struct State {
     pub player: Character,
@@ -89,4 +89,27 @@ impl State {
     pub fn current_level(&self) -> &Level {
         &self.dungeon.levels[self.level]
     }
+
+    fn can_step_on(tile: &TileType) -> bool {
+        match tile {
+            TileType::Floor => true,
+            TileType::StairsDown => true,
+            TileType::StairsUp => true,
+            _ => false,
+        }
+    }
+
+    pub fn move_player(&mut self, dir: Movement) -> Result<(), String> {
+        match &self.grid {
+            None => Err(String::from("No level loaded!")),
+            Some(grid) => {
+                let loc = apply_movement(*self.player.location(), dir)?;
+                // Is the new location colliding with anything?
+                if !State::can_step_on(grid.block_at(loc.0, loc.1)) {
+                    return Err(String::from("Can't move entity!"));
+                }
+                self.player.move_by(dir)
+            }
+        }
+    }
 }