X-Git-Url: https://git.ctpug.org.za/?a=blobdiff_plain;f=naja%2Fplayer.py;h=ff253da35ea3113216240649fa4dfe8a0dbf5dab;hb=616e138c8fa95c96280bc367e908178737ca180b;hp=f51e83ef257fdcdc7fe03d0062e00ab0a7ba7cec;hpb=07752d7d9657dde58cb6678d7248f6aefe0907e6;p=naja.git

diff --git a/naja/player.py b/naja/player.py
index f51e83e..ff253da 100644
--- a/naja/player.py
+++ b/naja/player.py
@@ -69,26 +69,25 @@ class Player(object):
             'position': list(self.position),
         }
 
+    def get_adjacent_position(self, direction):
+        x, y = self.position
+        if direction == BITS.NORTH and y > 0:
+            return (x, y - 1)
+        elif direction == BITS.SOUTH and y < 4:
+            return (x, y + 1)
+        elif direction == BITS.EAST and x < 4:
+            return (x + 1, y)
+        elif direction == BITS.WEST and x > 0:
+            return (x - 1, y)
+        else:
+            # Not a legal space.
+            return None
+
     def move(self, direction):
         if not self.bits.check_bit(direction):
             return False
-        # TODO: Something cleaner than this.
-        x, y = self.position
-        if direction == BITS.NORTH:
-            if y > 0 and self.bits.check_bit(BITS.NORTH):
-                self.position = (x, y - 1)
-                return True
-        elif direction == BITS.SOUTH:
-            if y < 4 and self.bits.check_bit(BITS.SOUTH):
-                self.position = (x, y + 1)
-                return True
-        elif direction == BITS.EAST:
-            if x < 4 and self.bits.check_bit(BITS.EAST):
-                self.position = (x + 1, y)
-                return True
-        elif direction == BITS.WEST:
-            if x > 0 and self.bits.check_bit(BITS.WEST):
-                self.position = (x - 1, y)
-                return True
-
+        new_position = self.get_adjacent_position(direction)
+        if new_position is not None:
+            self.position = new_position
+            return True
         return False