refactor: remove unwraps

though unlikely too panic coz we only compute median on numbers, still cleaner...
dathere · Dec 8, 2024 · a2c1597 · a2c1597
1 parent 4c5f28c
commit a2c1597
Showing 1 changed file with 4 additions and 4 deletions.
diff --git a/src/unsorted.rs b/src/unsorted.rs
@@ -114,15 +114,15 @@ where
 {
     Some(match data.len() {
         0 => return None,
-        1 => data.first()?.to_f64().unwrap(),
+        1 => data.first()?.to_f64()?,
         len if len % 2 == 0 => {
             let idx = len / 2;
-            let v1 = data.get(idx - 1)?.to_f64().unwrap();
-            let v2 = data.get(idx)?.to_f64().unwrap();
+            let v1 = data.get(idx - 1)?.to_f64()?;
+            let v2 = data.get(idx)?.to_f64()?;
             (v1 + v2) / 2.0
         }
         // Safety: we know the index is within bounds
-        len => unsafe { data.get_unchecked(len / 2) }.to_f64().unwrap(),
+        len => unsafe { data.get_unchecked(len / 2) }.to_f64()?,
     })
 }