Refactored "make actor" into Actor::new()

src/actor.rs

@@ -1,4 +1,5 @@
 use ggez::graphics;
+use ggez::Context;
 
 #[path = "board.rs"]
 mod board;
@@ -23,3 +24,13 @@ pub struct Actor {
     pub posn: ActorPosn,
     pub behaviour: u8, //index into the behaviours array
 }
+
+impl Actor {
+    pub fn new(ctx: &mut Context, img_path: &str, _x: u8, _y: u8, b: u8) -> Actor {
+        Actor {
+            image: graphics::Image::new(ctx, img_path).unwrap(),
+            posn: ActorPosn::BoardPosn { x: _x, y: _y },
+            behaviour: b,
+        }
+    }
+}

src/apr.rs

@@ -52,11 +52,11 @@ impl Apr {
         let mut actors: Vec<actor::Actor> = Vec::new();
         for i in 0..r {
             // white side
-            actors.push(Apr::make_actor(_ctx, "/pawn.png", i, 0, 0));
+            actors.push(actor::Actor::new(_ctx, "/pawn.png", i, 0, 0));
         }
         for i in 0..r {
             // black side
-            actors.push(Apr::make_actor(_ctx, "/pawn2.png", i, c - 1, 0));
+            actors.push(actor::Actor::new(_ctx, "/pawn2.png", i, c - 1, 0));
         }
         Ok(Apr {
             grid: make_grid(_ctx, r, c, board.board.as_slice())?,
@@ -70,14 +70,6 @@ impl Apr {
         })
     }
 
-    pub fn make_actor(ctx: &mut Context, img_path: &str, _x: u8, _y: u8, b: u8) -> actor::Actor {
-        actor::Actor {
-            image: graphics::Image::new(ctx, img_path).unwrap(),
-            posn: actor::ActorPosn::BoardPosn { x: _x, y: _y },
-            behaviour: b,
-        }
-    }
-
     pub fn set_colour(&mut self, r: usize, c: usize, col: u8) {
         //You should only modify self.board through this method.
         self.board.board[r][c] = col; //Making a getter for self.board would sadly break borrowing in the same way that closest_square_coords does.