refactor(06/2024): improve variable naming

manhunto · Dec 7, 2024 · 919c2ca · 919c2ca
1 parent ff97520
commit 919c2ca
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 5 deletions.
diff --git a/src/solutions/year2024/day06.rs b/src/solutions/year2024/day06.rs
@@ -16,10 +16,10 @@ impl Solution for Day06 {
         let grid: Grid<char> = Grid::from(input);
 
         let obstructions = grid.get_all_positions(&OBSTRUCTION);
-        let guard = grid.get_first_position(&STARTING_POSITION).unwrap();
+        let starting_point = grid.get_first_position(&STARTING_POSITION).unwrap();
         let surface = grid.surface_range();
 
-        let mut guard = Vector::new(guard, North);
+        let mut guard = Vector::new(starting_point, North);
         let mut visited_positions: HashSet<Point> = HashSet::new();
 
         while surface.contains(guard.position()) {
@@ -35,10 +35,10 @@ impl Solution for Day06 {
         let grid: Grid<char> = Grid::from(input);
 
         let obstructions = grid.get_all_positions(&OBSTRUCTION);
-        let guard = grid.get_first_position(&STARTING_POSITION).unwrap();
+        let starting_point = grid.get_first_position(&STARTING_POSITION).unwrap();
         let surface = grid.surface_range();
 
-        let mut guard = Vector::new(guard, North);
+        let mut guard = Vector::new(starting_point, North);
         guard = self.next_step(guard, &obstructions); // skip initial position
 
         let mut loop_count: u32 = 0;

diff --git a/src/utils/range.rs b/src/utils/range.rs
@@ -104,7 +104,7 @@ mod tests {
     use crate::utils::range::Range;
 
     #[test]
-    fn is_in_range() {
+    fn contains() {
         let range = Range::new(5, 7).unwrap();
 
         assert!(!range.contains(4));