[2024/4] refactor to use Grid.find_all() - it does the row/col enumer…

…ation and position building already

2024/4/script.py

@@ -38,27 +38,16 @@ def main():
     grid = Grid(aoc.read_lines())
 
     xmas_count = 0
-    for y, row in enumerate(grid):
-        for x, cell in enumerate(row):
-            if cell == 'X':
-                xmas_count += search_xmas(grid, Vec2(x, y))
-            else:
-                continue
+    for pos in grid.find_all('X'):
+        xmas_count += search_xmas(grid, Vec2(pos))
     print("p1:", xmas_count)
 
     masx_count = 0
-    for y, row in enumerate(grid):
-        # skip the top and bottom rows
-        if y in [0, grid.height() - 1]:
+    for pos in grid.find_all('A'):
+        # skip As at the edge of the grid
+        if pos[0] in [0, grid.width() - 1] or pos[1] in [0, grid.height() - 1]:
             continue
-        for x, cell in enumerate(row):
-            # skip the left and right columns
-            if x in [0, grid.width() - 1]:
-                continue
-            if cell == 'A':
-                masx_count += search_masx(grid, Vec2(x, y))
-            else:
-                continue
+        masx_count += search_masx(grid, Vec2(pos))
     print("p2:", masx_count)
 
 

README.md

@@ -3,10 +3,10 @@ My solutions to the yearly Advents of Code
 
 <!-- AOC TILES BEGIN -->
 <h1 align="center">
-  Advent of Code - 168/458 ⭐
+  Advent of Code - 169/458 ⭐
 </h1>
 <h1 align="center">
-  2024 - 7 ⭐ - Python
+  2024 - 8 ⭐ - Python
 </h1>
 <a href="2024/1/script.py">
   <img src=".aoc_tiles/tiles/2024/01.png" width="161px">