chore: fmt + clippy

src/bin/01.rs

@@ -136,8 +136,7 @@ impl State {
     fn scan_steps(&mut self, steps: Vec<Step>) -> Vec<Pos> {
         steps
             .into_iter()
-            .map(|step| self.all_locations_towards(&step))
+            .flat_map(|step| self.all_locations_towards(&step))
-            .flatten()
             .collect()
     }
 }
@@ -158,13 +157,13 @@ fn parse_input(_input: &str) -> Vec<Step> {
     _input
         .to_string()
         .strip_suffix("\n")
-        .unwrap_or(&_input)
+        .unwrap_or(_input)
         .split(", ")
         .map(|s| {
             let (direction, value) = s.split_at(1);
-            let length: u32 = value
+            let length: u32 = value.parse().unwrap_or_else(|_| {
-                .parse()
+                panic!("{}", format!("invalid length! {:?}", value).to_string())
-                .expect(&format!("invalid length! {:?}", value).to_string());
+            });
             match direction {
                 "L" => Step(Turn::Left, length),
                 "R" => Step(Turn::Right, length),

src/bin/02.rs

@@ -29,25 +29,19 @@ impl PosP1 {
             _y if _y > 1 => 1,
             _ => y,
         };
-        PosP1 {x: x, y: y}
+        PosP1 { x, y }
     }
 }
 
 impl Keypad for PosP1 {
     fn init() -> PosP1 {
-        Self {
+        Self { x: 0, y: 0 }
-            x: 0,
-            y: 0,
-        }
     }
 
     fn to_digit(self) -> char {
-        let row_offset = (self.y * -1) + 1;
+        let row_offset = -self.y + 1;
         let col_offset = self.x + 1;
-        char::from_digit(
+        char::from_digit(((row_offset * 3) + col_offset + 1).try_into().unwrap(), 10).unwrap()
-            ((row_offset * 3) + col_offset + 1).try_into().unwrap(),
-            10
-        ).unwrap()
     }
 
     fn move_to(self, dir: &Dir) -> PosP1 {
@@ -61,11 +55,11 @@ impl Keypad for PosP1 {
 }
 
 const DISP_P2: [[char; 5]; 5] = [
-    ['0', '0', '1', '0', '0', ],
+    ['0', '0', '1', '0', '0'],
-    ['0', '2', '3', '4', '0', ],
+    ['0', '2', '3', '4', '0'],
-    ['5', '6', '7', '8', '9', ],
+    ['5', '6', '7', '8', '9'],
-    ['0', 'A', 'B', 'C', '0', ],
+    ['0', 'A', 'B', 'C', '0'],
-    ['0', '0', 'D', '0', '0', ],
+    ['0', '0', 'D', '0', '0'],
 ];
 
 #[derive(Debug, Copy, Clone)]
@@ -77,12 +71,9 @@ struct PosP2 {
 impl PosP2 {
     fn from(row: usize, col: usize) -> Option<Self> {
         if row > 4 || col > 4 {
-            return None
+            return None;
         }
-        let res = PosP2 {
+        let res = PosP2 { row, col };
-            row: row,
-            col: col,
-        };
         if res.to_digit() == '0' {
             None
         } else {
@@ -91,13 +82,9 @@ impl PosP2 {
     }
 }
 
-
 impl Keypad for PosP2 {
     fn init() -> PosP2 {
-        Self {
+        Self { row: 2, col: 0 }
-            row: 2,
-            col: 0
-        }
     }
 
     fn to_digit(self) -> char {
@@ -122,21 +109,18 @@ enum Dir {
     Right,
 }
 
-fn compute_digit<T: Keypad>(input: T, path: &Vec<Dir>) -> T {
+fn compute_digit<T: Keypad>(input: T, path: &[Dir]) -> T {
-    path
+    path.iter().fold(input, |pos, dir| pos.move_to(dir))
-        .into_iter()
-        .fold(input, |pos, dir| pos.move_to(dir))
 }
 
-
 fn parse_input(input: &str) -> Vec<Vec<Dir>> {
     input
         .to_string()
         .strip_suffix("\n")
-        .unwrap_or(&input)
+        .unwrap_or(input)
         .split("\n")
-        .map(|s| s
+        .map(|s| {
-            .chars()
+            s.chars()
                 .map(|c| match c {
                     'U' => Dir::Up,
                     'D' => Dir::Down,
@@ -145,20 +129,23 @@ fn parse_input(input: &str) -> Vec<Vec<Dir>> {
                     _ => panic!("Invalid input!"),
                 })
                 .collect()
-    ).collect()
+        })
+        .collect()
 }
 
 fn solve<T: Keypad + Copy>(input: &str) -> Option<String> {
     let dirs = parse_input(input);
     let digit: T = T::init();
-    let res: Vec<char> = dirs.into_iter().scan(digit, |digit, ds| {
+    let res: Vec<char> = dirs
+        .into_iter()
+        .scan(digit, |digit, ds| {
             *digit = compute_digit(*digit, &ds);
             Some(digit.to_digit())
-    }).collect();
+        })
+        .collect();
     Some(String::from_iter(res))
 }
 
-
 pub fn part_one(input: &str) -> Option<String> {
     solve::<PosP1>(input)
 }

src/bin/03.rs

@@ -1,10 +1,7 @@
 advent_of_code::solution!(3);
 
 fn parse_line(input: &str) -> [u32; 3] {
-    let nb: Result<Vec<u32>, _> = input
+    let nb: Result<Vec<u32>, _> = input.split_whitespace().map(str::parse).collect();
-        .split_whitespace()
-        .map(str::parse)
-        .collect();
     if let Ok(nb) = nb {
         if nb.len() == 3 {
             return [nb[0], nb[1], nb[2]];
@@ -26,7 +23,7 @@ fn parse_input(input: &str) -> Vec<[u32; 3]> {
 fn parse_input_p2(input: &str) -> Vec<[u32; 3]> {
     parse_input(input)
         .chunks(3)
-        .map(|c| {
+        .flat_map(|c| {
             let v = c.to_vec();
             if v.len() != 3 {
                 println!("{:?}", v);
@@ -34,44 +31,23 @@ fn parse_input_p2(input: &str) -> Vec<[u32; 3]> {
             }
             let vs = [v[0], v[1], v[2]];
             // LAAAAAAZY... Could have transposed
-            let [
+            let [[ta1, tb1, tc1], [ta2, tb2, tc2], [ta3, tb3, tc3]] = vs;
-                [ta1, tb1, tc1],
+            [[ta1, ta2, ta3], [tb1, tb2, tb3], [tc1, tc2, tc3]].to_vec()
-                [ta2, tb2, tc2],
-                [ta3, tb3, tc3],
-            ] = vs;
-            [
-                [ta1, ta2, ta3],
-                [tb1, tb2, tb3],
-                [tc1, tc2, tc3],
-            ].to_vec()
         })
-        .flatten()
         .collect()
 }
 
 fn is_valid(triangle: &[u32; 3]) -> bool {
     let &[a, b, c] = triangle;
-    (a + b) > c
+    (a + b) > c && (b + c) > a && (c + a) > b
-        && (b + c) > a
-        && (c + a) > b
 }
 
 pub fn part_one(input: &str) -> Option<u32> {
-    Some(
+    Some(parse_input(input).into_iter().filter(is_valid).count() as u32)
-        parse_input(input)
-            .into_iter()
-            .filter(is_valid)
-            .count() as u32
-    )
 }
 
 pub fn part_two(input: &str) -> Option<u32> {
-    Some(
+    Some(parse_input_p2(input).into_iter().filter(is_valid).count() as u32)
-        parse_input_p2(input)
-            .into_iter()
-            .filter(is_valid)
-            .count() as u32
-    )
 }
 
 #[cfg(test)]